-
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
Writing KeyPointVector bug #1012
Comments
Ok, I checked it with javacv-1.3 and it works. At first I tried just changing the javacpp.jar and javacv.jar to those from version 1.3 and at this point method argument was correct (KeyPointVector) - not uderlining, however the program still wasn't compiling because of it. Only after using complete set of files from javacv-1.3 it started running. At this point I'll try to use older version (howver I don't know if it won't cause any other trouble) but I'm hoping that this issue will be fixed shortly. Anyways I'm very grateful for your work, as the official OpenCV for java lacks a lot of features (this one is one of them actually). |
Ok, something more FYI, maybe it will help you to examine the issue.
After changing the opencv-windows-x86_64.jar file to the one from 1.3.3 everything seems to be working fine. So I don't know if the problem regards javacv itself or the OpenCV core maybe, but this is what I found out. |
If looks like these functions are deprecated. I'll enable them by defining |
@Maurice-Betzel If you have some time, could you take a look and update the sample? Thanks for your time on this! |
Ok, thank you very much for you reply. It's just weird that the function works for single KeyPoint and does not work for vector of them - why would anyone would like to change it that way ;) |
…tVector` and `DMatchVector` (issue bytedeco/javacv#1012)
I see, they want us to use the function templates. We'll do that when the legacy functions disappear, so we shouldn't need to change anything else once we do that:
bytedeco/javacpp-presets@12b0576#diff-098f93602679a6676e7669064dd1a9ebR22839 |
Ok, cool, thank you :) |
Any chance you could upload these few modified compiled *.jar files? I've spent a few hours trying to compile latest commits, but still getting some errors and I'm exhausted, I need the newest version as it contains push_back for KeyPointsVector container :| |
The CI servers automatically upload snapshots to the repository: http://bytedeco.org/builds/
|
O! I'm laughing through tears. I've spent so much time for nothing, but on the other hand I've got it working :) Thank you so much for such a quick answer solving my problem ;) |
Sorry about that :( |
Fix is included in version 1.4.2. Thanks again for reporting! |
Hello, I think I've found bug which is stopping my further work as I cannot proceed with my project without completing this step.
So, based on this sample:
https://github.com/bytedeco/javacv/blob/72d9fd9d55e86a32942afb5edd983cd4bb3647bc/samples/OpenCVFeatures2dSerialization.java
and particaularly this piece of code:
it should all work. However, in my case the write method does not take KeyPointVector as an argument but just the single KeyPoint. I'm going to try with previous versions of javacv, but I would be very grateful if someone could take a look and give me a solution or fix the bug ASAP.
The text was updated successfully, but these errors were encountered: