-
Notifications
You must be signed in to change notification settings - Fork 16
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
Different results when trying MKD, TFeat descriptors in kornia #22
Comments
Thanks for the report. The problem is that OpenCV drawing code stopped working at some point.
is wrong. when you convert bool (the correspondence is true or not) to int, you instead of filtering, index the points. Or - better - switch to kornia_moons visualization, as in this tutorial: https://kornia-tutorials.readthedocs.io/en/latest/image_matching_adalam.html |
@jac08h could you please share updated colab? |
Here is the version using kornia_moons visualization. To use it, I modified |
@jac08h I don't know why, but you are using very old version of kornia-examples. The key difference is in current version we have:
Whereas in yours:
We have dropped auto-conversion in After the fix above: |
You are right, that was the issue. Thank you so much for your quick help. |
Hello,
I tried to run MKD_TFeat_descriptors_in_kornia.ipynb, but my results differ significantly from the example notebook. Most importantly, using deep-learning-based descriptors does not yield better matches.
I did a single change to the example code: Instead of
matchesMask = inliers_mask.ravel().tolist(),
I useinliers_mask.astype(int).ravel().tolist()
, because the former was throwing a cv2.error - "Can't parse 'matchesMask'. Sequence item with index 0 has a wrong type".Package versions:
The output is in this notebook.
Do you have an idea what is causing the different behaviour?
Thanks a lot for your time!
The text was updated successfully, but these errors were encountered: