Skip to content
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 C++ gradients for some image operators. #21021

Merged
merged 1 commit into from Aug 4, 2018

Conversation

kbsriram
Copy link
Contributor

Added gradients and tests for

  • ResizeBilinear
  • ResizeBicubic
  • ResizeNearestNeighbor

Note: Some of the tests are for the operator itself rather than
the gradient, paralleling existing tests in image_grad.py

See #21019

Added gradients and tests for
- ResizeBilinear
- ResizeBicubic
- ResizeNearestNeighbor

Note: Some of the tests are for the operator itself rather than
the gradient, paralleling existing tests in image_grad.py

See tensorflow#21019
@caisq caisq requested a review from skye July 23, 2018 13:40
@caisq caisq self-assigned this Jul 23, 2018
@caisq caisq added the awaiting review Pull request awaiting review label Jul 23, 2018
Copy link
Member

@skye skye left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice, thanks!

@skye
Copy link
Member

skye commented Jul 24, 2018

Jenkins test this please

@tensorflowbutler tensorflowbutler removed the awaiting review Pull request awaiting review label Jul 25, 2018
@yifeif yifeif added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Jul 25, 2018
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Jul 25, 2018
@case540 case540 added the kokoro:force-run Tests on submitted change label Aug 1, 2018
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Aug 1, 2018
@tensorflow-copybara tensorflow-copybara merged commit a2c479b into tensorflow:master Aug 4, 2018
tensorflow-copybara pushed a commit that referenced this pull request Aug 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes ready to pull PR ready for merge process
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants