Skip to content

Commit 46aaa7b

Browse files
committed
Use SparseArraysBase version of isstored
1 parent f88fa98 commit 46aaa7b

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

Project.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ LabelledNumbers = "0.1.0"
4343
LinearAlgebra = "1.10"
4444
MacroTools = "0.5.13"
4545
MapBroadcast = "0.1.5"
46-
SparseArraysBase = "0.3"
46+
SparseArraysBase = "0.3.2"
4747
SplitApplyCombine = "1.2.3"
4848
TensorAlgebra = "0.1.0, 0.2"
4949
Test = "1.10"

src/abstractblocksparsearray/wrappedabstractblocksparsearray.jl

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ using BlockArrays:
99
blockedrange,
1010
mortar,
1111
unblock
12-
using DerivableInterfaces: DerivableInterfaces, @interface
12+
using DerivableInterfaces: DerivableInterfaces, @interface, DefaultArrayInterface
1313
using GPUArraysCore: @allowscalar
1414
using SplitApplyCombine: groupcount
1515
using TypeParameterAccessors: similartype
@@ -348,12 +348,10 @@ end
348348
# This circumvents issues passing certain kinds of SubArrays
349349
# to the more generic block sparse `isstored` definition,
350350
# for example `blocks(a)` is broken for certain slices.
351-
# TODO: Fix those issues and delete this in favor of using the generic
352-
# version.
353351
function SparseArraysBase.isstored(
354352
a::SubArray{<:Any,N,<:AbstractBlockSparseArray}, I::Vararg{Int,N}
355353
) where {N}
356-
return isstored(parent(a), Base.reindex(parentindices(a), I)...)
354+
return @interface DefaultArrayInterface() isstored(a, I...)
357355
end
358356

359357
function Base.replace_in_print_matrix(

0 commit comments

Comments
 (0)