-
Notifications
You must be signed in to change notification settings - Fork 76
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 support #2
Comments
Looks neat! Thanks @tclements Actually I was looking for some packages that put numpy on GPU. I remember you mentioned JAX from google, but it seems that library is still in trail and I was not able to install it on RC. The CuPy helps us to keep the same formate of sub functions for GPU and CPU versions. I am going to test the performance of CuPy shortly. |
Awesome. Looking forward to see the compute time and memory usage. I also notice that the main code syntax is the same but that the noise_module is different. You could just use this in the header of the code: |
For this part, we still need to profile the code to see where things are slow, and use GPU if it speeds up some of the calculations. |
I do not think that we ended up implementing this into a cuda module. I am reviving this issue as an enhancement feature. |
I was developing GPU-version but paused somewhere. I guess I already
finished 50%. Let me know if someone feels interested. We can get it done
together. -congcong
…On Tue, Mar 7, 2023 at 12:35 AM Marine Denolle ***@***.***> wrote:
Assigned #2 <#2> to @congcy
<https://github.com/congcy>.
—
Reply to this email directly, view it on GitHub
<#2 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AFLECJJUFRQJE46A4X3W32LW23CK3ANCNFSM4GSGKSBA>
.
You are receiving this because you were assigned.Message ID:
***@***.***>
|
Hey, I saw @ChengXin was implementing GPU support. Looks fast! We might want to consider using CuPy - it's a Numpy compatible library for CUDA. One nice thing about CuPy is we can use the same code on CPU/GPU. Here's the correlate functions with CuPy:
The text was updated successfully, but these errors were encountered: