Simplify, fix tests

Author: mtfishman

ext/KroneckerArraysBlockSparseArraysExt/KroneckerArraysBlockSparseArraysExt.jl

@@ -57,15 +57,15 @@ function block_axes(ax::NTuple{N,AbstractUnitRange{<:Integer}}, I::Block{N}) whe
 end
 
 ## TODO: Is this needed?
-## function Base.getindex(
-##   a::ZeroBlocks{2,KroneckerMatrix{T,A,B}}, I::Vararg{Int,2}
-## ) where {T,A<:AbstractMatrix{T},B<:AbstractMatrix{T}}
-##   ax_a1 = map(arg1, a.parentaxes)
-##   a1 = ZeroBlocks{2,A}(ax_a1)[I...]
-##   ax_a2 = map(arg2, a.parentaxes)
-##   a2 = ZeroBlocks{2,B}(ax_a2)[I...]
-##   return a1 ⊗ a2
-## end
+function Base.getindex(
+  a::ZeroBlocks{2,KroneckerMatrix{T,A,B}}, I::Vararg{Int,2}
+) where {T,A<:AbstractMatrix{T},B<:AbstractMatrix{T}}
+  ax_a1 = map(arg1, a.parentaxes)
+  a1 = ZeroBlocks{2,A}(ax_a1)[I...]
+  ax_a2 = map(arg2, a.parentaxes)
+  a2 = ZeroBlocks{2,B}(ax_a2)[I...]
+  return a1 ⊗ a2
+end
 function Base.getindex(
   a::ZeroBlocks{2,EyeKronecker{T,A,B}}, I::Vararg{Int,2}
 ) where {T,A<:Eye{T},B<:AbstractMatrix{T}}

src/fillarrays/kroneckerarray.jl

@@ -108,31 +108,6 @@ function _permutedims!!(dest::Delta, src::Delta, perm)
   return dest
 end
 
-using DerivableInterfaces: DerivableInterfaces, zero!
-function DerivableInterfaces.zero!(a::EyeKronecker)
-  zero!(a.b)
-  return a
-end
-function DerivableInterfaces.zero!(a::KroneckerEye)
-  zero!(a.a)
-  return a
-end
-function DerivableInterfaces.zero!(a::EyeEye)
-  return throw(ArgumentError("Can't zero out `Eye ⊗ Eye`."))
-end
-
-function DerivableInterfaces.zero!(a::DeltaKronecker)
-  zero!(a.b)
-  return a
-end
-function DerivableInterfaces.zero!(a::KroneckerDelta)
-  zero!(a.a)
-  return a
-end
-function DerivableInterfaces.zero!(a::DeltaDelta)
-  return throw(ArgumentError("Can't zero out `Delta ⊗ Delta`."))
-end
-
 using Base.Broadcast:
   AbstractArrayStyle, AbstractArrayStyle, BroadcastStyle, Broadcasted, broadcasted