TensorBFS / TropicalGEMM.jl

The fastest tropical matrix multiplication in the world!
MIT License
29 stars 2 forks source link

The CI/CD fails #20

Closed GiggleLiu closed 1 year ago

GiggleLiu commented 1 year ago
Test Summary: | Pass  Total     Time
inference     |   40     40  8m16.6s
┌ Info: Time complexity (number of element-wise multiplications) = 2^24.843441555097844
│ Space complexity (number of elements in the largest intermediate tensor) = 2^20.0
└ Read-write complexity (number of element-wise read and write) = 2^22.12346437841676
map: Error During Test at /home/runner/work/TensorInference.jl/TensorInference.jl/test/maxprob.jl:5
  Got exception outside of a @test
  MethodError: no method matching vifelse(::typeof(VectorizationBase.vfmadd_fast), ::VectorizationBase.EVLMask{4, UInt8}, ::TropicalNumbers.Tropical{VectorizationBase.Vec{4, Float64}}, ::TropicalNumbers.Tropical{VectorizationBase.VecUnroll{1, 4, Float64, VectorizationBase.Vec{4, Float64}}}, ::TropicalNumbers.Tropical{VectorizationBase.VecUnroll{1, 4, Float64, VectorizationBase.Vec{4, Float64}}})

  Closest candidates are:
    vifelse(::typeof(VectorizationBase.vfmadd231), ::Any, ::Any, ::Any, ::Any, ::Static.False)
     @ VectorizationBase ~/.julia/packages/VectorizationBase/0dXyA/src/llvm_intrin/intrin_funcs.jl:1023
    vifelse(::typeof(VectorizationBase.vfnmsub231), ::Any, ::Any, ::Any, ::Any, ::Static.False)
     @ VectorizationBase ~/.julia/packages/VectorizationBase/0dXyA/src/llvm_intrin/intrin_funcs.jl:1029
    vifelse(::typeof(VectorizationBase.vfmsub231), ::Any, ::Any, ::Any, ::Any, ::Static.False)
     @ VectorizationBase ~/.julia/packages/VectorizationBase/0dXyA/src/llvm_intrin/intrin_funcs.jl:1027
    ...

  Stacktrace:
     [1] macro expansion
       @ LoopVectorization ~/.julia/packages/LoopVectorization/91gMc/src/codegen/lower_compute.jl:249 [inlined]
     [2] ifelsepartial(f::F, m::VectorizationBase.AbstractMask{W}, ::Static.StaticInt{M}, ::Static.StaticInt{S}, vargs::Vararg{Any, K}) where {F, W, K, M, S}
       @ LoopVectorization ~/.julia/packages/LoopVectorization/91gMc/src/codegen/lower_compute.jl:249 [inlined]
     [3] macro expansion
       @ LoopVectorization ~/.julia/packages/LoopVectorization/91gMc/src/reconstruct_loopset.jl:1107 [inlined]
     [4] _turbo_!(::Val{(false, 0, 0, 0, false, 4, 32, 15, 64, 0x0000000000000001, 1, true)}, 
GiggleLiu commented 1 year ago

Hi @chriselrod , do you have any clue what is this error about?

chriselrod commented 1 year ago

Apparently it's a missing method. Defining it should fix the problem.

GiggleLiu commented 1 year ago

Thanks for the prompt reply. I made a PR in #21 , it would be great if you could take a look and confirm it is a reasonable fix.