-
-
Notifications
You must be signed in to change notification settings - Fork 611
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
move to CUDA.jl #1258
move to CUDA.jl #1258
Conversation
Co-authored-by: Carlo Lucibello <carlo.lucibello@gmail.com>
Co-authored-by: Carlo Lucibello <carlo.lucibello@gmail.com>
bors r+ |
Build failed: |
I would recommend pushing to that branch directly rather than have multiple prs do the same thing. I can give you commit bit on that |
bors r+ |
yeah, I wasn't sure how to do that since that branch is on your personal fork. This is on the original repo, so you should already be able to push to it |
@maleadt I observe julia> using Flux
┌ Warning: `haskey(::TargetIterator, name::String)` is deprecated, use `Target(; name = name) !== nothing` instead.
│ caller = llvm_compat(::VersionNumber) at compatibility.jl:181
└ @ CUDA ~/.julia/packages/CUDA/42B9G/deps/compatibility.jl:181 here. If |
Build failed: |
bors r+ |
Why remove the doctest? |
Build succeeded: |
1261: reinstate curnn test r=CarloLucibello a=CarloLucibello this is passing locally on my GPU, but in #1258 bors didn't seem to like it. Let's try again EDIT: actually this is not passing test locally, but let's see what BORS says in any case cfr #1262 Co-authored-by: CarloLucibello <carlo.lucibello@gmail.com>
1261: reinstate curnn test r=DhairyaLGandhi a=CarloLucibello this is passing locally on my GPU, but in #1258 bors didn't seem to like it. Let's try again EDIT: actually this is not passing test locally, but let's see what BORS says in any case cfr #1262 Co-authored-by: CarloLucibello <carlo.lucibello@gmail.com>
continuing #1204