-
Notifications
You must be signed in to change notification settings - Fork 160
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
Foxy backport Add rosparam verb load (#590) #596
Conversation
* Add rosparam verb load Signed-off-by: Victor Lopez <victor.lopez@pal-robotics.com> * Move load_parameter_file implementation to api, add test_verb_load Signed-off-by: Victor Lopez <victor.lopez@pal-robotics.com> * Apply fixes for linter Signed-off-by: Victor Lopez <victor.lopez@pal-robotics.com> * Remove TODO comment Signed-off-by: Victor Lopez <victor.lopez@pal-robotics.com> * Fix linter errors Signed-off-by: Audrow Nash <audrow.nash@gmail.com> Co-authored-by: Audrow Nash <audrow.nash@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
Let's wait to merge this in until another PR has been made addressing wildcard support (mentioned in #590), and we can merge them at the same time, in order.
Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
Signed-off-by: Victor Lopez <victor.lopez@pal-robotics.com>
@audrow I've updated the merge request with what was merged on master. I've had to cherry-pick e91f9bd from @ivanpauno as well for consistency. |
Hey guys, can we please move ahead with this one as I can see it's been greenlighted by all parties involved? This (and some PRs of us depending on it) would significantly improve some quality of life for |
Agreed, would you like want me to create both backports PR? |
I'm more inclined to this, as those PRs include related fixes. |
I think we already added them to this PR, right? On Feb 23rd. |
Oh sorry 😂. Last CI looked good, windows job was yellow because of unrelated reasons. |
Thanks! Do you have an ETA for the foxy 0.9.9 release? |
IDK, last foxy sync was a week ago, so I guess next one will be in about a month. |
Signed-off-by: Victor Lopez victor.lopez@pal-robotics.com
Signed-off-by: Victor Lopez victor.lopez@pal-robotics.com
Signed-off-by: Victor Lopez victor.lopez@pal-robotics.com
Signed-off-by: Victor Lopez victor.lopez@pal-robotics.com
Signed-off-by: Audrow Nash audrow.nash@gmail.com
Co-authored-by: Audrow Nash audrow.nash@gmail.com