Skip to content

Conversation

samestep
Copy link
Contributor

Summary: This PR makes fast_nvcc stop at failing commands, rather than continuing on to run commands that would otherwise run after those commands. It is still possible for fast_nvcc to run more commands than nvcc would run if there's no dependency between them, but this should still help to reduce noise from failing fast_nvcc runs.

Test Plan: Unfortunately the test suite for this script is FB-internal. It would probably be a good idea to move it into the PyTorch GitHub repo, but I'm not entirely sure how to do so, since I don't believe we currently have a good place to put tests for things in tools.

Reviewed By: malfet

Differential Revision: D26007788

Summary: This PR makes `fast_nvcc` stop at failing commands, rather than continuing on to run commands that would otherwise run after those commands. It is still possible for `fast_nvcc` to run more commands than `nvcc` would run if there's no dependency between them, but this should still help to reduce noise from failing `fast_nvcc` runs.

Test Plan: Unfortunately the test suite for this script is FB-internal. It would probably be a good idea to move it into the PyTorch GitHub repo, but I'm not entirely sure how to do so, since I don't believe we currently have a good place to put tests for things in `tools`.

Reviewed By: malfet

Differential Revision: D26007788

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

facebook-github-bot commented Jan 25, 2021

💊 CI failures summary and remediations

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


  • 2/2 failures possibly* introduced in this PR
    • 2/2 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 to the (internal) Dr. CI Users group.

@facebook-github-bot
Copy link
Contributor

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

@codecov
Copy link

codecov bot commented Jan 25, 2021

Codecov Report

Merging #51043 (f997244) into master (f7b339d) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #51043      +/-   ##
==========================================
- Coverage   80.91%   80.91%   -0.01%     
==========================================
  Files        1926     1926              
  Lines      210014   210014              
==========================================
- Hits       169942   169939       -3     
- Misses      40072    40075       +3     

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 5adbace.

@samestep samestep deleted the export-D26007788 branch July 14, 2021 22:43
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.

2 participants