Skip to content

[CUDA] Add addmv for complex dtypes#40238

Closed
anjali411 wants to merge 9 commits intomasterfrom
addmv_complex
Closed

[CUDA] Add addmv for complex dtypes#40238
anjali411 wants to merge 9 commits intomasterfrom
addmv_complex

Conversation

@anjali411
Copy link
Copy Markdown
Contributor

No description provided.

@anjali411 anjali411 requested a review from zasdfgbnm June 18, 2020 20:24
@anjali411 anjali411 added the module: complex Related to complex number support in PyTorch label Jun 18, 2020
@dr-ci
Copy link
Copy Markdown

dr-ci Bot commented Jun 18, 2020

💊 CI failures summary and remediations

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


  • 5/5 failures introduced in this PR

🕵️ 5 new failures recognized by patterns

The following CI failures do not appear to be due to upstream breakages:

See CircleCI build pytorch_windows_vs2019_py36_cpu_test1 (1/5)

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

RuntimeError: test_nn failed!
  File "C:\Jenkins\Miniconda3\lib\site-packages\scipy\_distributor_init.py", line 61, in <module> 
    WinDLL(os.path.abspath(filename)) 
  File "C:\Jenkins\Miniconda3\lib\ctypes\__init__.py", line 348, in __init__ 
    self._handle = _dlopen(self._name, mode) 
OSError: [WinError 126] The specified module could not be found 
Traceback (most recent call last): 
  File "run_test.py", line 721, in <module> 
    main() 
  File "run_test.py", line 714, in main 
    raise RuntimeError(message) 
RuntimeError: test_nn failed! 
 
(base) circleci@PACKER-5EE89590 C:\Users\circleci\project\test>if ERRORLEVEL 1 exit /b 1  
+ cleanup
+ retcode=1
+ set +x

See CircleCI build pytorch-linux-xenial-py3-clang5-android-ndk-r19c-gradle-build-x86_32 (2/5)

Step: "pytorch android gradle build only x86_32 (for PR)" (full log | diagnosis details | 🔁 rerun)

Jun 22 14:51:08 ./.circleci/scripts/build_android_gradle.sh: line 73: /opt/gradle/gradle-6.5/bin/gradle: No such file or directory
Jun 22 14:51:08 ANDROID_NDK_HOME=/opt/ndk 
Jun 22 14:51:08 INSTALLED_DB= 
Jun 22 14:51:08 _=/usr/bin/env 
Jun 22 14:51:08 + export GRADLE_LOCAL_PROPERTIES=/var/lib/jenkins/workspace/android/local.properties 
Jun 22 14:51:08 + GRADLE_LOCAL_PROPERTIES=/var/lib/jenkins/workspace/android/local.properties 
Jun 22 14:51:08 + rm -f /var/lib/jenkins/workspace/android/local.properties 
Jun 22 14:51:08 + echo sdk.dir=/opt/android/sdk 
Jun 22 14:51:08 + echo ndk.dir=/opt/ndk 
Jun 22 14:51:08 + echo cmake.dir=/usr/local 
Jun 22 14:51:08 + /opt/gradle/gradle-6.5/bin/gradle -p android assembleRelease --debug --stacktrace -PABI_FILTERS=x86 --offline 
Jun 22 14:51:08 ./.circleci/scripts/build_android_gradle.sh: line 73: /opt/gradle/gradle-6.5/bin/gradle: No such file or directory 

See CircleCI build pytorch_windows_vs2019_py36_cuda10.1_on_cpu_test1 (3/5)

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

RuntimeError: test_nn failed!
  File "C:\Jenkins\Miniconda3\lib\site-packages\scipy\_distributor_init.py", line 61, in <module> 
    WinDLL(os.path.abspath(filename)) 
  File "C:\Jenkins\Miniconda3\lib\ctypes\__init__.py", line 348, in __init__ 
    self._handle = _dlopen(self._name, mode) 
