Learning about parameters - tutorial - #7031
Conversation
…om/3di-for-openrobotics/ros2_documentation into learning-about-parameters-tutorial
…arameters-tutorial
kscottz
left a comment
There was a problem hiding this comment.
This is fine as is. My only concern is that if we are touching this tutorial we really should add a section that asks the user to run ros2 param --help. Most of the intro tutorials really should be showing users how they can help themselves figure out a command.
|
Tick the box to add this pull request to the merge queue (same as
|
fujitatomoya
left a comment
There was a problem hiding this comment.
lgtm with green CI.
|
@kacperbojakowski-3di thanks for the PR, all the backports to the downstream branches have some conflicts. can you resolve those? |
|
Backport conflicts resolved via fresh PRs (no write access to push onto the Mergify branches):
Each keeps only the parameters tutorial rewrite on the |
Hi @fujitatomoya, Thanks! Sorry for the noise with extra requests, but apparently I don’t have the write access to push onto Mergify branches, so instead I opened four clean replacement backports:
You can merge those, and then the conflicted Mergify PRs can be closed. Unless I should have handled this differently, then let me know. The big caveat here is that I have resolved these with the assumption that the structure on lyrical, kilted, jazzy and humble stays as is, but eventually the topic is going to change its position once #6998 is backported. @dmarshant FYI, not sure how you would like to go about this. |
|
@kacperbojakowski-3di thanks for creating backports. before moving on to review indivisual backport PRs, something i want to confirm here. i think we need to backport #6998 in lyrical (and downstream branches) in the 1st place? that said, we can avoid those conflicts during brackports. to keep the consistency and maitainability (this is exactly why the conflicts are happening), i say we need to backport #6998 to the downstream branches in the 1st. what is your plan with OSRA ROS TGC? |
Updated the article "Learning about parameters - tutorial".
This update is part of #6827.
Description
Related to: ros2/ros2#1725
ros2 param getworks only on Lyrical, Rolling, and later.Did you use Generative AI?
Yes, GPT-5 for the FAQ section.
Additional Information
Documentation updates from 3di Information Solutions, agreed with Geoff and Tully.
Label
Please add backport-lyrical.