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

Feature proposal applySubtreeVisitor() #737

Closed
galou opened this issue Jan 9, 2024 · 2 comments
Closed

Feature proposal applySubtreeVisitor() #737

galou opened this issue Jan 9, 2024 · 2 comments
Labels
wontfix This will not be worked on

Comments

@galou
Copy link
Contributor

galou commented Jan 9, 2024

In order to allow the combination of C++ (action servers) and BehaviorTree, we preferred to rely on copying the required C++ variables to each subtree's blackboard rather than using _autoremap. My solution was to implement applySubtreeVisitor, cf. 0843158. This cannot be merged as-is because I remove the applyVisitor function rather than deprecating it but this is easy to change.

@facontidavide what do you think about this?

@facontidavide
Copy link
Collaborator

not sure about this design pattern, in general

@facontidavide facontidavide added the wontfix This will not be worked on label May 20, 2024
@facontidavide
Copy link
Collaborator

not doing this, for the time being

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants