Skip to content

Add RoboPlan packages - #38

Open
sea-bass wants to merge 8 commits into
RoboStack:mainfrom
sea-bass:add-roboplan
Open

Add RoboPlan packages#38
sea-bass wants to merge 8 commits into
RoboStack:mainfrom
sea-bass:add-roboplan

Conversation

@sea-bass

@sea-bass sea-bass commented Aug 23, 2026

Copy link
Copy Markdown

Adds all the roboplan / roboplan-ros packages, except for roboplan_ros_franka which requires mujoco_ros2_control.

Also does a full rebuild to update Pinocchio and Coal, bumping the build number to 25.

Closes #28

Comment thread patch/ros-rolling-mujoco-ros2-control.patch Outdated
Comment thread patch/ros-rolling-toppra.patch Outdated
Comment thread patch/dependencies.yaml
Comment on lines +299 to +303
roboplan_ros_examples:
# package.xml only declares ament_cmake_python, but CMakeLists.txt does
# find_package(ament_cmake REQUIRED) and build_type is ament_cmake.
# Fixed for roboplan_ros 0.7.0, so it can be removed when this releases.
add_host: ["ros-rolling-ament-cmake"]

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Comment thread pkg_additional_info.yaml
dep_name: proxsuite
max_pin: 'x.x'
override_version: '0.7.2'
override_version: '0.7.3'

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Note to self/other reviewers: in general we try to avoid doing this for abi compat version, but as this is an header only library isn ot a problem.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I'd be fine with proxsuite 0.7.2 as well, though, if it's better

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I guess if you install ros2-proxsuite 0.7.2 , you end up with proxsuite 0.7.3 being installed, as ros2-proxsuite has a proxsuite >=0.7.2,<0.8.0a0 dep on proxsuite, see https://prefix.dev/channels/robostack-rolling/packages/ros2-proxsuite (this derive from max_pin: 'x.x') used here.

@sea-bass
sea-bass force-pushed the add-roboplan branch 3 times, most recently from dfedb4b to 091b462 Compare August 24, 2026 11:08
Comment thread vinca.yaml Outdated
@sea-bass
sea-bass requested a review from traversaro August 24, 2026 14:49
@sea-bass

Copy link
Copy Markdown
Author

It does appear the macos and Windows jobs blew past the 6 hour timeout 🤣

@traversaro

Copy link
Copy Markdown
Member

It does appear the macos and Windows jobs blew past the 6 hour timeout 🤣

The cache is there and active, so just restarting should work fine.

@sea-bass

sea-bass commented Aug 24, 2026

Copy link
Copy Markdown
Author

The cache is there and active, so just restarting should work fine.

Even without removing packages and letting the job run to completion?

(I reverted the change, but will need help restarting the cancelled jobs since I don't have permissions)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Package request: roboplan

2 participants