-
Notifications
You must be signed in to change notification settings - Fork 89
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
GPU broken #966
Comments
OK I fixed it directly on master (hopefully) can you test? |
Also should we report this to a GPU package? Which one? |
Does not yet fix it:
BTW can you see the output at https://git.uni-paderborn.de/herbstm/DFTK.jl/-/jobs/258733 ? If yes you can debug this yourself, by making a branch ending in name in |
Also I think your current fix does not offload to GPU in case the eigenvalues are already sorted, right ? |
Fixed now. |
Sorry completely forgot about this, great if it's fixed! |
@antoine-levitt GPU CI returns an error after merging #964. I think you should first sort, then transfer.
The text was updated successfully, but these errors were encountered: