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

Eliminate a division by 0 in 3D convolutions. #49356

Merged
merged 3 commits into from May 24, 2021

Conversation

geetachavan1
Copy link
Contributor

No description provided.

@google-cla
Copy link

google-cla bot commented May 20, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@google-cla google-cla bot added the cla: no label May 20, 2021
@gbaned gbaned self-assigned this May 20, 2021
@gbaned gbaned added cla: yes comp:core issues related to core part of tensorflow size:M CL Change Size: Medium and removed cla: no labels May 20, 2021
@gbaned gbaned added this to Assigned Reviewer in PR Queue via automation May 20, 2021
@gbaned gbaned requested a review from mihaimaruseac May 20, 2021 06:50
@gbaned gbaned assigned mihaimaruseac and unassigned gbaned May 20, 2021
PR Queue automation moved this from Assigned Reviewer to Reviewer Requested Changes May 20, 2021
tensorflow/core/kernels/conv_grad_ops_3d.cc Outdated Show resolved Hide resolved
tensorflow/core/kernels/conv_grad_ops_3d.cc Outdated Show resolved Hide resolved
tensorflow/core/kernels/conv_grad_ops_3d.cc Outdated Show resolved Hide resolved
tensorflow/core/kernels/conv_grad_ops_3d.cc Outdated Show resolved Hide resolved
@google-cla
Copy link

google-cla bot commented May 23, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@google-cla google-cla bot added cla: no and removed cla: yes labels May 23, 2021
PR Queue automation moved this from Reviewer Requested Changes to Approved by Reviewer May 23, 2021
@google-cla
Copy link

google-cla bot commented May 23, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@google-cla google-cla bot added cla: no and removed cla: yes labels May 23, 2021
@mihaimaruseac mihaimaruseac merged commit 69f1467 into tensorflow:r2.4 May 24, 2021
PR Queue automation moved this from Approved by Reviewer to Merged May 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes comp:core issues related to core part of tensorflow size:M CL Change Size: Medium
Projects
PR Queue
  
Merged
Development

Successfully merging this pull request may close these issues.

None yet

3 participants