-
Notifications
You must be signed in to change notification settings - Fork 76
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
Unable to reproduce paper results on HPatches #2
Comments
Hi, thanks for your interest in our project!!! The RANSAC produces some variance but it should give comparable results. My torch version is : 1.2.0 Could you run the command for coarse flow: I would like to see whether the problem is in the coarse alignment or in the fine flow. Thanks a lot!!! -Xi |
My kornia is the same 0.1.4.post2, but the pytorch is 1.5.0. Probably it is the source of the problem. I will check with 1.2.0 + 0.4.0 and come back. |
After re-install of pytorch, torchvision and Pillow, but without feature re-computation: coarse:
fine
So it looks like that the evaluation script, not the extraction, really depends on the library versions... |
And for both extraction and eval - same as you have. Thanks for your time! pytorch 1.2.0 + torchvision 0.4.0
pytorch 1.5.0 + torchvision 0.6.0:
As a suggestion, maybe it is worth mentioning in readme and/or pinning pytorch/torchvision versions in requirements.sh? |
Thanks for the suggestion !!! After checking the doc of Pytorch, there might have a difference about the function F.grid_sample. Also mentioned in stackoverflow Not sure this is the pb, I will carefully check it and come back to you. |
If the difference indeed is because of difference 'align_corners' defaults, you could pass the parameter and get the same results regardless the version. P.S. And if the reason is the "align_corners=True" vs "align_corners=False", I believe in the new results more, tbh. Of course, I don't expect ranking to be changed, just the absolute values. |
There is no parameter align_corners in Pytorch 1.2.0. For your question, I think it is more about the training-test consistency. |
My bad. It is hardcoded to True somewhere in backend then.
Sorry, I don't quite follow. The features are off-the-shelf ImageNet, aren't they? |
The imageNet feature is employed to estimate homographies. Fine flow is estimated from the coarsely aligned pair. F.grid_sample has been largely used in my training code to learn the fine flow. |
Hi,
Thank you for the repository!
I am trying to do some sanity checks before treating to build on top of your work and have some issues with reproducing. Specifically I run
And get the following output:
As opposed to Table 1, where error is from 0.51 to 5.16:
Could you please point out what I am doing wrong?
The text was updated successfully, but these errors were encountered: