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

Add the possibility to remove a blackboard entry. #709

Merged
merged 2 commits into from
Dec 19, 2023

Conversation

galou
Copy link
Contributor

@galou galou commented Dec 5, 2023

Add a node to use this functionality.

Add a node to use this functionality.

Signed-off-by: Gaël Écorchard <gael@km-robotics.cz>
@galou
Copy link
Contributor Author

galou commented Dec 5, 2023

This is related to #706 . I need the ability to have "local" variables. Instead of variable scope, I can now delete some blackboard entries at the end of my subtree so that I have a clean blackboard in the case that the subtree is called again.

@facontidavide
Copy link
Collaborator

this looks useful.
See my suggested change in the copyright. After that fix, I will merge it

@galou
Copy link
Contributor Author

galou commented Dec 6, 2023

Sorry, I don't see the suggested change.

@facontidavide
Copy link
Collaborator

I was able to commit that myself. Merging as soon as Ci turns green

@corot
Copy link

corot commented Dec 16, 2023

Agree this looks useful. But wouldn't be neater to use SetBackboard with an empty input?
Would be that possible?
🤔

@facontidavide facontidavide merged commit b0dd2b2 into BehaviorTree:master Dec 19, 2023
7 checks passed
facontidavide added a commit that referenced this pull request Dec 19, 2023
@facontidavide
Copy link
Collaborator

merged with some changes. Thanks

@galou galou deleted the unset_blackboard branch December 22, 2023 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants