Skip to content

Fix code review from #48659 and #48116#48731

Closed
ezyang wants to merge 2 commits intogh/ezyang/883/basefrom
gh/ezyang/883/head
Closed

Fix code review from #48659 and #48116#48731
ezyang wants to merge 2 commits intogh/ezyang/883/basefrom
gh/ezyang/883/head

Conversation

@ezyang
Copy link
Copy Markdown
Contributor

@ezyang ezyang commented Dec 2, 2020

Stack from ghstack:

Signed-off-by: Edward Z. Yang ezyang@fb.com

Differential Revision: D25278034

Signed-off-by: Edward Z. Yang <ezyang@fb.com>

[ghstack-poisoned]
Signed-off-by: Edward Z. Yang <ezyang@fb.com>

[ghstack-poisoned]
@ezyang ezyang requested a review from bhosmer December 2, 2020 19:02
@codecov
Copy link
Copy Markdown

codecov bot commented Dec 3, 2020

Codecov Report

Merging #48731 (1ebd284) into gh/ezyang/883/base (8e534f4) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@                  Coverage Diff                   @@
##           gh/ezyang/883/base   #48731      +/-   ##
======================================================
- Coverage               80.83%   80.83%   -0.01%     
======================================================
  Files                    1860     1860              
  Lines                  200668   200678      +10     
======================================================
+ Hits                   162218   162223       +5     
- Misses                  38450    38455       +5     

ezyang added a commit to ezyang/pytorch that referenced this pull request Dec 3, 2020
Signed-off-by: Edward Z. Yang <ezyang@fb.com>

ghstack-source-id: 1de9cce
Pull Request resolved: pytorch#48731
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@ezyang merged this pull request in f9a0abf.

@facebook-github-bot facebook-github-bot deleted the gh/ezyang/883/head branch December 7, 2020 15:21
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.

3 participants