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

Added the Padding and Activation scenarios #26062

Merged

Conversation

amitsrivastava78
Copy link
Contributor

This was one of the TODO items

@rthadur rthadur requested a review from aselle February 25, 2019 17:51
@rthadur rthadur self-assigned this Feb 25, 2019
@rthadur rthadur added this to Assigned Reviewer in PR Queue via automation Feb 25, 2019
@rthadur rthadur added awaiting review Pull request awaiting review size:M CL Change Size: Medium labels Feb 25, 2019
@tensorflowbutler
Copy link
Member

Nagging Reviewer @aselle: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 14 days with no activity and the awaiting review label has been applied.

@rthadur rthadur requested review from haozha111 and removed request for aselle March 12, 2019 18:17
@amitsrivastava78
Copy link
Contributor Author

@haozha111 , thanks for pointing this out, i have made the changes as suggested by you, kindly check and approve.

@amitsrivastava78
Copy link
Contributor Author

@haozha111 Sure, made the change as per the comments, kindly check and approve

@tensorflowbutler tensorflowbutler removed the awaiting review Pull request awaiting review label Mar 13, 2019
@amitsrivastava78
Copy link
Contributor Author

@haozha111 , thanks for the feedback, i have updated the code as per your suggestion. Kindly check and approave.

@amitsrivastava78
Copy link
Contributor Author

@haozha111 , sorry for the overlook, i have fixed the issue, kindly check and approve.

This was one of the TODO items
@amitsrivastava78
Copy link
Contributor Author

@haozha111 , I have updated the code as per the suggestions, kindly check and approve,

Regards
Amit

PR Queue automation moved this from Assigned Reviewer to Approved by Reviewer Mar 17, 2019
@rthadur rthadur added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Mar 18, 2019
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Mar 18, 2019
@tensorflow-copybara tensorflow-copybara merged commit 6eb82aa into tensorflow:master Mar 18, 2019
PR Queue automation moved this from Approved by Reviewer to Merged Mar 18, 2019
tensorflow-copybara pushed a commit that referenced this pull request Mar 18, 2019
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 size:M CL Change Size: Medium
Projects
PR Queue
  
Merged
Development

Successfully merging this pull request may close these issues.

None yet

7 participants