Jmeyer1292 / robot_cal_tools

A suite of tools focused on calibration of sensors for robotic workcell development
Apache License 2.0
140 stars 40 forks source link

Ceres has removed LocalParameterization. #111

Closed l0kol closed 2 months ago

l0kol commented 5 months ago

Hello.

I'm in the process of building your package and I noticed that if I install the latest Ceres version I get error at LocalParameterization lines. I checked the Ceres docs and it says: LocalParameterization has been removed, use Manifold instead..

So maybe Ceres version should be specified in the readme before the code is updated with Manifold. Or maybe I just didn't see it, I apologize in that case for writing all this :).

Best regards

marip8 commented 2 months ago

See #115 for a fix. Also see this comment about alternatively using an improved version of industrial_calibration where this has already been addressed.

l0kol commented 2 months ago

Ah okay I see. Thank you for your answer.