Skip to content

Full rebuild August 2026 II : bump ros2-distro-mutex to 0.20.0 and build number to 25 + Add RoboPlan packages - #38

Merged
traversaro merged 12 commits into
RoboStack:mainfrom
sea-bass:add-roboplan
Aug 30, 2026
Merged

Full rebuild August 2026 II : bump ros2-distro-mutex to 0.20.0 and build number to 25 + Add RoboPlan packages#38
traversaro merged 12 commits into
RoboStack:mainfrom
sea-bass:add-roboplan

Conversation

@sea-bass

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

Copy link
Copy Markdown
Contributor

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

@sea-bass sea-bass mentioned this pull request Aug 23, 2026
5 tasks
Comment thread patch/ros-rolling-mujoco-ros2-control.patch Outdated
Comment thread patch/ros-rolling-toppra.patch Outdated
Comment thread patch/dependencies.yaml
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
Contributor 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 force-pushed the add-roboplan branch 3 times, most recently from 41734b9 to 6d9f7f3 Compare August 24, 2026 13:16
@sea-bass
sea-bass requested a review from traversaro August 24, 2026 14:49
@sea-bass

Copy link
Copy Markdown
Contributor 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
Contributor 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)

@Tobias-Fischer Tobias-Fischer changed the title Add RoboPlan packages Full rebuild + Add RoboPlan packages Aug 25, 2026
@Tobias-Fischer Tobias-Fischer changed the title Full rebuild + Add RoboPlan packages Full rebuild August 2026 II + Add RoboPlan packages Aug 25, 2026
@traversaro

Copy link
Copy Markdown
Member

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

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

Yes, the cache is saved also on jobs with timeout.

@traversaro

Copy link
Copy Markdown
Member

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

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

Yes, the cache is saved also on jobs with timeout.

Ok, apparently not, but it will be again soon: #40 .

@sea-bass

Copy link
Copy Markdown
Contributor Author

I just rebased on main to include #40 , saw that it was already merged!

@sea-bass

Copy link
Copy Markdown
Contributor Author

I thin the macos arm run failed due to a race with the recipes checker, probably just needs a re-run? thanks!

@mini-1235

Copy link
Copy Markdown
Contributor

I see that the build timed out, but the cache was uploaded properly this time. I think we just need to close and reopen the PR a few times and it should eventually pass, assuming there are no actual build errors. From my previous experience, a full Windows rebuild take around 18 hours

@sea-bass

sea-bass commented Aug 28, 2026

Copy link
Copy Markdown
Contributor Author

So it appears there was an issue on Windows. The cached build for moveit_core seems to have been empty/incomplete, which then failed the build on the first dependent package (chomp_motion_planner).

I've modified the workflow YAML temporarily to remove the bad cache since there seems to be precedent for that. But maybe we can kick off this rerun the 2nd macos job completes so we don't accidentally damage that one 🙃

@sea-bass

Copy link
Copy Markdown
Contributor Author

ok the macos jobs have all passed, this should be good to approve the workflow and get the windows build passing now -- thanks, all!

@sea-bass sea-bass closed this Aug 29, 2026
@sea-bass sea-bass reopened this Aug 29, 2026
@sea-bass

Copy link
Copy Markdown
Contributor Author

@traversaro this seems ready to go!

@traversaro traversaro changed the title Full rebuild August 2026 II + Add RoboPlan packages Full rebuild August 2026 II : bump ros2-distro-mutex to 0.20.0 and build number to 25 + Add RoboPlan packages Aug 30, 2026
@traversaro

Copy link
Copy Markdown
Member

@traversaro this seems ready to go!

Thanks a lot! I would just de-vendor mujoco as done in the other distros to avoid confusion, let me do that change.

@traversaro

Copy link
Copy Markdown
Member

@traversaro this seems ready to go!

Thanks a lot! I would just de-vendor mujoco as done in the other distros to avoid confusion, let me do that change.

Actually mujoco was already devendored, I just added an option to ensure it was never accidentally vendored again.

@traversaro
traversaro merged commit c66af84 into RoboStack:main Aug 30, 2026
6 checks passed
@sea-bass
sea-bass deleted the add-roboplan branch August 30, 2026 14:17
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

4 participants