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

[Example 15] Using multiple controller managers under different namespaces #423

Merged
merged 8 commits into from
May 28, 2024

Conversation

christophfroehlich
Copy link
Contributor

@christophfroehlich christophfroehlich commented Dec 25, 2023

A rewrite of #170 including StoglRobotics-forks#8

An open questions for me:

  • It uses the rrbot_base.launch.py: Should we move it to the description package? I deleted it once for the other examples for sake of simplicity. But here it makes sense IMHO.

Copy link
Member

@saikishor saikishor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. The tutorial works too. Just some minor changes.

Thank you

README.md Outdated Show resolved Hide resolved
doc/index.rst Outdated Show resolved Hide resolved
doc/index.rst Outdated Show resolved Hide resolved
example_15/README.md Outdated Show resolved Hide resolved
example_15/doc/userdoc.rst Outdated Show resolved Hide resolved
example_15/doc/userdoc.rst Outdated Show resolved Hide resolved
example_15/doc/userdoc.rst Outdated Show resolved Hide resolved
example_15/doc/userdoc.rst Outdated Show resolved Hide resolved
Copy link
Contributor

mergify bot commented Feb 18, 2024

This pull request is in conflict. Could you fix it @christophfroehlich?

Copy link
Collaborator

@olivier-stasse olivier-stasse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for this great example.
Tested on my machine (22.04 LTS + head of ros2_control and ros2_controllers)

@olivier-stasse
Copy link
Collaborator

I don't understand which change is missing from @saikishor review.

@christophfroehlich
Copy link
Contributor Author

I don't understand which change is missing from @saikishor review.

Thanks for your review. I addressed his comments, only the final approval was missing.

@christophfroehlich christophfroehlich merged commit a97760f into ros-controls:master May 28, 2024
12 checks passed
@christophfroehlich christophfroehlich deleted the rebase_170 branch May 28, 2024 19:01
mergify bot pushed a commit that referenced this pull request May 28, 2024
…paces (#423)

* Add example15 to workflows+packages

* Initial commit from old PR

Co-authored-by: Denis Štogl <destogl@users.noreply.github.com>

* Convert md to rst syntax

* Apply suggestions from code review

Co-authored-by: Sai Kishor Kothakota <saisastra3@gmail.com>

---------

Co-authored-by: Denis Štogl <destogl@users.noreply.github.com>
Co-authored-by: Sai Kishor Kothakota <saisastra3@gmail.com>
(cherry picked from commit a97760f)
mergify bot pushed a commit that referenced this pull request May 28, 2024
…paces (#423)

* Add example15 to workflows+packages

* Initial commit from old PR

Co-authored-by: Denis Štogl <destogl@users.noreply.github.com>

* Convert md to rst syntax

* Apply suggestions from code review

Co-authored-by: Sai Kishor Kothakota <saisastra3@gmail.com>

---------

Co-authored-by: Denis Štogl <destogl@users.noreply.github.com>
Co-authored-by: Sai Kishor Kothakota <saisastra3@gmail.com>
(cherry picked from commit a97760f)
christophfroehlich added a commit that referenced this pull request May 28, 2024
…paces (#423) (#527)

* Add example15 to workflows+packages

* Initial commit from old PR

Co-authored-by: Denis Štogl <destogl@users.noreply.github.com>

* Convert md to rst syntax

* Apply suggestions from code review

Co-authored-by: Sai Kishor Kothakota <saisastra3@gmail.com>

---------

Co-authored-by: Denis Štogl <destogl@users.noreply.github.com>
Co-authored-by: Sai Kishor Kothakota <saisastra3@gmail.com>
(cherry picked from commit a97760f)

Co-authored-by: Christoph Fröhlich <christophfroehlich@users.noreply.github.com>
christophfroehlich added a commit that referenced this pull request May 28, 2024
…paces (#423) (#526)

* Add example15 to workflows+packages

* Initial commit from old PR

Co-authored-by: Denis Štogl <destogl@users.noreply.github.com>

* Convert md to rst syntax

* Apply suggestions from code review

Co-authored-by: Sai Kishor Kothakota <saisastra3@gmail.com>

---------

Co-authored-by: Denis Štogl <destogl@users.noreply.github.com>
Co-authored-by: Sai Kishor Kothakota <saisastra3@gmail.com>
(cherry picked from commit a97760f)

Co-authored-by: Christoph Fröhlich <christophfroehlich@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants