Skip to content

[Servo] Restore namespace to parameters#2171

Merged
sjahr merged 2 commits intomoveit:mainfrom
ibrahiminfinite:fix_servo_namespace
May 11, 2023
Merged

[Servo] Restore namespace to parameters#2171
sjahr merged 2 commits intomoveit:mainfrom
ibrahiminfinite:fix_servo_namespace

Conversation

@ibrahiminfinite
Copy link
Copy Markdown
Contributor

@ibrahiminfinite ibrahiminfinite commented May 11, 2023

Description

Fixes #2162
This PR adds the moveit_servo namespace to MoveIt Servo parameters.

Checklist

@ibrahiminfinite
Copy link
Copy Markdown
Contributor Author

Pinging @tylerjw and @sea-bass for review.

@codecov
Copy link
Copy Markdown

codecov bot commented May 11, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (e3539be) 50.48% compared to head (217d25b) 50.47%.

❗ Current head 217d25b differs from pull request most recent head 72ae89e. Consider uploading reports for the commit 72ae89e to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2171      +/-   ##
==========================================
- Coverage   50.48%   50.47%   -0.00%     
==========================================
  Files         387      387              
  Lines       31790    31790              
==========================================
- Hits        16047    16044       -3     
- Misses      15743    15746       +3     
Impacted Files Coverage Δ
moveit_ros/moveit_servo/src/servo_node.cpp 75.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Copy Markdown
Contributor

@sea-bass sea-bass left a comment

Choose a reason for hiding this comment

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

I did mean to approve the PR, since my comment is minor

@sjahr sjahr merged commit f807733 into moveit:main May 11, 2023
sjahr pushed a commit to PickNikRobotics/moveit2 that referenced this pull request May 12, 2023
* Add  namespace to parameters

* Minor cleanups
@ibrahiminfinite ibrahiminfinite deleted the fix_servo_namespace branch June 3, 2023 14:15
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.

Restore namespace in MoveIt Servo parameters

3 participants