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

Update Eigen #37

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update Eigen #37

wants to merge 1 commit into from

Conversation

Paul92
Copy link
Collaborator

@Paul92 Paul92 commented Feb 17, 2020

Eigen moved to gitlab - the previous source is outdated.

Also, moving to a more recent version gets rid of many warnings introduced by newer C++ standards.

@mihaibujanca
Copy link
Owner

Will be testing this on a few devices to make sure there aren't any compatibility issues, however if we are to update it, we'll need to fix the version to a working commit rather than the latest master as future changes might break the build

@mihaibujanca mihaibujanca mentioned this pull request Mar 18, 2020
5 tasks
mihaibujanca added a commit that referenced this pull request Mar 12, 2021
Rosbag support for TUM datasets
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