Skip to content

Use moveit_configs_utils for launch files#365

Merged
JafarAbdi merged 5 commits intomoveit:ros2from
JafarAbdi:pr-moveit_configs_utils
May 26, 2022
Merged

Use moveit_configs_utils for launch files#365
JafarAbdi merged 5 commits intomoveit:ros2from
JafarAbdi:pr-moveit_configs_utils

Conversation

@JafarAbdi
Copy link
Member

@JafarAbdi JafarAbdi commented May 25, 2022

NOTE: you need moveit/moveit2#1274 and PickNikRobotics/rosparam_shortcuts#27 to be able to run the demos

@codecov
Copy link

codecov bot commented May 25, 2022

Codecov Report

Merging #365 (b7fc895) into ros2 (9908086) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             ros2     #365   +/-   ##
=======================================
  Coverage   38.85%   38.85%           
=======================================
  Files          78       78           
  Lines        7520     7520           
=======================================
  Hits         2921     2921           
  Misses       4599     4599           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9908086...b7fc895. Read the comment docs.

Copy link
Contributor

@stephanie-eng stephanie-eng left a comment

Choose a reason for hiding this comment

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

I found a few imports I think we can clean up

Copy link
Contributor

@stephanie-eng stephanie-eng left a comment

Choose a reason for hiding this comment

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

Sorry about the above mix-up, tested with the the two PR's you linked and all works!

Copy link
Member

@henningkayser henningkayser left a comment

Choose a reason for hiding this comment

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

This is such a huge improvement!

@JafarAbdi JafarAbdi merged commit 0128cd9 into moveit:ros2 May 26, 2022
@JafarAbdi JafarAbdi deleted the pr-moveit_configs_utils branch May 26, 2022 16:29
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.

3 participants