Skip to content
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

Open
fnawratil opened this issue Dec 1, 2017 · 6 comments
Open

EntityComponents aus der Octoawesome.dll entfernen #208

fnawratil opened this issue Dec 1, 2017 · 6 comments
Assignees
Labels
suggestion A suggestion for a new feature / function To Do Next to be processed by the community

Comments

@fnawratil
Copy link
Member

No description provided.

@Gallimathias Gallimathias added this to the Clean Up milestone Dec 28, 2017
@Gallimathias Gallimathias self-assigned this Dec 28, 2017
@Gallimathias Gallimathias added the input needed It waits for the answer to queries. label Dec 28, 2017
@Gallimathias
Copy link
Member

@fnawratil Was willst du uns damit sagen?

Bitte mal ne ordentliche Beschreibung einfügen oder selber machen und einen pull request machen

@XYZLassi
Copy link
Member

Komponenten müssten in eine eigene DLL verschoben werden wie z.B. Plugin.
Es geht mit #207 einher.

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.

@Gallimathias Gallimathias removed this from the Clean Up milestone Dec 28, 2017
@Gallimathias Gallimathias added suggestion A suggestion for a new feature / function and removed input needed It waits for the answer to queries. labels Dec 28, 2017
@Gallimathias Gallimathias added the To Do Next to be processed by the community label Dec 29, 2017
@HierGibtEsDrachen
Copy link

HierGibtEsDrachen commented Feb 21, 2018

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.
Wäre für eine komplette Überarbeitung der PhysikEngine Architektur :D
oder übersehe ich etwas, dass dann nicht mehr geht wenn die Add und Remove Funktionalität der Componenten (für die Berechnungen -> Calc(entity, gametime)) entfernt wird. Sowie eine andere Basisklasse.
Btw. eine Entity ist ein selbstständiges Objekt und somit sollte die Unterscheidung zwischen UpdatableEntity und Entity wegfallen...

@ManuelHu
Copy link
Member

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 OctoAwesome.dll befinden, da sie vom Client aus referenziert werden. Andere Entity-Komponenten liegen aber in OctoAwesome.Basics.dll, da diese nur von Simulationskomponenten aus der Basics-Erweiterung verwendet werden.

Btw. eine Entity ist ein selbstständiges Objekt und somit sollte die Unterscheidung zwischen UpdatableEntity und Entity wegfallen...

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!

Wäre für eine komplette Überarbeitung der PhysikEngine Architektur :D

Mach bitte ein Issue mit Vorschlägen dazu auf, wenn du dich dort einbringen willst!

@HierGibtEsDrachen
Copy link

Verweise auf #241, könnte hiermit geschlossen werden.

@Gallimathias
Copy link
Member

#241 und dieses Issue sind glaube ich ganz gut in zwei verschiedenen Issues aufgehoben @HierGibtEsDrachen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
suggestion A suggestion for a new feature / function To Do Next to be processed by the community
Projects
None yet
Development

No branches or pull requests

5 participants