-
Notifications
You must be signed in to change notification settings - Fork 12
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
Fix CI #82
Conversation
There was only a performance regression, IIRC everything still compiled correctly for 1.6. |
Yeah it's the CUDA.jl update that break things, I figured you had updated that dependency too. |
Pushed a fix for diagonal matrices on 1.6. I'm not that experienced with I'll have a closer look at the dynamic function invocation issues that occur on 1.7 later. |
Actually it's just the upgrade to Julia 1.7, as CI reveals here.
It should, yes.
It's something to do with indexing into the the result of
|
Fixed the dynamic invocation, but now the code allocates on 1.7 (dynamic memory, which is only meant to be used in error paths). I fear this may be an instance of JuliaLang/julia#41800. |
Codecov Report
@@ Coverage Diff @@
## master #82 +/- ##
==========================================
+ Coverage 41.06% 42.31% +1.25%
==========================================
Files 9 9
Lines 414 423 +9
==========================================
+ Hits 170 179 +9
Misses 244 244
Continue to review full report at Codecov.
|
in a previous commit (merge JuliaGPU#82), dynamic allocation of shared memory was changed from the deprecated `@cuDynamicSharedMem` macro to the replacing `CuDynamicSharedArray` function, this breaks compatability with CUDA.jl versions before v3.5.0 as the `CuDynamicSharedArray` function was only added in CUDA.jl v3.5.
in a previous commit (merge JuliaGPU#82), dynamic allocation of shared memory was changed from the deprecated `@cuDynamicSharedMem` macro to the replacing `CuDynamicSharedArray` function, this breaks compatability with CUDA.jl versions before v3.5.0 as the `CuDynamicSharedArray` function was only added in CUDA.jl v3.5.
in a previous commit (merge JuliaGPU#82), dynamic allocation of shared memory was changed from the deprecated `@cuDynamicSharedMem` macro to the replacing `CuDynamicSharedArray` function, this breaks compatability with CUDA.jl versions before v3.5.0 as the `CuDynamicSharedArray` function was only added in CUDA.jl v3.5.
Update CUDA.jl compat bounds and adapt to some changes.
@thomasfaingnaert Kernels don't compile, but you were already looking into a regression so I take it you have a fix already?