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

Rename NewCriterionTest to CriterionTest. #44056

Closed
wants to merge 2 commits into from

Conversation

gchanan
Copy link
Contributor

@gchanan gchanan commented Sep 2, 2020

Stack from ghstack:

Differential Revision: D23482573

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
[ghstack-poisoned]
@dr-ci
Copy link

dr-ci bot commented Sep 2, 2020

💊 CI failures summary and remediations

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



❄️ 1 failure tentatively classified as flaky

but reruns have not yet been triggered to confirm:

See CircleCI build pytorch_linux_bionic_py3_8_gcc9_coverage_test (1/1)

Step: "Run tests" (full log | diagnosis details | 🔁 rerun) ❄️

Sep 02 22:35:47 RuntimeError: Process 2 terminated or timed out after 100.05614304542542 seconds
Sep 02 22:35:47 ====================================================================== 
Sep 02 22:35:47 ERROR [100.100s]: test_backward_node_failure (__main__.ProcessGroupDistAutogradTestWithSpawn) 
Sep 02 22:35:47 ---------------------------------------------------------------------- 
Sep 02 22:35:47 Traceback (most recent call last): 
Sep 02 22:35:47   File "/opt/conda/lib/python3.8/site-packages/torch/testing/_internal/common_distributed.py", line 224, in wrapper 
Sep 02 22:35:47     self._join_processes(fn) 
Sep 02 22:35:47   File "/opt/conda/lib/python3.8/site-packages/torch/testing/_internal/common_distributed.py", line 337, in _join_processes 
Sep 02 22:35:47     self._check_return_codes(elapsed_time) 
Sep 02 22:35:47   File "/opt/conda/lib/python3.8/site-packages/torch/testing/_internal/common_distributed.py", line 375, in _check_return_codes 
Sep 02 22:35:47     raise RuntimeError('Process {} terminated or timed out after {} seconds'.format(i, elapsed_time)) 
Sep 02 22:35:47 RuntimeError: Process 2 terminated or timed out after 100.05614304542542 seconds 
Sep 02 22:35:47  
Sep 02 22:35:47 ---------------------------------------------------------------------- 
Sep 02 22:35:47 Ran 305 tests in 524.691s 
Sep 02 22:35:47  
Sep 02 22:35:47 FAILED (errors=1, skipped=6) 
Sep 02 22:35:47  
Sep 02 22:35:47 Generating XML reports... 
Sep 02 22:35:47 Generated XML report: test-reports/dist-gloo/TEST-ProcessGroupDdpComparisonTestWithSpawn-20200902222702.xml 
Sep 02 22:35:47 Generated XML report: test-reports/dist-gloo/TEST-ProcessGroupDdpUnderDistAutogradTestWithSpawn-20200902222702.xml 
Sep 02 22:35:47 Generated XML report: test-reports/dist-gloo/TEST-ProcessGroupDistAutogradTestWithSpawn-20200902222702.xml 

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.

@gchanan gchanan changed the title Rename CriterionTest to NewCriterionTest. Rename NewCriterionTest to CriterionTest. Sep 3, 2020
@gchanan gchanan requested a review from zou3519 September 3, 2020 14:57
@facebook-github-bot
Copy link
Contributor

@gchanan merged this pull request in 5973b44.

@facebook-github-bot facebook-github-bot deleted the gh/gchanan/317/head branch September 7, 2020 14:16
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.

None yet

4 participants