-
Notifications
You must be signed in to change notification settings - Fork 125
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
[NDTensorsCUDAExt] Fix bug in slicing Adjoint CuMatrix #1236
Conversation
Thanks! Could you add a regression test in NDTensors? Also maybe a good time to add a test for DMRG using NDTensorsCUDAExt as we discussed. |
… into kmp5/debug/issue_1232
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #1236 +/- ##
===========================================
- Coverage 85.42% 54.14% -31.29%
===========================================
Files 89 87 -2
Lines 8400 7853 -547
===========================================
- Hits 7176 4252 -2924
- Misses 1224 3601 +2377 ☔ View full report in Codecov by Sentry. |
…alls generic contraction on CPU. This can be fixed by
Co-authored-by: Matt Fishman <[email protected]>
Looks good, thanks for sticking with all of the comments! |
Description
CUDA was numerically failing for DMRG when the number of sites was above 8 this is related to a truncation bug found in SVD. This merge should fix this bug and give DMRG equivalent results on GPU and CPU.