-
Notifications
You must be signed in to change notification settings - Fork 443
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
Add legacy ssd-like IRs support #1970
Add legacy ssd-like IRs support #1970
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you so much for the fix. We need to be careful for this kind of change in the future. By the way, could you update the change log? It's a new convention and I think this change is worth to note. :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this for old version of openvino model inference? It would be nice if you add some summary and motivation of this work.
otx/algorithms/detection/adapters/openvino/model_wrappers/openvino_models.py
Show resolved
Hide resolved
otx/algorithms/detection/adapters/openvino/model_wrappers/openvino_models.py
Show resolved
Hide resolved
Added a brief description in summary |
Summary
In old SSD-like IRs bbox and label layers have different shapes than in current implementation. Wrapper was fixes to support both IR formats.
How to test
Checklist
License
Feel free to contact the maintainers if that's a concern.