Hierarchical-Localization icon indicating copy to clipboard operation
Hierarchical-Localization copied to clipboard

Sift based local feature extraction is not working bcause of update in pycolmap.

Open dev111ce opened this issue 1 year ago • 2 comments

The local feature extractor SIFT based DOG extractor class model in hloc/extractors/dog.py seems to be now broken becuase of the update in the pycolmap. With the latest pycolmap 0.6.1, pycolmap.sift.extract returns only two parameters while three are expected in the current code. keypoints, scores, descriptors = self.sift.extract(image_np). Upon execution, we get the below error:

ValueError: not enough values to unpack (expected 3, got 2)

Upon performing a quick RCA, we observed that the current pycolmap i.e., 0.6.1 version, returns just keypoints and descriptors and omits scores, which is the reason for the breakdown.

Any suggestion/fix on how to obtain the score value or fix the existing solution is greatly appreciated. Thanks in advance.

dev111ce avatar Feb 21 '24 05:02 dev111ce

This has been fixed in https://github.com/cvg/Hierarchical-Localization/commit/936040e8d67244cc6c8c9d1667701f3ce87bf075, please update your clone of hloc.

sarlinpe avatar Feb 21 '24 11:02 sarlinpe

I am installing the latest pycolmap 0.6.1, so I have to go back to 0.6.0?

dev111ce avatar Mar 06 '24 08:03 dev111ce