-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
stitching example for openvn not working, complaining about missing openvc_ffmpeg_... #336
Comments
What about this sample?
https://github.com/bytedeco/javacpp-presets/tree/master/opencv#sample-usage
Does it fail in the same way?
|
That's the example I'm trying to run. Don't you have any integration tests in your project? Why don't you put examples as integration tests into it and make sure they actually work? I couldn't find anything meaningful that's actually working regarding JavaCPP yet. You can run your github project in the travis integration test system for free simply by putting a one-line .travis.yml file at the top of your github project and activating the project on the travis service (see https://github.com/jjYBdx4IL/example-maven-project-setups). |
I've just tried the sample exactly as it appears here: As for integration tests, sure, that's the plan. If you would like to give a few hours of your time to make this happen and make a contribution, that would be most welcome! The issue you've opened previously about this is still open: bytedeco/javacpp-presets#26 |
eh, your stuff is working, not mine. so it would be better if you built a working IT :-) The problem with this test unit is simple: I can make the errors go away by manually loading the libraries, so there is something broken with the recursive library loading from the jars. |
Ah, wait a minute, you've got the test ignored. Ok, so it fails on Linux as well. And you want me to debug this, ok. |
Ok, so it looks there are a few dependencies missing from the presets. Calling the following solves this issue: Loader.load(opencv_calib3d.class);
Loader.load(opencv_shape.class);
Loader.load(opencv_xfeatures2d.class); Thanks for reporting! |
btw you can read those dependencies directly from the library files using "readelf -d" on Linux. Put that dependency information into properties files and let the lib loader use it. Or just merge all libs into one single one. :-) http://stackoverflow.com/questions/915128/merge-multiple-so-shared-libraries |
Yes, I know, and that's how I do it. But it changes with every version and it changes also with the flags used at build time. I'm just one guy doing this right now during my spare time. I can't keep up with everything. So, like I said, if you can take a few hours here and there to help, it would be very welcome! |
If you're asking why this isn't automated, the answer is the same: I do not have the time to do everything. Your contributions would be very welcome! |
Fix is in version 1.2! Thanks for reporting |
You can try the example, it is stored at github:
https://github.com/jjYBdx4IL/example-maven-project-setups/tree/master/javacpp-opencv-example
Here is the log with lib loader output enabled:
The text was updated successfully, but these errors were encountered: