Skip to content
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

Added EigenPlaces for retrieval #337

Merged
merged 1 commit into from
Jan 11, 2024
Merged

Conversation

gmberton
Copy link
Contributor

Hi @sarlinpe !
Following our conversation on the CosPlace PR, I have now changed the file hloc/extractors/cosplace.py with hloc/extractors/eigenplaces.py and added a variant config entry that can be switched between CosPlace and EigenPlaces (with the latter as default).

Here are the results on Aachen, although they don't really show much difference between NetVLAD and EigenPlaces, although I believe this might be due to Aachen being saturated (there were similar results with CosPlace).

Method --num_loc Aachen Day Aachen Night
NetVLAD 10 89.0 / 95.0 / 98.2 79.6 / 88.8 / 95.9
EigenPlaces 10 88.6 / 96.0 / 99.2 80.6 / 89.8 / 96.9
NetVLAD 50 90.5 / 96.4 / 99.3 84.7 / 92.9 / 100.0
EigenPlaces 50 90.2 / 96.2 / 99.6 83.7 / 90.8 / 96.9

@sarlinpe
Copy link
Member

Great, thank you!

@sarlinpe sarlinpe merged commit 988dd3a into cvg:master Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants