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
Multiprocessing support for NT #110292
Multiprocessing support for NT #110292
Conversation
[ghstack-poisoned]
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/110292
Note: Links to docs will display an error until the docs builds have been completed. ✅ You can merge normally! (2 Unrelated Failures)As of commit a514685 with merge base 46a5558 (): UNSTABLE - The following jobs failed but were likely due to flakiness present on trunk and has been marked as unstable:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
ghstack-source-id: 710581dcefd934ef187785e1fa396a0f0927e488 Pull Request resolved: #110292
@pytorchbot merge |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Merge failedReason: 1 jobs have failed, first few of them are: trunk / macos-12-py3-arm64 / test (default, 1, 3, macos-m1-12) Details for Dev Infra teamRaised by workflow job |
Failures look real unfortunately @jbschlosser |
Fixes #110161 Allows NTs to be used in DataLoaders with `num_workers > 1`. [ghstack-poisoned]
ghstack-source-id: 8aae0acad796757c7f9f5aad6379c56ad901a1dd Pull Request resolved: #110292
@pytorchbot merge |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
❌ 🤖 pytorchbot command failed:
Try |
❌ 🤖 pytorchbot command failed:
Try |
@pytorchbot revert -m "Address review comments" -c "weird" |
@pytorchbot successfully started a revert job. Check the current status here. |
Fixes #110161 Allows NTs to be used in DataLoaders with `num_workers > 1`. [ghstack-poisoned]
@pytorchbot merge |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Merge failedReason: 1 jobs have failed, first few of them are: trunk / macos-12-py3-arm64 / build Details for Dev Infra teamRaised by workflow job |
@pytorchbot merge -f "ignore spurious failure" |
Merge startedYour change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Pull Request resolved: #110404 Approved by: https://github.com/cpuhrsch, https://github.com/albanD ghstack dependencies: #110292
@pytorchbot revert -m "Causes CUDA memory leaks" -c nosignal
https://github.com/pytorch/pytorch/actions/runs/6425541384/job/17449001020 |
@pytorchbot successfully started a revert job. Check the current status here. |
@jbschlosser your PR has been successfully reverted. |
This reverts commit f17fe89. Reverted #110292 on behalf of https://github.com/kit1980 due to Causes CUDA memory leaks ([comment](#110292 (comment)))
Fixes #110161 Allows NTs to be used in DataLoaders with `num_workers > 1`. [ghstack-poisoned]
@pytorchbot merge |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Pull Request resolved: #110404 Approved by: https://github.com/cpuhrsch, https://github.com/albanD ghstack dependencies: #110292
Stack from ghstack (oldest at bottom):
Fixes #110161
Allows NTs to be used in DataLoaders with
num_workers > 1
.