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

Replaced get_shape() to shape. #27194

Merged
merged 7 commits into from
Apr 2, 2019

Conversation

amitsrivastava78
Copy link
Contributor

This is the recommended method.

Amit Srivastava added 6 commits March 27, 2019 15:34
This is the recommended method.
This is the recommended method.
This is the recommended method.
This is the recommended method.
This is the recommended method.
This is the recommended method.
@gbaned gbaned self-assigned this Mar 27, 2019
@gbaned gbaned added this to Assigned Reviewer in PR Queue via automation Mar 27, 2019
@gbaned gbaned added comp:keras Keras related issues size:S CL Change Size: Small labels Mar 27, 2019
@gbaned gbaned requested a review from omalleyt12 March 27, 2019 10:21
Copy link
Contributor

@omalleyt12 omalleyt12 left a comment

Choose a reason for hiding this comment

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

Thanks for the PR!

This is the recommended method.
@amitsrivastava78
Copy link
Contributor Author

@omalleyt12 , thanks for the review, i have updated the code as per your suggestion. Kindly check and approve.

Regards
Amit

PR Queue automation moved this from Assigned Reviewer to Approved by Reviewer Mar 29, 2019
@gbaned gbaned added the kokoro:force-run Tests on submitted change label Apr 1, 2019
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Apr 1, 2019
@gbaned gbaned added the ready to pull PR ready for merge process label Apr 1, 2019
@tensorflow-copybara tensorflow-copybara merged commit 289745a into tensorflow:master Apr 2, 2019
PR Queue automation moved this from Approved by Reviewer to Merged Apr 2, 2019
tensorflow-copybara pushed a commit that referenced this pull request Apr 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes comp:keras Keras related issues ready to pull PR ready for merge process size:S CL Change Size: Small
Projects
PR Queue
  
Merged
Development

Successfully merging this pull request may close these issues.

None yet

6 participants