Skip to content

Conversation

@shubhambhokare1
Copy link
Collaborator

Update arange op to support out argument

@dr-ci
Copy link

dr-ci bot commented Aug 28, 2020

💊 CI failures summary and remediations

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


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

Extra GitHub checks: 1 failed


ci.pytorch.org: 1 failed


codecov.io: 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 19 times.

@ezyang ezyang added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label Sep 1, 2020
@codecov
Copy link

codecov bot commented Sep 2, 2020

Codecov Report

Merging #43777 into master will decrease coverage by 0.02%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #43777      +/-   ##
==========================================
- Coverage   68.00%   67.97%   -0.03%     
==========================================
  Files         382      382              
  Lines       49391    49409      +18     
==========================================
  Hits        33586    33586              
- Misses      15805    15823      +18     
Impacted Files Coverage Δ
torch/onnx/symbolic_helper.py 42.94% <0.00%> (ø)
torch/onnx/symbolic_opset11.py 22.96% <0.00%> (-0.22%) ⬇️
torch/onnx/symbolic_opset9.py 35.05% <0.00%> (-0.32%) ⬇️

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 2a87742...c68dd39. Read the comment docs.

Copy link
Contributor

@neginraoof neginraoof left a comment

Choose a reason for hiding this comment

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

Thanks!

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.

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

@facebook-github-bot
Copy link
Contributor

@bzinodev merged this pull request in da11d93.

xuzhao9 pushed a commit that referenced this pull request Sep 18, 2020
Summary:
Update arange op to support out argument

Pull Request resolved: #43777

Reviewed By: albanD

Differential Revision: D23674583

Pulled By: bzinodev

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

Labels

Merged open source triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants