-
Notifications
You must be signed in to change notification settings - Fork 531
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
Ports moveit/moveit/pull/3519 to ros2 (backport #3055) #3062
base: iron
Are you sure you want to change the base?
Conversation
* Ports moveit/moveit/pull/3519 to ros2 * Applies formatting * Fixes compilation errors * Fixes compilation errors * Fixes compilation errors * Fixes compilation errors * Replaces RobotState with pointer in tests * Fixes tests (cherry picked from commit 70e1aae) # Conflicts: # moveit_planners/pilz_industrial_motion_planner/include/pilz_industrial_motion_planner/planning_context_base.h # moveit_planners/pilz_industrial_motion_planner/src/trajectory_functions.cpp # moveit_planners/pilz_industrial_motion_planner/src/trajectory_generator.cpp # moveit_planners/pilz_industrial_motion_planner/test/unit_tests/src/unittest_trajectory_generator_circ.cpp # moveit_planners/pilz_industrial_motion_planner/test/unit_tests/src/unittest_trajectory_generator_lin.cpp
Cherry-pick of 70e1aae has failed:
To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
@sjahr Any idea what might've caused these failures? |
@sjahr Could the jobs be rerun on this one? I think they may be unrelated |
Ah, the Iron branch has been failing for a while... very likely not your doing. I'm starting to think since Iron EOL is in a week or 2, it may not be worth fixing this branch. |
Yeah happy to close once iron reaches EOL |
FYI - we've already missed the window for another Iron deb release - https://discourse.ros.org/t/preparing-for-final-iron-irwini-patch-release-and-sync-2024-11-18/40619 |
Yep! I was just poking around to see how easy it would be to fix source builds. Thanks for the heads up though. |
Description
Ports moveit/moveit#3519 to ROS2. This may partially fix #2857 for the LIN planner
Checklist
This is an automatic backport of pull request #3055 done by [Mergify](https://mergify.com).