-
Notifications
You must be signed in to change notification settings - Fork 70
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
CUDA: add support for onehot #2203
Comments
Also, in forward, I get the error |
It does explicitly support KA. Can you post a MWE without DifferentiationInterface? |
Yes, here it is
Same errors (takes a lot more times to throw for some reason). I don't remember how to make a jacobian from autodiff which is why I stayed on high level jacobian. |
full error : forward
reverse
|
works like a charm on cpu btw
|
the reverse error I tjhink is a 1.11 issue, can you try 1.10? For forward mode, it looks like we just ought add a onehot implementation for a CuArray |
not working in 1.10 but different error (for reverse)
|
Try with this? #2204 |
new one hehe
|
pushed again, retry? |
I still get the same error for now, maybe I need to gc some stuff wait a sec |
ok so back to scalar indexing even in reverse
|
ok awesome, yeah these are resolvable as well, we just should override the equivalent of onehot |
resolved by #2220 |
Is Enzyme supposed to differentiate through KA kernels ? I wanted to try it on this simple exemple and I get an error
GPU compilation of MethodInstance for (::GPUArrays.var"#34#36")(::CUDA.CuKernelContext, ::CuDeviceVector{…}, ::Base.Broadcast.Broadcasted{…}, ::Int64) failed KernelError: passing and using non-bitstype argument
The text was updated successfully, but these errors were encountered: