DEAL_NeurIPS_2021 icon indicating copy to clipboard operation
DEAL_NeurIPS_2021 copied to clipboard

PR to kornia?

Open ducha-aiki opened this issue 2 years ago • 3 comments

Hi,

Would you be interested in integrating DEAL into kornia.features alongside with HardNet?

-- Best, Dmytro

ducha-aiki avatar Feb 10 '23 17:02 ducha-aiki

Hello Dmytro,

Yes, we are very interested in contributing to Kornia! I will check Kornia's PR guidelines and will start the process of adapting the code of this repo. I will update you on this issue before I create a PR to Kornia. Thank you for the invitation.

Kind regards,

Guilherme Potje

guipotje avatar Feb 13 '23 12:02 guipotje

@guipotje I am glad to hear that :) I think that the most reasonable would be to inherit from kornia.feature.LAFDescriptor and put there the deformation-aware patch extraction and description.

We also don't use OpenCV keypoints, and use kornia.feature.LAF instead. The conversion is simple, one can generate LAF using kornia.feature. laf_from_center_scale_ori, e.g. as in this https://github.com/ducha-aiki/kornia_moons/blob/master/kornia_moons/feature.py#L19

ducha-aiki avatar Feb 14 '23 13:02 ducha-aiki

I think that the most reasonable would be to inherit from kornia.feature.LAFDescriptor and put there the deformation-aware patch extraction and description.

Sounds nice @ducha-aiki, thank you for the tips. I'll adapt the code for that, and also use the kornia.feature.LAF instead of OpenCV keypoints.

guipotje avatar Feb 15 '23 15:02 guipotje