[OpenVINO] Implement image transforms (affine, perspective, elastic)#22696
[OpenVINO] Implement image transforms (affine, perspective, elastic)#22696hertschuh merged 5 commits intokeras-team:masterfrom
Conversation
There was a problem hiding this comment.
Code Review
This pull request implements affine_transform, perspective_transform, and elastic_transform for the OpenVINO backend, enabling several previously excluded image operation tests. The implementation uses OpenVINO opsets for coordinate mapping and homography computation. Feedback identifies a hardcoded fill_mode in perspective_transform that should be parameterized for API consistency, a docstring error regarding return dimensions, and opportunities to optimize coordinate gathering. Additionally, the review suggests improving robustness against division-by-zero in projective transformations and notes a performance bottleneck in elastic_transform due to a symbolic-to-NumPy conversion during seeding.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #22696 +/- ##
==========================================
- Coverage 78.82% 77.68% -1.15%
==========================================
Files 596 596
Lines 69293 69625 +332
Branches 10829 10861 +32
==========================================
- Hits 54622 54089 -533
- Misses 11941 12781 +840
- Partials 2730 2755 +25
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
hertschuh
left a comment
There was a problem hiding this comment.
Thanks for implementing these!
Description
This PR implements affine_transform, perspective_transform, and elastic_transform for the OpenVINO backend.
Removed 50 tests, which now pass after these implementations.
Closes: openvinotoolkit/openvino/issues/35412
Contributor Agreement
Please check all boxes below before submitting your PR for review: