-
Notifications
You must be signed in to change notification settings - Fork 892
cv2.findEssentialMat w/ USAC_ACCURATE throws LAPACK/Eigen error from pypi wheel #534
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
Comments
Eigen is header-only library, it's not visible in list of shared libraries or dependencies. |
Thanks! |
Hi @asmorkalov, just wanted to check back with you about this. I noticed that in the 4.5.0 opencv distribution, eigen/lapack are properly packaged. But in other later versions, such as 4.5.3, the packaging is broken. Was there a major change in the packaging between those versions? |
@johnwlambert The dependency is broken by mistake. We are working on infrastructure fix for the next release. |
Hi @johnwlambert, the issue was fixed in #547 and merged into |
Very nice, thanks @asenyaev for the patch. Do you know when the next release will be published? |
OpenCV 4.5.4 is already published in core repository: https://github.com/opencv/opencv/releases/tag/4.5.4. Python packages will be ready in a week or so. |
OpenCV-Python 4.5.4 packages just released to Pypi. LAPACK is enabled for all Linux configurations, MacOS (accelerate framework), but not Windows. The team will work on Windows support for the next release. |
Thank you! Confirmed on my end that the issue is resolved. |
Sounds great!. I reopen the issue as Windows configuration is not ready. |
hello @asmorkalov With WIN 10 I have the same problem and I am getting the error error: OpenCV(4.5.5) D:\a\opencv-python\opencv-python\opencv\modules\calib3d\src\usac\essential_solver.cpp:197: error: (-213:The function/feature is not implemented) To use essential matrix solver LAPACK or Eigen has to be installed! in function 'cv::usac::EssentialMinimalSolverStewenius5ptsImpl::estimate' cv.version I have installed LAPACK and Eigen using conda with conda install -c conda-forge lapack |
Then I downgraded to version 4.5.1.48. |
Yes. Windows packages does not contain LAPACK and the function is available on Linux and Mac for now. We are looking for good enough LAPACK implementation for Windows to cover all platforms. |
hello @asmorkalov Do you have any updates for USAC OpenCV implementation for windows? In my study, I want to use both RANSAC and USAC implementations. best regards |
Hello @mfatih7 We have some build issues with LAPACK and binaries distribution for now. Unfortunately I cannot guarantee that it'll be fixed before the next release. |
Expected behaviour
I expect the library not to crash or throw an error:
Actual behaviour
but I installed this from the pypi wheel. I would expect
auditwheel
ordelocate
to have provided the Eigen dependenciesSteps to reproduce
then
Issue submission checklist
opencv-python
The text was updated successfully, but these errors were encountered: