Skip to content

Refactor component containers#3129

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

Refactor component containers#3129
mini-1235 wants to merge 2 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

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