Keep aspect ratio of images?
javiabellan opened this issue · comments
I noticed that DKM resizes the image to a fixed size and aspect ratio (horizontal). I think preserving the original aspect ratio of the images could improve performance, especially for extreme image pairs like this one:
There is any tweak (code modification) to allow this into the current DKM weights?
Yes it possible. And for the pair you show I agree it would probably improve performance.
If you want to dig into the code, the main barriers are
-
For feature extraction the images are stacked. Instead we need to extract seperately.
-
In several places I just use "H,W". This needs to be H_A/H_B or something.
In tiny roma (over in the roma repo), I've made these changes. You can refer to the implementation there.