Bug fix/set blackboard by devis12 · Pull Request #955 · BehaviorTree/BehaviorTree.CPP · GitHub
Skip to content

Bug fix/set blackboard#955

Merged
facontidavide merged 5 commits into
BehaviorTree:masterfrom
devis12:bug-fix/set_blackboard
Apr 16, 2025
Merged

Bug fix/set blackboard#955
facontidavide merged 5 commits into
BehaviorTree:masterfrom
devis12:bug-fix/set_blackboard

Conversation

@devis12

@devis12 devis12 commented Apr 1, 2025

Copy link
Copy Markdown
Contributor

SetBlackboard action bug fix

Previous SetBlackboard action node did not update the sequence id, the timestamp when called on a pre-existing entry.
Moreover, it allowed to bypass type checking.

This occurred only when the input was a blackboard pointer.

By using the BT::Blackboard::set method I believe also is thread safe which before was not, but I provided test that were failing previously and now are passing exclusively for the two cases above

Devis Dal Moro and others added 3 commits March 31, 2025 12:37
Previous SetBlackboard action node did not update the sequence id, the timestamp when called on a pre-existing entry.
Moreover, it allowed to bypass type checking.
@devis12

devis12 commented Apr 7, 2025

Copy link
Copy Markdown
Contributor Author

root added 2 commits April 7, 2025 14:33
Made sure to handle possible type inconsistencies which might arise when using port remapping with subtrees.
@facontidavide facontidavide merged commit 3651a24 into BehaviorTree:master Apr 16, 2025
@facontidavide

Copy link
Copy Markdown
Collaborator

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.

2 participants