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

Docs: more details about CUDA models with multiprocessing #1338

Closed
Jiaming-Liu opened this issue Apr 23, 2017 · 2 comments
Closed

Docs: more details about CUDA models with multiprocessing #1338

Jiaming-Liu opened this issue Apr 23, 2017 · 2 comments

Comments

@Jiaming-Liu
Copy link
Contributor

In pytorch/docs/source/notes/cuda.rst, it is mentioned that:

unless care is taken to meet the data handling requirements exactly, it is likely that your program will have incorrect or undefined behavior.

IMO, more details on the requirements would be helpful for this part.

@apaszke
Copy link
Contributor

apaszke commented Apr 24, 2017

The most important thing are the requirements of the CUDA API. I'd discourage its use unless you know some lower level detail of CUDA IPC. The most important limitation is that if you allocate a CUDA tensor X in process A and send it to process B, then X should never go out of scope and be freed in A until it's used in B (NOTE: it's automatically freed upon A's exit). So it might be ok for sharing CUDA parameters across processes, but not e.g. for data loading.

@soumith soumith added this to Medium Priority in Issue Status Aug 23, 2017
@soumith soumith added this to usability / simple-fixes in Issue Categories Sep 13, 2017
@soumith soumith closed this as completed Apr 13, 2018
eqy pushed a commit to eqy/pytorch that referenced this issue Jan 20, 2022
fixes the assertion from pytorch#1325 on our devel branch.

1. update alias information after graph mutation
2. patch unsqueeze: i. support negative dimension; ii. fixing range check
@pytorch-bot
Copy link

pytorch-bot bot commented Jul 29, 2022

🔗 Helpful Links

🧪 See artifacts and rendered test results here

Note: Links to docs will display an error until the docs builds have been completed.

This comment was automatically generated by Dr. CI and updates every 15 minutes.

jithunnair-amd pushed a commit that referenced this issue Mar 18, 2024
Co-authored-by: Wang, Yanyao <yanyao.wang@amd.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Issue Categories
usability / simple-fixes
Issue Status
Medium Priority
Development

No branches or pull requests

3 participants