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.

Sign up for GitHub

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

Jump to bottom

Added NetVLAD model #2021

Merged
merged 6 commits into from
Feb 9, 2021
Merged

Added NetVLAD model #2021

merged 6 commits into from
Feb 9, 2021

Conversation

Anna-Mironova
Copy link
Contributor

@Anna-Mironova Anna-Mironova commented Jan 26, 2021 •
edited
Loading

TODO:

  • Add demo

@Anna-Mironova Anna-Mironova added the model All about model, from enabling to issues label Jan 26, 2021
@Anna-Mironova Anna-Mironova force-pushed the am/netvlad branch 2 times, most recently from 978ad82 to d26bdea Compare January 26, 2021 11:37
eizamaliev
eizamaliev suggested changes Jan 27, 2021
View reviewed changes
models/public/netvlad/netvlad.md Outdated Show resolved Hide resolved
IRDonch
IRDonch suggested changes Jan 27, 2021
View reviewed changes
models/public/netvlad/netvlad.md Outdated Show resolved Hide resolved
models/public/netvlad/pre-convert.py Outdated Show resolved Hide resolved
models/public/netvlad/pre-convert.py Outdated Show resolved Hide resolved
models/public/netvlad/pre-convert.py Outdated Show resolved Hide resolved
IRDonch
IRDonch suggested changes Feb 1, 2021
View reviewed changes
demos/place_recognition_demo/python/README.md Show resolved Hide resolved
demos/place_recognition_demo/python/place_recognition_demo.py Outdated Show resolved Hide resolved
demos/place_recognition_demo/python/place_recognition_demo.py Outdated Show resolved Hide resolved
demos/place_recognition_demo/python/place_recognition_demo/common.py Outdated Show resolved Hide resolved
demos/place_recognition_demo/python/place_recognition_demo/place_recognition.py Outdated Show resolved Hide resolved
demos/place_recognition_demo/python/place_recognition_demo/place_recognition.py Outdated Show resolved Hide resolved
models/public/netvlad/pre-convert.py Outdated Show resolved Hide resolved
models/public/netvlad/pre-convert.py Outdated Show resolved Hide resolved
tools/downloader/README.md Show resolved Hide resolved
@IRDonch IRDonch requested a review from Wovchena February 3, 2021 14:49
@eaidova
Copy link
Collaborator

eaidova commented Feb 4, 2021

@eizamaliev please take a look. Do you agree with changes?

@eaidova eaidova merged commit 8002dfa into openvinotoolkit:develop Feb 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Wovchena Wovchena Wovchena approved these changes

@eaidova eaidova eaidova approved these changes

@IRDonch IRDonch IRDonch approved these changes

@eizamaliev eizamaliev eizamaliev approved these changes

Assignees
No one assigned
Labels
model All about model, from enabling to issues
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Anna-Mironova @eaidova @Wovchena @IRDonch @eizamaliev