-
Notifications
You must be signed in to change notification settings - Fork 31
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
EntityComponents aus der Octoawesome.dll entfernen #208
Comments
@fnawratil Was willst du uns damit sagen? Bitte mal ne ordentliche Beschreibung einfügen oder selber machen und einen pull request machen |
Komponenten müssten in eine eigene DLL verschoben werden wie z.B. Plugin. Das Problem ist die Komponenten gehören eigentlich in Basics dürfen aber bis jetzt nicht vom Client referenziert werden, weil das ein Mod ist. Dafür müsste eigentlich die Architektur umgearbeitet werden. |
Soweit ich mich richtig erinnere war die Basics von Tom persönlich als Extension gedacht und die Octoawesom.dll als Library für alles. Find es nicht sinnvoll eine eigene Library für Extensions zu schreiben wenn in Octoawesom.dll alles drin ist. |
Erstmal herzlich willkommen hier bei uns auf GitHub ;-) Ich weiß nicht, ob du hier etwas verwechselst. Die Add- und Remove-Methoden befinden sich in den Simulationskomponenten, die durch dieses Issue gar nicht tangiert werden. Die EntityComponents sind halten in der Regel nur Daten und weisen einem Spielobjekt geisse Attribute zu (z.B. "hat Masse X", "hat Abmessungen", "hat ein Inventar"). Einige stören sich daran, dass diese sich in der
Darüber lässt sich sicher diskutieren (z.B. CPU-Last bei Objekten die eigentlich gar nichts tun). Ist aber Thema für ein eigenes Issue, nicht zögern eins aufzumachen!
Mach bitte ein Issue mit Vorschlägen dazu auf, wenn du dich dort einbringen willst! |
Verweise auf #241, könnte hiermit geschlossen werden. |
#241 und dieses Issue sind glaube ich ganz gut in zwei verschiedenen Issues aufgehoben @HierGibtEsDrachen |
No description provided.
The text was updated successfully, but these errors were encountered: