Skip to content

Conversation

@vchuravy
Copy link
Member

@vchuravy vchuravy commented Feb 9, 2022

No description provided.

@vchuravy vchuravy changed the title Excise Cassette from CPU implementation Excise Cassette Feb 9, 2022
@leios
Copy link
Contributor

leios commented Feb 9, 2022

I just tested and this fixes #286,

@vchuravy
Copy link
Member Author

vchuravy commented Feb 9, 2022

Depends on JuliaGPU/AMDGPU.jl#187

@vchuravy
Copy link
Member Author

vchuravy commented Feb 9, 2022

Ok CI seems to be since we need, JuliaGPU/CUDA.jl#1361

@vchuravy vchuravy marked this pull request as ready for review February 9, 2022 22:39
Co-authored-by: Julian Samaroo <jpsamaroo@jpsamaroo.me>
@vchuravy
Copy link
Member Author

So at this point the only failure besides KernelGradient is gamma!

@jpsamaroo
Copy link
Member

jpsamaroo commented Feb 16, 2022

Final failure should be fixed by: JuliaGPU/AMDGPU.jl#196

Co-authored-by: Julian Samaroo <jpsamaroo@jpsamaroo.me>
@jpsamaroo
Copy link
Member

The reflection failures are the same as for CUDAKernels.

@vchuravy vchuravy merged commit f5ceb33 into master Feb 28, 2022
@vchuravy vchuravy deleted the vc/excise_cassette branch February 28, 2022 21:40
@vchuravy vchuravy mentioned this pull request Mar 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants