keras-preprocessing
keras-preprocessing copied to clipboard
keras-team
Removed SciPy dependency.
Summary
Related Issues
PR Overview
- [n] This PR requires new unit tests [y/n] (make sure tests are included)
- [n] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
- [y] This PR is backwards compatible [y/n]
- [n] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)
This is the second (last) part of the effort to remove SciPy dependency (#322)
Note: currently two tests are failing because of a bug tensorflow/addons#2357. Since the issue is not critical -- it only affects boundaries, we can adjust unit tests to follow the buggy behavior of tensorflow-addons.
Switching to tf_addons is a pretty big change so I will reach out to Francois.
Thank you for the PR!
I think, there's no reason to keep "external" dependencies. I would suggest to get rid of numpy too.
On Thu, Jan 21, 2021, 6:19 PM Frédéric Branchaud-Charron < [email protected]> wrote:
Switching to tf_addons is a pretty big change so I will reach out to Francois.
Thank you for the PR!
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/keras-team/keras-preprocessing/pull/334#issuecomment-764762818, or unsubscribe https://github.com/notifications/unsubscribe-auth/AASS73RSVVMIYW4Z7YQCZULS3BHZDANCNFSM4WMUHG6A .
In fact we do not need anything but TF here. Instead of tensorflow_addons one can use here tf.raw_ops.ImageProjectiveTransformV3
P. S.
updated the PR to use tensorflow only.
Once tensorflow/tensorflow#46637 is merged I think we can merge this pr.
Thank you!
Once tensorflow/tensorflow#46637 is merged I think we can merge this pr.
Thank you!
They will not approve it because it breaks internal tests (that are relying on the current behavior). Should we update our unit tests to rely on the TF behavior?