Parse xacro args from .setup_assistant config in MoveIt Configs Builder#2172
Parse xacro args from .setup_assistant config in MoveIt Configs Builder#2172JafarAbdi merged 6 commits intomoveit:mainfrom
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #2172 +/- ##
==========================================
- Coverage 50.49% 50.47% -0.02%
==========================================
Files 387 387
Lines 31790 31790
==========================================
- Hits 16050 16043 -7
- Misses 15740 15747 +7 ☔ View full report in Codecov by Sentry. |
moveit_configs_utils/moveit_configs_utils/moveit_configs_builder.py
Outdated
Show resolved
Hide resolved
|
Servo tests are flaky, I just retriggered CI |
Gotcha, thanks! |
| self.__robot_description: load_xacro( | ||
| robot_description_file_path, mappings=mappings | ||
| robot_description_file_path, | ||
| mappings=mappings or self.__urdf_xacro_args, |
There was a problem hiding this comment.
Why do we need this or when you assigned mappings on L226 (does 226 need to be removed like Jafar suggested)?
There was a problem hiding this comment.
Yeah, I think that line needs to be removed
There was a problem hiding this comment.
The assignment should be removed now.
There was a problem hiding this comment.
I thought I cut it out of there but I must have accidentally undo'd that change. Thank you @JafarAbdi for cleaning that up for me
moveit_configs_utils/moveit_configs_utils/moveit_configs_builder.py
Outdated
Show resolved
Hide resolved
MarqRazz
left a comment
There was a problem hiding this comment.
I was not able to test these changes but they look good to me.
* Parse xacro args from .setup_assistant config in MoveIt Configs Builder (#2172) Co-authored-by: Jafar <jafar.uruc@gmail.com> (cherry picked from commit 79a2fa5) * Fix xacro args loading issue (#2684) * Fixed xacro args loading issue * Formatting fixes with pre-commit action --------- (cherry picked from commit cdb20ae) --------- Co-authored-by: Anthony Baker <abake48@users.noreply.github.com>
Description
Update MoveIt Config Builder to parse xacro_args parameter from MSA generated MoveIt 2 configs that use xacro generated robot descriptions.
Checklist