Skip to content

Conversation

@riv-tnoble
Copy link
Contributor

@riv-tnoble riv-tnoble commented Nov 8, 2024

Description

Ports moveit/moveit#3522 to moveit2. Should fix #2857

Checklist

  • Required by CI: Code is auto formatted using clang-format
  • Extend the tutorials / documentation reference (fixes bug)
  • Document API changes relevant to the user in the MIGRATION.md notes (fixes bug)
  • Create tests, which fail without this PR reference
  • Include a screenshot if changing a GUI (No GUI changes)
  • While waiting for someone to review your request, please help review another open pull request to support the maintainers

@mergify
Copy link

mergify bot commented Nov 8, 2024

This pull request is in conflict. Could you fix it @rr-tom-noble?

@riv-tnoble
Copy link
Contributor Author

@sjahr I believe the remaining failures are unrelated to this PR (gz_ros2_control install, robotiq_driver compile error, failing panda kdl test).

Additionally, it'd be great if this could be marked for backporting to humble. I'll be able to test that #2857 is fixed.

Copy link
Contributor

@sjahr sjahr left a comment

Choose a reason for hiding this comment

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

Thanks! Just FYI: The failing test cases are expected to fail (unfortunately) so don't mind them in future PRs.

@sjahr sjahr added backport-humble Mergify label that triggers a PR backport to Humble backport-iron Mergify label that triggers a PR backport to Iron labels Nov 8, 2024
@sjahr sjahr added this pull request to the merge queue Nov 8, 2024
Merged via the queue into moveit:main with commit 4c28a74 Nov 8, 2024
11 of 13 checks passed
mergify bot pushed a commit that referenced this pull request Nov 8, 2024
* Makes PTP changes

* Makes LIN changes

* Makes CIRC changes

* Makes CIRC changes

* Makes CIRC changes

* Makes CIRC changes

* Makes LIN changes

* Makes LIN changes

* Fixes formatting

* Replace ROS_WARN with RCLCPP_WARN

* Fixes formatting

* Fixes RCLCPP_WARN calls

* Fixes compiler errors

* Fixes if statement

* Fixes formatting

* Fixes msg type

(cherry picked from commit 4c28a74)
mergify bot pushed a commit that referenced this pull request Nov 8, 2024
* Makes PTP changes

* Makes LIN changes

* Makes CIRC changes

* Makes CIRC changes

* Makes CIRC changes

* Makes CIRC changes

* Makes LIN changes

* Makes LIN changes

* Fixes formatting

* Replace ROS_WARN with RCLCPP_WARN

* Fixes formatting

* Fixes RCLCPP_WARN calls

* Fixes compiler errors

* Fixes if statement

* Fixes formatting

* Fixes msg type

(cherry picked from commit 4c28a74)

# Conflicts:
#	moveit_planners/pilz_industrial_motion_planner/src/trajectory_generator_circ.cpp
#	moveit_planners/pilz_industrial_motion_planner/src/trajectory_generator_lin.cpp
@riv-tnoble riv-tnoble deleted the enhancement/ports-moveit-3522 branch November 24, 2024 08:47
Markus-Simonsen pushed a commit to Markus-Simonsen/moveit2 that referenced this pull request Aug 12, 2025
* Makes PTP changes

* Makes LIN changes

* Makes CIRC changes

* Makes CIRC changes

* Makes CIRC changes

* Makes CIRC changes

* Makes LIN changes

* Makes LIN changes

* Fixes formatting

* Replace ROS_WARN with RCLCPP_WARN

* Fixes formatting

* Fixes RCLCPP_WARN calls

* Fixes compiler errors

* Fixes if statement

* Fixes formatting

* Fixes msg type
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-humble Mergify label that triggers a PR backport to Humble backport-iron Mergify label that triggers a PR backport to Iron

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PILZ Industrial motion planner is ignoring the goal frame_id passed

2 participants