Skip to content

Conversation

@mcarilli
Copy link
Collaborator

@mcarilli mcarilli commented Sep 14, 2020

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@ngimel has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@codecov
Copy link

codecov bot commented Sep 15, 2020

Codecov Report

Merging #44667 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #44667      +/-   ##
==========================================
- Coverage   67.95%   67.95%   -0.01%     
==========================================
  Files         384      384              
  Lines       49705    49705              
==========================================
- Hits        33777    33776       -1     
- Misses      15928    15929       +1     
Impacted Files Coverage Δ
torch/utils/_benchmark/utils/common.py 77.68% <0.00%> (-2.48%) ⬇️
.../testing/_internal/distributed/distributed_test.py 31.10% <0.00%> (+0.05%) ⬆️
torch/testing/_internal/expecttest.py 78.57% <0.00%> (+1.02%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 62ebad4...f6e6373. Read the comment docs.

@facebook-github-bot
Copy link
Contributor

@ngimel merged this pull request in 2fd142a.

xuzhao9 pushed a commit that referenced this pull request Sep 18, 2020
Summary:
requested by https://discuss.pytorch.org/t/what-is-the-correct-way-of-computing-a-grad-penalty-using-amp/95827/3

Pull Request resolved: #44667

Reviewed By: mruberry

Differential Revision: D23692768

Pulled By: ngimel

fbshipit-source-id: 83c61b94e79ef9f86abed2cc066f188dce0c8456
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants