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

[ROCm] Add ROCm support for 2D and 3D convolution ops. #29185

Merged
merged 3 commits into from Jun 28, 2019

Conversation

whchung
Copy link
Contributor

@whchung whchung commented May 30, 2019

MIOpen has its own algorithm find logic and would return the size of scratch
memory required to client applications. Modify AlgorithmConfig to track such
information.

This is a follow-up PR for #29172 .

@tensorflow-bot tensorflow-bot bot added the size:M CL Change Size: Medium label May 30, 2019
@whchung whchung requested a review from chsigg May 30, 2019 20:31
@rthadur rthadur self-assigned this May 30, 2019
@rthadur rthadur added this to Assigned Reviewer in PR Queue via automation May 30, 2019
@rthadur rthadur added the cuda label May 30, 2019
@rthadur rthadur requested review from chsigg and removed request for chsigg May 30, 2019 20:54
@whchung
Copy link
Contributor Author

whchung commented Jun 5, 2019

looking at the log I don't believe those failures are relevant to changes proposed in this PR.

@whchung whchung added the kokoro:force-run Tests on submitted change label Jun 7, 2019
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Jun 7, 2019
tensorflow/core/kernels/conv_grad_ops_3d.cc Outdated Show resolved Hide resolved
tensorflow/stream_executor/dnn.h Outdated Show resolved Hide resolved
@rthadur rthadur requested a review from chsigg June 12, 2019 19:43
@whchung
Copy link
Contributor Author

whchung commented Jun 13, 2019

looking at the log I don't believe those failures are relevant to changes proposed in this PR.

@whchung
Copy link
Contributor Author

whchung commented Jun 18, 2019

@chsigg a gentle ping.

MIOpen has its own algorithm find logic and would return the size of scratch
memory required to client applications. Modify AlgorithmConfig to track such
information.
@whchung whchung added the kokoro:force-run Tests on submitted change label Jun 26, 2019
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Jun 26, 2019
@whchung whchung added the kokoro:force-run Tests on submitted change label Jun 27, 2019
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Jun 27, 2019
@dagamayank
Copy link

@chsigg a gentle ping for this particular PR.

@whchung
Copy link
Contributor Author

whchung commented Jun 27, 2019

Again looking at the log I don't believe those failures are relevant to changes proposed in this PR.

@tensorflow-bot tensorflow-bot bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Jun 27, 2019
PR Queue automation moved this from Assigned Reviewer to Approved by Reviewer Jun 27, 2019
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Jun 27, 2019
@tensorflow-copybara tensorflow-copybara merged commit e3dd27c into tensorflow:master Jun 28, 2019
PR Queue automation moved this from Approved by Reviewer to Merged Jun 28, 2019
tensorflow-copybara pushed a commit that referenced this pull request Jun 28, 2019
@whchung whchung deleted the google_upstream_conv_ops branch December 20, 2019 17:38
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