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

[resubmit] [pytorch][PR] Fix for num_threads==1 in OpenMP "parallel for" #39533

Closed
wants to merge 1 commit into from

Conversation

ilia-cher
Copy link
Contributor

Test Plan: CI

Differential Revision: D21889269

@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

…or" (pytorch#39533)

Summary: Pull Request resolved: pytorch#39533

Test Plan: CI

Differential Revision: D21889269

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

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

@dr-ci
Copy link

dr-ci bot commented Jun 12, 2020

💊 CI failures summary and remediations

As of commit 77ab8f2 (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_bazel_test (1/1)

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

Jun 12 05:26:31 TIMEOUT: //:integration_test (Summary)
Jun 12 05:24:38 [       OK ] OptimTest.AddParameter_LBFGS (1 ms) 
Jun 12 05:24:38 [----------] 28 tests from OptimTest (287227 ms total) 
Jun 12 05:24:38  
Jun 12 05:24:38 [----------] Global test environment tear-down 
Jun 12 05:24:38 [==========] 28 tests from 1 test suite ran. (287227 ms total) 
Jun 12 05:24:38 [  PASSED  ] 28 tests. 
Jun 12 05:24:38 ================================================================================ 
Jun 12 05:24:47 [2,526 / 2,527] 24 / 25 tests; Testing //:integration_test; 211s processwrapper-sandbox 
Jun 12 05:25:44 [2,526 / 2,527] 24 / 25 tests; Testing //:integration_test; 268s processwrapper-sandbox 
Jun 12 05:26:31  
Jun 12 05:26:31 TIMEOUT: //:integration_test (Summary) 
Jun 12 05:26:31       /var/lib/jenkins/.cache/bazel/_bazel_jenkins/fdf6d09bf4b4f04a71e2a7dfceb40620/execroot/pytorch/bazel-out/k8-fastbuild/testlogs/integration_test/test.log 
Jun 12 05:26:31 INFO: From Testing //:integration_test: 
Jun 12 05:26:31 ==================== Test output for //:integration_test: 
Jun 12 05:26:31 Running main() from gmock_main.cc 
Jun 12 05:26:31 Note: Google Test filter = -*CUDA 
Jun 12 05:26:31 [==========] Running 1 test from 1 test suite. 
Jun 12 05:26:31 [----------] Global test environment set-up. 
Jun 12 05:26:31 [----------] 1 test from IntegrationTest 
Jun 12 05:26:31 [ RUN      ] IntegrationTest.CartPole 
Jun 12 05:26:31 -- Test timed out at 2020-06-12 05:26:16 UTC -- 

🚧 1 fixed upstream failure:

These were probably caused by upstream breakages that were already fixed.

Please rebase on the viable/strict branch (expand for instructions)

Since your merge base is older than viable/strict, run these commands:

git fetch https://github.com/pytorch/pytorch viable/strict
git rebase FETCH_HEAD

Check out the recency history of this "viable master" tracking branch.


ci.pytorch.org: 2 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 4 times.

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in cc3fc78.

xwang233 pushed a commit to xwang233/pytorch that referenced this pull request Jun 20, 2020
…or" (pytorch#39533)

Summary: Pull Request resolved: pytorch#39533

Test Plan: CI

Reviewed By: ngimel

Differential Revision: D21889269

fbshipit-source-id: 5ba13a0a3ec11edd0b6a7c3fdb35396b847a3d9e
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.

3 participants