-
Notifications
You must be signed in to change notification settings - Fork 17
CUDA, CUTENSOR, CUQUANTUM #580
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
Comments
Are you discussing the presets created in #579? With the current settings (without #579), enabling CUDA will not automatically enable cuTENSOR and cuQuantum. In #579, CMakeLists.txt follows the current settings. cuTENSOR and cuQuantum are not enabled when configuring with the command I updated the PR message of #579. |
Question 1: Can one build gpu support without using CUTENSOR and CUQUANTUM? |
With commit 46dc390 If I do
|
I have to explicitly turn off the cutensor/cuquantum.
|
Hmm. I just find out that: If I do If I do
|
I checked CMakeLists.txt. Yes, the current default setting in Install.sh and CMakeLists.txt are not aligned. With #579, the behavior in CMakeLists.txt is aligned with the behavior in the current Install.sh. cuTENSOR and cuQuantum are not enabled with CUDA. |
Because I fail to compile gpu support when I turn off cutensor/cuquantum. I am wondering, with the current code.
|
No, I can't either. I think it's a bug. Changing this line Line 1099 in 5256162
to std::shared_ptr<Node> root = this->CtTree.nodes_container.back(); makes build sucessful. |
This is to confirm that, after changing line 1099 to
It compiles OK. |
Following this thread, I propose to using CuQuantum as the main dependency to avoid the complicated compiling options. |
Also, I wonder if CUTT is already integrated into cuTensorNet in cuQuantum? |
It seems that if I turn cuda option on via
-DUSE_CUDA=ON
, then one automatically turns on cutensor and cuquantum. If this is the case, then maybe we don't need-DUSE_CUTENSOR=ON
and-DUSE_CUQUANTUM=ON
?The text was updated successfully, but these errors were encountered: