crigroup / handeye

ROS package for calibrating sensors to a known reference frame.
BSD 3-Clause "New" or "Revised" License
51 stars 25 forks source link

Release to Noetic and ROS2 #5

Open JStech opened 4 years ago

JStech commented 4 years ago

Thanks for the great software. We've seen a lot of interest in MoveIt Calibration, which is built on handeye.

Could you please make a Noetic release?

fsuarez6 commented 3 years ago

ROS Noetic PRs:

fsuarez6 commented 3 years ago

@JStech About ROS2, is it worth releasing for the three distros (Dashing Diademata, Eloquent Elusor and Foxy Fitzroy)?

JStech commented 3 years ago

I'd say definitely Foxy and maybe Eloquent (nice-to-have), but Dashing can be skipped.

For some context, we use this package in MoveIt Calibration. I don't think we'll release MoveIt Cal for Eloquent because OpenCV 3.2 (which ships with Ubuntu 18.04) has buggy pose detection of ArUco boards, but otherwise I think 18.04 and Eloquent are still pretty widely used, so you might want to release for them.

JStech commented 3 years ago

The release builds for criutils and handeye are failing. It seems that the appropriate tags in the release repos weren't created--there's no debian/ros-noetic-* tag in either criutils-release or handeye-release. I'm gonna see if I can debug this any further--I'm a total newbie to all this, but happy to learn more.

fsuarez6 commented 3 years ago

Good catch. For some reason I failed to push the changes to the *-release repos. I've triggered a new release for criutils and handeye:

That should do the trick.