OSError: [WinError 126] The specified module could not be found 
Traceback (most recent call last): 
  File "run_test.py", line 721, in <module> 
    main() 
  File "run_test.py", line 714, in main 
    raise RuntimeError(message) 
RuntimeError: test_nn failed! 
 
(base) circleci@PACKER-5ECD3249 C:\Users\circleci\project\test>if ERRORLEVEL 1 exit /b 1  
+ cleanup
+ retcode=1
+ set +x

See CircleCI build pytorch_windows_vs2019_py36_cuda10.1_test1 (4/5)

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

RuntimeError: test_nn failed!
  File "C:\Jenkins\Miniconda3\lib\site-packages\scipy\_distributor_init.py", line 61, in <module> 
    WinDLL(os.path.abspath(filename)) 
  File "C:\Jenkins\Miniconda3\lib\ctypes\__init__.py", line 348, in __init__ 
    self._handle = _dlopen(self._name, mode) 
OSError: [WinError 126] The specified module could not be found 
Traceback (most recent call last): 
  File "run_test.py", line 721, in <module> 
    main() 
  File "run_test.py", line 714, in main 
    raise RuntimeError(message) 
RuntimeError: test_nn failed! 
 
(base) circleci@PACKER-5EE89583 C:\Users\circleci\project\test>if ERRORLEVEL 1 exit /b 1  
+ cleanup
+ retcode=1
+ set +x

See CircleCI build pytorch_windows_vs2019_py36_cuda10.1_test2 (5/5)

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

RuntimeError: test_cuda failed!
  File "C:\Jenkins\Miniconda3\lib\site-packages\scipy\_distributor_init.py", line 61, in <module> 
    WinDLL(os.path.abspath(filename)) 
  File "C:\Jenkins\Miniconda3\lib\ctypes\__init__.py", line 348, in __init__ 
    self._handle = _dlopen(self._name, mode) 
OSError: [WinError 126] The specified module could not be found 
Traceback (most recent call last): 
  File "run_test.py", line 721, in <module> 
    main() 
  File "run_test.py", line 714, in main 
    raise RuntimeError(message) 
RuntimeError: test_cuda failed! 
 
(base) circleci@PACKER-5ECD3242 C:\Users\circleci\project\test>if ERRORLEVEL 1 exit /b 1  
+ cleanup
+ retcode=1
+ set +x

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 37 times.

Comment thread aten/src/ATen/cuda/CUDABlas.h Outdated
@zasdfgbnm
Copy link
Copy Markdown
Collaborator

I remember I saw this PR before, was that one reverted due to windows failure?

@anjali411
Copy link
Copy Markdown
Contributor Author

I remember I saw this PR before, was that one reverted due to windows failure?

yeah

Comment thread aten/src/ATen/native/cuda/Blas.cu Outdated
Comment thread aten/src/ATen/cuda/CUDABlas.cpp Outdated

template <>
void gemv<c10::complex<double>>(CUDABLAS_GEMV_ARGTYPES(c10::complex<double>)) {
cublasHandle_t handle = at::cuda::getCurrentCUDABlasHandle();
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe you need to ifdef the function body for ROCm as well?

Copy link
Copy Markdown
Collaborator

@zasdfgbnm zasdfgbnm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, given that ROCm CI is not failing

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

@anjali411 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@anjali411 merged this pull request in c72ab19.

@facebook-github-bot facebook-github-bot deleted the addmv_complex branch July 13, 2020 17:53
@anjali411 anjali411 changed the title Add addmv for complex dtypes [CUDA] Add addmv for complex dtypes Jul 22, 2020
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary: Pull Request resolved: pytorch#40238

Differential Revision: D22160528

Pulled By: anjali411

fbshipit-source-id: 04093e5929318a7acc9c9b502c76d0a8bf15d5e1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged module: complex Related to complex number support in PyTorch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants