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 task_name to InitializeMTCTask #31

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DaniGarciaLopez
Copy link

Allowing to set the name of the task from the tree. Specially useful when you're planning multiple tasks in sequence.

@captain-yoshi
Copy link
Owner

Yes very useful.

Just waiting the discussion to see if we could use name instead of task_name. BehaviorTree/BehaviorTree.CPP#901

@DaniGarciaLopez
Copy link
Author

I agree. I also think using just name is better. This change would also be beneficial for stage_name and container_name.

However, I suppose a change like this would make the btcpp API incompatible, requiring a new version to be released, similar to what happened with the ports order. I think it will take some time to be ready.

Let's see if Davide can reply to us, as I haven’t seen him very active in the Discussions tab lately.

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