-
Notifications
You must be signed in to change notification settings - Fork 48
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
Conda Linux CI Failure End of July 2023 #1446
Comments
Looking at the recent changes in the resolved environment, this seems a eigen 3.4.0 -> 3.4.1 regression. |
Interestingly, I could not find any 3.4.1 reference in Eigen documentation, just a branch called 3.4.1 . I wonder if that is not a release. |
Looking at the recent changes in the resolved environment, this seems a eigen 3.4.0 -> 3.4.1 regression. |
Fixed by conda-forge/admin-requests#763, see conda-forge/eigen-feedstock#39 for more background. Anyhow, it seems that there is an incompatibility between the Eigen 3.4.1 that is going to be released at some point (see https://gitlab.com/libeigen/eigen/-/issues/2702#note_1485913208) and latest releases of OpenCV . We should report this to either Eigen or OpenCV, so that we solve it before it actually hits a release of Eigen. |
3.4.1 is out and unfortunately, I get OpenCV build issues. Until a fix is made, I build opencv using |
Are you sure it is out? I do not see any news in https://eigen.tuxfamily.org/index.php?title=Main_Page in or a tag in https://gitlab.com/libeigen/eigen/-/tags . As stated in https://gitlab.com/libeigen/eigen/-/issues/2702 by Eigen mantainers, the branch |
I confirm that the Eigen branch 3.4 is in fact a 3.4.1 release since November 5th. The issue is related to the unsupported headers of Eigen, which is included here. Concerning these headers, this is Eigen's disclaimer:
I can see that unsupported/Eigen/CXX11/src/Tensor is in active development. The issue that we are facing has been fixed in Eigen Master branch with this commit. Actually, I don't think we can blame Eigen for this issue since they explicitly state that these faulty headers are not supported. The blame goes to OpenCv which does not exclude this Eigen version from the build. |
Thanks for digging the issue and providing the details! Probably it may be worth reporting this to Eigen maintainers, then they can decide if they want to backport the fix to 3.4 or not. |
It is not a release, it is the branch we use to prepare releases. Releases will be announced, have a git tag, and a release archive under https://gitlab.com/libeigen/eigen/-/releases. |
Thank you for clarifying. I appreciate the update on the Eigen branch 3.4.1 ! |
fyi @GiulioRomualdi @S-Dafarra
The text was updated successfully, but these errors were encountered: