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
mathematica: use opencv4 #101227
mathematica: use opencv4 #101227
Conversation
The default opencv2 is marked broken at 5e68d20, and it is suggested there that the current release is 4.x.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be fine. I don't think anyone without Mathematica will actually be able to test this, unfortunately.
@lovesegfault I have Mathematica if you need me to test? |
Please test if you have use cases involving OpenCV. |
I was going to just go to whatever reference pages Wolfram has for OpenCV and run whatever examples are there. I suspect that will be fine? |
There is an example from Wolfram, and it uses the C language, but the C interface is only available in OpenCV 2. OpenCV 4 provides only C++ interfaces. In this example, OpenCV is used as an external library, so it should be possible to rewrite it in C++ instead, and it should work as long as the Also, there is a bundled OpenCV library in directory |
Sorry, I have not had time to look into this yet. I will try to get to it in the next few days. |
Looks like I actually won't be able to provide any help with this due to a lack of time, sorry! @herberteuler do you have an explanation about why |
I do not know either, it was already there when I started to maintain this package. It is used as an external library in the example, but also bundled with the package. |
Motivation for this change
The default opencv2 is marked broken at 5e68d20, and it is suggested there that the current release is 4.x.
Things done
Updated the dependency on opencv from release 2 to release 4. The build is broken if using
opencv2
. Some tests were done to ensure the build works.sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)