Skip to content

Also set stdin to subprocess pipe in FindCUDA windows popen call#10379

Closed
mattdawkins wants to merge 1 commit intopytorch:masterfrom
mattdawkins:fix-popen-call
Closed

Also set stdin to subprocess pipe in FindCUDA windows popen call#10379
mattdawkins wants to merge 1 commit intopytorch:masterfrom
mattdawkins:fix-popen-call

Conversation

@mattdawkins
Copy link
Contributor

Background: we run pytorch in embedded C++ pipelines, running in C++ GUIs in https://github.com/Kitware/VIAME and without this addition, the call was failing with the below error, but only on certain windows platforms/configurations:

OSError: [WinError6] The handle is invalid
At:
C:\Program Files\VIAME\Python36\site-packages\torch\cuda_init_.py(162):_lazy_init
C:\Program Files\VIAME\Python36\site-packages\torch\nn\modules\module.py(249):
C:\Program Files\VIAME\Python36\site-packages\torch\nn\modules\module.py(182): _apply
C:\Program Files\VIAME\Python36\site-packages\torch\nn\modules\module.py(176): _apply
C:\Program Files\VIAME\Python36\site-packages\torch\nn\modules\module.py(249): cuda
C:\Program Files\VIAME\lib\python3.6None\site-packages\kwiver\arrows\pytorch\pytorch_resnet_f_extractor.py(74):init
C:\Program Files\VIAME\lib\python3.6None\site-packages\kwiver\processes\resnet_descriptors.py(132): _configure

@mattdawkins
Copy link
Contributor Author

Full remaining log:

OSError: [WinError6] The handle is invalid

At:
C:\ProgramData\Anaconda3\Lib\subprocess.py(955): make_inheritable
C:\ProgramData\Anaconda3\Lib\subprocess.py(905): get_handles
C:\ProgramData\Anaconda3\Lib\subprocess.py(667): init
C:\Program Files\VIAME\Python36\site-packages\torch\cuda_init
.py(30): find_cuda_windows_lib
C:\Program Files\VIAME\Python36\site-packages\torch\cuda_init
.py(59): load_cudart
C:\Program Files\VIAME\Python36\site-packages\torch\cuda_init
.py(162): _lazy_init

@ssnl
Copy link
Collaborator

ssnl commented Aug 14, 2018

@peterjc123 does this seem reasonable?

@peterjc123
Copy link
Collaborator

@ssnl I think it should be okay.

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

ezyang is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
…orch#10379)

Summary:
Background: we run pytorch in embedded C++ pipelines, running in C++ GUIs in https://github.com/Kitware/VIAME and without this addition, the call was failing with the below error, but only on certain windows platforms/configurations:

OSError: [WinError6] The handle is invalid
At:
C:\Program Files\VIAME\Python36\site-packages\torch\cuda_init_.py(162):_lazy_init
C:\Program Files\VIAME\Python36\site-packages\torch\nn\modules\module.py(249): <lambda>
C:\Program Files\VIAME\Python36\site-packages\torch\nn\modules\module.py(182): _apply
C:\Program Files\VIAME\Python36\site-packages\torch\nn\modules\module.py(176): _apply
C:\Program Files\VIAME\Python36\site-packages\torch\nn\modules\module.py(249): cuda
C:\Program Files\VIAME\lib\python3.6None\site-packages\kwiver\arrows\pytorch\pytorch_resnet_f_extractor.py(74):_init_
C:\Program Files\VIAME\lib\python3.6None\site-packages\kwiver\processes\resnet_descriptors.py(132): _configure
Pull Request resolved: pytorch#10379

Differential Revision: D9330772

Pulled By: ezyang

fbshipit-source-id: 657ae7590879004558158d3c4abef2ec11d9ed57
facebook-github-bot pushed a commit that referenced this pull request Sep 9, 2018
…1435)

Summary:
Same issue as #10379, just in a different place (adding this resolves it)
Pull Request resolved: #11435

Differential Revision: D9736396

Pulled By: soumith

fbshipit-source-id: 220a52b8009fc2bee9313c5a091443c68f85f62f
PenghuiCheng pushed a commit to PenghuiCheng/pytorch that referenced this pull request Sep 11, 2018
…torch#11435)

Summary:
Same issue as pytorch#10379, just in a different place (adding this resolves it)
Pull Request resolved: pytorch#11435

Differential Revision: D9736396

Pulled By: soumith

fbshipit-source-id: 220a52b8009fc2bee9313c5a091443c68f85f62f
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.

5 participants