Skip to content

Conversation

@z-a-f
Copy link

@z-a-f z-a-f commented Sep 10, 2020

Stack from ghstack:

Differential Revision: D23637508

z-a-f pushed a commit that referenced this pull request Sep 10, 2020
ghstack-source-id: 6a863a9
Pull Request resolved: #44513
@jerryzh168
Copy link
Contributor

Is there a test exercising this path?

@z-a-f
Copy link
Author

z-a-f commented Sep 10, 2020

Is there a test exercising this path?

I don't think so -- it was not caught by a unittest. I guess, I'll add a test

@dr-ci
Copy link

dr-ci bot commented Sep 10, 2020

💊 CI failures summary and remediations

As of commit 3b106ed (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 5 times.

z-a-f pushed a commit that referenced this pull request Sep 12, 2020
ghstack-source-id: 011e9e6
Pull Request resolved: #44513
@codecov
Copy link

codecov bot commented Sep 13, 2020

Codecov Report

Merging #44513 into gh/z-a-f/61/base will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@                 Coverage Diff                  @@
##           gh/z-a-f/61/base   #44513      +/-   ##
====================================================
- Coverage             67.98%   67.98%   -0.01%     
====================================================
  Files                   384      384              
  Lines                 49567    49567              
====================================================
- Hits                  33697    33696       -1     
- Misses                15870    15871       +1     
Impacted Files Coverage Δ
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 05c1f1d...3b106ed. Read the comment docs.

@facebook-github-bot
Copy link
Contributor

@z-a-f merged this pull request in d0a56ca.

@facebook-github-bot facebook-github-bot deleted the gh/z-a-f/61/head branch September 18, 2020 14:17
xuzhao9 pushed a commit that referenced this pull request Sep 18, 2020
Summary: Pull Request resolved: #44513

Test Plan: Imported from OSS

Reviewed By: jerryzh168

Differential Revision: D23637508

Pulled By: z-a-f

fbshipit-source-id: d19d4c1b234b05e8d9813e864863d937b6c35bf5
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.

5 participants