Skip to content

Conversation

gchanan
Copy link
Contributor

@gchanan gchanan commented Aug 24, 2020

Stack from ghstack:

The code is also wrong, see #43228.

Differential Revision: D23305461

The code is also wrong, see #43228.
@gchanan gchanan requested a review from apaszke as a code owner August 24, 2020 22:18
gchanan added a commit that referenced this pull request Aug 24, 2020
The code is also wrong, see #43228.

ghstack-source-id: 8c2817c
Pull Request resolved: #43523
@gchanan gchanan requested a review from albanD August 24, 2020 23:27
@dr-ci
Copy link

dr-ci bot commented Aug 25, 2020

💊 CI failures summary and remediations

As of commit 86199dc (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


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 9 times.

Copy link
Collaborator

@albanD albanD left a comment

Choose a reason for hiding this comment

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

Code removal 🎉

@codecov
Copy link

codecov bot commented Aug 27, 2020

Codecov Report

Merging #43523 into gh/gchanan/301/base will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@                 Coverage Diff                  @@
##           gh/gchanan/301/base   #43523   +/-   ##
====================================================
  Coverage                69.32%   69.33%           
====================================================
  Files                      378      378           
  Lines                    46749    46741    -8     
====================================================
- Hits                     32408    32406    -2     
+ Misses                   14341    14335    -6     
Impacted Files Coverage Δ
torch/nn/functional.py 92.04% <ø> (+0.51%) ⬆️
torch/testing/_internal/expecttest.py 77.55% <0.00%> (-1.03%) ⬇️

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 3aeb70d...86199dc. Read the comment docs.

@facebook-github-bot
Copy link
Contributor

@gchanan merged this pull request in 1dcc4fb.

gchanan added a commit to gchanan/pytorch that referenced this pull request Aug 31, 2020
The code is also wrong, see pytorch#43228.

ghstack-source-id: 592ee73
Pull Request resolved: pytorch#43523
@facebook-github-bot facebook-github-bot deleted the gh/gchanan/301/head branch September 4, 2020 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants