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.

Sign up for GitHub

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

Jump to bottom

[trees] optional node, node name arguments to the tree setup #188

Merged
merged 1 commit into from
Jan 1, 2023
Merged

[trees] optional node, node name arguments to the tree setup #188

merged 1 commit into from
Jan 1, 2023

Conversation

stonier
Copy link
Member

@stonier stonier commented Jan 1, 2023 •
edited
Loading

@stonier stonier added this to the 2.2 - Humble milestone Jan 1, 2023
@stonier stonier self-assigned this Jan 1, 2023
@stonier
Copy link
Member Author

stonier commented Jan 1, 2023 •
edited
Loading

  • I need to get some full tree tests happening in this repo → Tree tests #189.
  • Verified that py-trees-blackboard-watcher still discovers it automatically, despite blackboard services moving:
/foo/blackboard_streams/close
/foo/blackboard_streams/get_variables
/foo/blackboard_streams/open
/foo/describe_parameters

@stonier stonier merged commit 113036b into devel Jan 1, 2023
@stonier stonier deleted the stonier/optional_node_name branch January 1, 2023 03:49
This was referenced Jan 1, 2023
Closed
Closed
@stonier stonier mentioned this pull request Feb 4, 2023
Open
61 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@stonier stonier

Labels
component:trees type:enhancement
Projects
None yet
Milestone
2.2 - Humble
Development

Successfully merging this pull request may close these issues.

Consider allowing passing existing ROS2 Node object to setup() method of py_trees_ros.trees.BehaviourTree
1 participant
@stonier