-
Notifications
You must be signed in to change notification settings - Fork 490
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
[MSA] Existing Package Loading Tweaks #1212
Conversation
Please target the |
Codecov Report
@@ Coverage Diff @@
## feature/msa #1212 +/- ##
===============================================
- Coverage 61.02% 59.80% -1.22%
===============================================
Files 275 275
Lines 23761 23761
===============================================
- Hits 14498 14207 -291
- Misses 9263 9554 +291
Continue to review full report at Codecov.
|
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.
I think this makes sense, I did something similar on my msa branch to test out things. Thanks a lot!
Description
The key change in this PR is basically one line:
which sets the package name for the MoveIt config to be the existing package name when loading from an existing MoveIt config. Easy enough....
...except
new_package_name_
was also set insetPackagePath
, called automatically from theConfigurationFilesWidget
. But that caused problems because the value of the package path was automatically derived from the existing package path, which means that thenew_package_name_
would not reflect the actual package name, but instead reflect the name of the folder.This is most notable with the
moveit_resources
where the folder ispanda_moveit_config
but the actual package name ismoveit_resources_panda_moveit_config
Anyway, so this PR splits the functionality into two distinct paths, one for setting the package path, and one for setting the package name.
Checklist