Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ceres has removed LocalParameterization. #111

Closed
l0kol opened this issue Jun 5, 2024 · 2 comments
Closed

Ceres has removed LocalParameterization. #111

l0kol opened this issue Jun 5, 2024 · 2 comments

Comments

@l0kol
Copy link

l0kol commented Jun 5, 2024

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
Copy link
Collaborator

marip8 commented Sep 26, 2024

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
Copy link
Author

l0kol commented Sep 27, 2024

Ah okay I see. Thank you for your answer.

@l0kol l0kol closed this as completed Sep 27, 2024
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

No branches or pull requests

2 participants