-
Notifications
You must be signed in to change notification settings - Fork 4
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
Remove the gpu() functions if not needed #1
Comments
You still need the call to Is it helpful for this to be a package? The main difference seems to be using |
Sorry - I just meant those |
The other benefits of a package could probably be things like CI and an easy way to install, but not the most pressing point for now. |
Ah, got it – yeah these notebooks definitely need a refresh then. I don't know if there's a canonical way to do CI for notebooks, but we could probably get something together with nbinclude or similar. |
Is there any chance to find out in what versions were the Flux GitHub stuff written? Because most of it doesnt work for me even when i read readme files and just copy paste it... |
Maybe |
IIRC, the
gpu()
functions are not needed any more and to run on GPUs, one only needsCuArrays
. If so, can the notebooks be updated accordingly?Also, should we register as a Julia package?
The text was updated successfully, but these errors were encountered: