Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When creating a single parition skip the output nodes, but process possible nodes after it. #60370

Closed
wants to merge 1 commit into from

Conversation

protonu
Copy link

@protonu protonu commented Jun 21, 2021

Summary: When creating a single parition skip the output nodes, but process possible nodes after it.

Test Plan: Run all CI tests.

Differential Revision: D29265278

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Jun 21, 2021

💊 CI failures summary and remediations

As of commit acef0f6 (more details on the Dr. CI page and at hud.pytorch.org/pr/60370):


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

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 to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D29265278

1 similar comment
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D29265278

@codecov
Copy link

codecov bot commented Jun 22, 2021

Codecov Report

Merging #60370 (acef0f6) into master (48509b1) will increase coverage by 8.22%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master   #60370      +/-   ##
==========================================
+ Coverage   67.88%   76.11%   +8.22%     
==========================================
  Files        2019     2062      +43     
  Lines      204496   206154    +1658     
==========================================
+ Hits       138825   156905   +18080     
+ Misses      65671    49249   -16422     

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D29265278

…ssible nodes after it. (pytorch#60370)

Summary:
Pull Request resolved: pytorch#60370

When creating a single parition skip the output nodes, but process possible nodes after it.

Test Plan: Run all CI tests.

Reviewed By: jfix71

Differential Revision: D29265278

fbshipit-source-id: c8f13a6bb8099bbe6b20e1d9f0d1be79017c42ac
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D29265278

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 5b1f5c8.

asuhan pushed a commit to asuhan/pytorch that referenced this pull request Jun 28, 2021
…ssible nodes after it. (pytorch#60370)

Summary:
Pull Request resolved: pytorch#60370

When creating a single parition skip the output nodes, but process possible nodes after it.

Test Plan: Run all CI tests.

Reviewed By: jfix71

Differential Revision: D29265278

fbshipit-source-id: 2242009973a54498d8027cce5a294558a1206fdf
asuhan pushed a commit that referenced this pull request Jun 30, 2021
…ssible nodes after it. (#60370)

Summary:
Pull Request resolved: #60370

When creating a single parition skip the output nodes, but process possible nodes after it.

Test Plan: Run all CI tests.

Reviewed By: jfix71

Differential Revision: D29265278

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

None yet

2 participants