-
Notifications
You must be signed in to change notification settings - Fork 566
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
Fix backport for #3227 #3282
base: mergify/bp/humble/pr-3227
Are you sure you want to change the base?
Fix backport for #3227 #3282
Conversation
This pull request is in conflict. Could you fix it @pac48? |
Please target the |
This pull request is in conflict. Could you fix it @pac48? |
6ba52b6
to
42b9ea2
Compare
Signed-off-by: Paul Gesel <paul.gesel@picknik.ai>
42b9ea2
to
14dbd17
Compare
Thanks @pac48! Seems the build is failing due to targets that don't exist on the humble branch. Might need to remove those. https://github.com/moveit/moveit2/actions/runs/13039259023/job/36377097965?pr=3282#step:11:301 |
moveit_butterworth_filter_parameters | ||
moveit_ruckig_filter | ||
moveit_ruckig_filter_parameters | ||
TARGETS |
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.
OK so here the acceleration and ruckig filters are not in the humble branch
moveit_kinematics_metrics | ||
moveit_macros |
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.
This doesn't seem to be on Humble either
Okay, I’ll take another look soon. The merge is definitely not trivia in this case due to all of the differences. |
No description provided.