Add behavior trees #167

Merged
chris merged 2 commits from refs/pull/167/head into main 2025-12-09 10:54:46 +01:00
chris commented 2025-12-08 22:59:18 +01:00 (Migrated from git.sopranium.de)
No description provided.
johannesp (Migrated from git.sopranium.de) reviewed 2025-12-08 22:59:18 +01:00
johannesp (Migrated from git.sopranium.de) approved these changes 2025-12-09 10:22:45 +01:00
johannesp (Migrated from git.sopranium.de) left a comment

Es gibt kleinere Änderungsvorschläge, die aber auf zukünftige PRs postponed werden können.

Also fürs erste LGTM

Es gibt kleinere Änderungsvorschläge, die aber auf zukünftige PRs postponed werden können. Also fürs erste **LGTM**
@ -158,3 +62,2 @@
);
_bt = new BehaviorTree(
johannesp (Migrated from git.sopranium.de) commented 2025-12-09 10:27:56 +01:00

Implementierung gefällt mir soweit gut, ich finde aber die Aktionen sollten im Behavior stehen und nicht im BehaviorTree als Nodes definiert werden.

Implementierung gefällt mir soweit gut, ich finde aber die Aktionen sollten im Behavior stehen und nicht im BehaviorTree als Nodes definiert werden.
johannesp (Migrated from git.sopranium.de) commented 2025-12-09 10:23:12 +01:00

Für den Moment ok, aber sollte in Zukunft refactored werden

Für den Moment ok, aber sollte in Zukunft refactored werden
johannesp (Migrated from git.sopranium.de) commented 2025-12-09 10:23:43 +01:00

Ebenfalls noch ok für jetzt, aber sollte bald umgeschrieben werden

Ebenfalls noch ok für jetzt, aber sollte bald umgeschrieben werden
@ -0,0 +3,4 @@
namespace Cooldown.Entities.Intelligence.Nodes.Custom;
public class FlyTowardsNode(Entity referenceEntity, float speed, Wrapper<Entity> targetWrapper) : BtNode
johannesp (Migrated from git.sopranium.de) commented 2025-12-09 10:25:59 +01:00

Sollten Aktionen den selben Bereich wie Behaviors erfüllen?

Sollten Aktionen den selben Bereich wie Behaviors erfüllen?
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
ufr/sopra10!167
No description provided.