Skip to content

Conversation

@ylsung
Copy link
Contributor

@ylsung ylsung commented Sep 20, 2020

As the title.

@ylsung ylsung requested a review from apaszke as a code owner September 20, 2020 07:24
@ylsung ylsung changed the title change typo 'momemtum' to 'momentum' Change typo 'momemtum' to 'momentum' Sep 20, 2020
@dr-ci
Copy link

dr-ci bot commented Sep 20, 2020

💊 CI failures summary and remediations

As of commit 19d6dd9 (more details on the Dr. CI page):


  • 1/1 failures possibly* introduced in this PR
    • 1/1 non-CircleCI failure(s)

ci.pytorch.org: 1 failed


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group.

See how this bot performed.

This comment has been revised 1 time.

@codecov
Copy link

codecov bot commented Sep 20, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #45045   +/-   ##
=======================================
  Coverage   67.83%   67.83%           
=======================================
  Files         384      384           
  Lines       49962    49962           
=======================================
+ Hits        33892    33893    +1     
+ Misses      16070    16069    -1     
Impacted Files Coverage Δ
torch/nn/modules/batchnorm.py 73.79% <ø> (ø)
torch/nn/modules/instancenorm.py 55.26% <ø> (ø)
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 9e5045e...19d6dd9. Read the comment docs.

Copy link
Contributor

@mrshenli mrshenli 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 fixing!

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.

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

@facebook-github-bot
Copy link
Contributor

@mrshenli merged this pull request in f77ba0e.

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