-
-
Notifications
You must be signed in to change notification settings - Fork 56.2k
Added TF resize_bilinear behaviour alignment #18996
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
Conversation
7515325
to
8ff518d
Compare
8ff518d
to
14bb8a8
Compare
Take a look at failed test: https://pullrequest.opencv.org/buildbot/builders/precommit_custom_windows/builds/1719. You may skip it for specific backend or implement in |
f2447d7
to
c16d1da
Compare
@LupusSanctus , please check comments at opencv/opencv_extra#825 |
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!
Thank you for contribution! This patch should go into 3.4 branch first. Please:
Note: no needs to re-open PR, apply changes "inplace". Also please make similar steps for opencv_extra's PR. |
Fixes OpenCV issue opencv#18721
c16d1da
to
a82c50e
Compare
@alalek, the commit was rebased onto 3.4 branch. |
Test data for OpenCV PR opencv#18996
Merge with extra: opencv/opencv_extra#825
Fixes #18721: TF resize bilinear upsampling layer shift
Added
half_pixel_centers
parameter to coveralign_corners=False, half_pixel_centers=True
case for TF resize_bilinear.Pull Request Readiness Checklist
See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request
Patch to opencv_extra has the same branch name.