LoopVectorization.jl icon indicating copy to clipboard operation
LoopVectorization.jl copied to clipboard

LoopVectorization fails with Tullio and HybridArrays

Open prittjam opened this issue 3 years ago • 2 comments

Please see this issue,

https://github.com/mcabbott/Tullio.jl/issues/155#issuecomment-1237286119

prittjam avatar Sep 05 '22 16:09 prittjam

Minimal reproducer:

julia> using ArrayInterface, HybridArrays

julia> d1 = HybridMatrix{3,StaticArrays.Dynamic()}(rand(3,100));

julia> ArrayInterface.dense_dims(d1)

julia> ArrayInterface.dense_dims(d1.data)
(static(true), static(true))

It should not return nothing.

Either HybridArrays should support ArrayInterface, or you should add an ArrayInterfaceHybridArrays to https://github.com/JuliaArrays/ArrayInterface.jl/tree/master/lib.

chriselrod avatar Sep 05 '22 17:09 chriselrod

Seems dense_dims is missing from https://github.com/JuliaArrays/HybridArrays.jl/blob/master/src/array_interface_compat.jl

chriselrod avatar Sep 05 '22 17:09 chriselrod