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 in faster rcnn #541

Merged
merged 1 commit into from
Jul 3, 2019
Merged

Added in faster rcnn #541

merged 1 commit into from
Jul 3, 2019

Conversation

benhoff
Copy link
Contributor

@benhoff benhoff commented Jul 2, 2019

No description provided.

@nmanovic nmanovic merged commit cd5d431 into cvat-ai:develop Jul 3, 2019
@nmanovic
Copy link
Contributor

nmanovic commented Jul 3, 2019

@benhoff , the script also doesn't work.

@benhoff
Copy link
Contributor Author

benhoff commented Jul 3, 2019

@nmanovic Which version of openvino are you using? There's an API change in openvino 2019 that I've accounted for locally that I'm not sure how to add into the project

@nmanovic
Copy link
Contributor

nmanovic commented Jul 3, 2019

@benhoff, I'm using the latest version of OpenVINO 2019 R1.1 (144).

@benhoff
Copy link
Contributor Author

benhoff commented Jul 3, 2019

@nmanovic see openvinotoolkit/openvino#128

Specifically, alalek's comment on April 25th:

MO 2018R5 generates model IRs with these inputs/outputs (see the first post about used parameters of MO):

network.inputs = ['image_tensor']
network.outputs = ['detection_output']

MO 2019R1 IRs:

network.inputs = ['image_tensor', 'image_info']
network.outputs = ['detection_output']

If you look at the Cvat line that handles the input for OpenVino, it's pretty simple.

Relevant code: https://github.com/opencv/cvat/blob/021b77cceeab5927632684b737cd11f66ce087dc/cvat/apps/auto_annotation/model_loader.py#L31

I've hacked in some handling locally. I can split it out of my local dev and drop a pull request and we can talk about the best handling from there?

@benhoff benhoff deleted the faster-rcnn branch July 3, 2019 14:18
@benhoff
Copy link
Contributor Author

benhoff commented Jul 3, 2019

See: #545

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.

2 participants