Skip to content

Refactor component containers#3129

Open
mini-1235 wants to merge 3 commits intoros2:rollingfrom
mini-1235:refactor/component_containers
Open

Refactor component containers#3129
mini-1235 wants to merge 3 commits intoros2:rollingfrom
mini-1235:refactor/component_containers

Conversation

@mini-1235
Copy link
Copy Markdown
Contributor

Description

Supersedes #3055

Is this user-facing behavior change?

Yes,

  • When users use component_container_mt/component_container_event/component_container_isolated, they will see a warning message
  • When using component_container without any arguments, it will default to non isolated + single threaded, so users will probably not notice this change

Did you use Generative AI?

Yes, Github Copilot

Additional Information

Signed-off-by: mini-1235 <mauricepurnawan@gmail.com>
Signed-off-by: mini-1235 <mauricepurnawan@gmail.com>
@skyegalaxy
Copy link
Copy Markdown
Member

Would we be able to also include component containers for the new callback group events executor? I had a draft PR to do so but it was blocked by several cascading PRs waiting on green CI to get in

@mini-1235
Copy link
Copy Markdown
Contributor Author

Sure :) I didn't include it here since this PR is mainly focused on refactoring the existing component containers. Should I go ahead and include it in this one?

@mini-1235
Copy link
Copy Markdown
Contributor Author

I took a look through the core repositories, and I don't think anything is currently using component_container_mt/event/isolated, the only thing that seems to need updating is ros2/ros2_documentation#6401

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