-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[CPU] Fix RegionYolo (v3) node and Ngraph reference implementations #5748
[CPU] Fix RegionYolo (v3) node and Ngraph reference implementations #5748
Conversation
RegionYolo description |
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.
CPU part lgtm
@lazarevevgeny could you please review ngraph reference changes? thanks |
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.
The update in output size calculation is correct.
But I want to see more test to check that output values are calculated correctly.
a0a6851
to
cf2cd66
Compare
Rebasing |
07ee116
to
8b6229e
Compare
Destination size was calculated incorrectly for v3 versions
8b6229e
to
660374b
Compare
@dmitry-gorokhov Rebased, CI passed and the change approved by Ngraph team. Ready to be merged. |
Details:
Tickets: