Skip to content

Conversation

@talregev
Copy link
Contributor

@talregev talregev commented Dec 27, 2025

This PR is only for testing.
Showing you the compilation error while compile gtsam with eigen3 version 5.0.1. to fix them in different PR.

On vcpkg, they do this patch to compile gtsam with eigen3 version 5.0.1:
https://github.com/microsoft/vcpkg/blob/master/ports/gtsam/eigen3-fixes.patch

@talregev
Copy link
Contributor Author

@dellaert @ProfFan
I have a compilation fix, but one test is start to failing and need to change.
Do you want that I create a PR for that and you take a look?

@dellaert
Copy link
Member

Yes, please. Happy to take a look!
PS do you see performance benefits with Eigen 5?

@talregev
Copy link
Contributor Author

I created a PR: #2331
I don't think there is a performance difference in v5.0.1.

@dellaert
Copy link
Member

dellaert commented Jan 2, 2026

Still needed?

@talregev
Copy link
Contributor Author

talregev commented Jan 2, 2026

No, I am close this PR.
Thank you for the review.

@talregev talregev closed this Jan 2, 2026
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.

2 participants