Skip to content

Reduce excessive CI printing in TestHub#22043

Closed
ssnl wants to merge 1 commit intopytorch:masterfrom
ssnl:ci_prt
Closed

Reduce excessive CI printing in TestHub#22043
ssnl wants to merge 1 commit intopytorch:masterfrom
ssnl:ci_prt

Conversation

@ssnl
Copy link
Copy Markdown
Collaborator

@ssnl ssnl commented Jun 20, 2019

#21132 reverted #19606.

Now these tests again print like 40% lines of CI outputs (e.g., https://circleci.com/gh/pytorch/pytorch/2041825?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link)

This PR now uses the functionality introduced in pytorch/vision#862.

@pytorchbot pytorchbot added the module: tests Issues related to tests (not the torch.testing module) label Jun 20, 2019
@ssnl
Copy link
Copy Markdown
Collaborator Author

ssnl commented Jun 20, 2019

cc @ailzhang

Copy link
Copy Markdown
Contributor

@ailzhang ailzhang left a comment

Choose a reason for hiding this comment

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

Yea I forgot the progress bar :P
Thanks a lot for the fix!

@ssnl
Copy link
Copy Markdown
Collaborator Author

ssnl commented Jun 21, 2019

@ailzhang Test failure is bogus. I think this is good to go :)

Copy link
Copy Markdown
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.

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

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@ailzhang merged this pull request in 7d637de.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged module: tests Issues related to tests (not the torch.testing module) open source

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants