search

BehaviorTree / Groot

Graphical Editor to create BehaviorTrees. Compliant with BehaviorTree.CPP
726 stars 237 forks source link

Support subtree include #42

Open doisyg opened 5 years ago

doisyg commented 5 years ago

Groot currently doesn't support trees referencing subtree through the include tag, even the simplest form when the second xml is in the same folder or specified with the full absolute path. A tab is open for the subtree referenced but it is empty.

ramilmsh commented 5 years ago

@facontidavide, any progress or thoughts?

kb0n commented 3 years ago

bumping this, would be nice to see progress here. Is there any? :)

I honestly cannot fathom why Groot has its own implementation of XML-parsing rather than letting BehaviorTree.CPP do that part and just use the resulting tree(s). This is doomed to always run into problems sooner or later.

facontidavide commented 3 years ago

dear @kb0n , I don't have bandidth to support Groot anymore.

be my quest and submit a pull request

kb0n commented 3 years ago

The usual problem we all face unfortunately :)

I'll have the evaluate the urgency for us and see if can make up some capacities for tackling this. Was there a particular reason for Groot to use its own parser? Just so I can avoid running into the same problems you already faced. Thanks!

facontidavide commented 3 years ago

The main reason is that the original parser is use to create an instance of a Tree.

Groot is NOT supposed to create an instance of the tree, therefore I don't see how code reuse is even possible (spoiler alert: is not).