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

Link to issue against upstream Eigen? #5

Open
EricCousineau-TRI opened this issue Oct 19, 2022 · 6 comments
Open

Link to issue against upstream Eigen? #5

EricCousineau-TRI opened this issue Oct 19, 2022 · 6 comments
Labels

Comments

@EricCousineau-TRI
Copy link

README mentions defect, but shows no traceability towards letting upstream know, nor does it give users of this package a way to know if/when they can stop using it depending on their Eigen version.

\cc @IanTheEngineer @sloretz

@EricCousineau-TRI EricCousineau-TRI changed the title Link to Eigen issue tracker? Link to issue against upstream Eigen? Oct 19, 2022
@clalancette
Copy link
Contributor

This package may not be needed anymore in Ubuntu 20.04 and later. If that's the case, we would love to retire it, but we'd have to verify that all of the known issues with finding Eigen are gone first.

@EricCousineau-TRI
Copy link
Author

Thanks! Though it's not immediately clear to me why it may not be needed anymore in future OS versions.

Was there an upstream patch that we should keep an eye out for?

@EricCousineau-TRI
Copy link
Author

My guess is maybe this one? https://gitlab.com/libeigen/eigen/-/issues/1386

@clalancette
Copy link
Contributor

My guess is maybe this one? https://gitlab.com/libeigen/eigen/-/issues/1386

Yeah, possibly. I'm going off of my memory here (since we don't have this written down), but my understanding is that later Eigen has fixed the CMake issues that caused this package to be needed in the first place. If we can verify that on our target platforms, then we can consider removing this.

@sloretz
Copy link
Contributor

sloretz commented Oct 31, 2022

IIRC the current state is Eigen's upstream's CMake should work great now that ROS 2 uses modern CMake targets, except that we ship our own Eigen3Config.cmake in the chocolatey package which doesn't make the exported targets available: ros2/choco-packages#19 . I think this cmake module can be removed once we fix the chocolatey package.

@RFRIEDM-Trimble
Copy link

Works great for me following this guide on galactic ubuntu 20.04.
https://eigen.tuxfamily.org/dox/TopicCMakeGuide.html

package.xml for my project - use the rosdep source of eigen, which pulls in eigen3 on Ubuntu.

 <depend>eigen</build>

CMakeLists for my project:

find_package (Eigen3 3.3 REQUIRED NO_MODULE)

ament_auto_find_build_dependencies(..)
ament_auto_add_library(..)
target_link_libraries(${PROJECT_NAME}_lib Eigen3::Eigen)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants