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

overfeat failed #13

Closed
Be997398715 opened this issue Dec 8, 2020 · 5 comments
Closed

overfeat failed #13

Be997398715 opened this issue Dec 8, 2020 · 5 comments

Comments

@Be997398715
Copy link

I try to use overfeat model by python2 to reconstruct your paper result, however i can't. many problems to deal. could you give the final overfeat pr curve and map as well as other models? thanks for your help.

@craymichael
Copy link
Owner

craymichael commented Dec 8, 2020

Hi, as the README states, I am not the author of the paper - I merely implemented the methods as I understood them, so I do not have the exact results from the paper (mine don't fare as well). Furthermore, the implementation of OverFeat that I used is now broken, please see sermanet/OverFeat#39 and my response in another issue #10 (comment). Other CNN models are available as detailed in the README.

@1274622976
Copy link

1274622976 commented Dec 8, 2020 via email

@craymichael
Copy link
Owner

You'd have to modify cnn_models.py with whatever other implementation that you're able to find. In get_model_features(...) you need to override the overfeat model with something that returns feature descriptors for all images (imgs). In the smaller version of overfeat, they use the 19th layer, and the 20th layer for the larger model.

I've moved away from SLAM several years ago, so I'm not familiar with what the state-of-the-art looks like. In the past I have used full-blown packages but nothing targetting LCD specifically. I would check out the list of LCD algorithms on PapersWithCode (ranked by GitHub stars): https://paperswithcode.com/task/loop-closure-detection/

@1274622976
Copy link

1274622976 commented Dec 8, 2020 via email

@craymichael
Copy link
Owner

craymichael commented Dec 8, 2020 via email

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

No branches or pull requests

3 participants