Fix CUDSS extension for v0.5 and v0.6 compatibility #471
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR fixes the
ArrayInterfaceCUDSSExtto properly support CUDSS v0.5 and v0.6 by updating the extension code to use the modern CUDSS API.Problem
PR #469 bumped the CUDSS compatibility to include v0.5 and v0.6, but the extension code itself was not updated to match the breaking API changes introduced in these versions. The extension was using
CudssMatrix, which was removed in favor of standard CUDA.jl types (CuVector) in CUDSS v0.5+.Breaking Changes in CUDSS v0.5/v0.6
According to the CUDSS v0.6.0 release notes:
CudssMatrixtype is no longer availableThe current CUDSS examples show the modern API pattern:
Changes
Updated extension code (
ext/ArrayInterfaceCUDSSExt.jl):CudssMatrix(T, n)withCUDA.CuVector{T}(undef, n)using CUDAto access the CuVector typeUpdated extension dependencies (
Project.toml):ArrayInterfaceCUDSSExt = "CUDSS"toArrayInterfaceCUDSSExt = ["CUDSS", "CUDA"]Updated compatibility bounds (
Project.toml):Testing
The changes follow the exact pattern used in the official CUDSS examples. The extension now uses the same API as CUDSS itself.
Related
🤖 Generated with Claude Code