search

TensorBFS / TensorInference.jl

Probabilistic inference using contraction of tensor networks
https://tensorbfs.github.io/TensorInference.jl/
MIT License
18 stars 2 forks source link

remove tropicalgemm as a dependency #48

Closed GiggleLiu closed 1 year ago

GiggleLiu commented 1 year ago

@mroavi Let us try to remove TropicalGEMM in the dependency, and see if it resolves #43

By doing so, we need to add using TropicalGEMM in the benchmark. The bug may be related to precompiling tools. I will try to remove the precompiling later.

codecov[bot] commented 1 year ago

Codecov Report

Merging #48 (bad349e) into main (5da73eb) will increase coverage by 18.45%. The diff coverage is 83.33%.

:exclamation: Current head bad349e differs from pull request most recent head cacfee2. Consider uploading reports for the commit cacfee2 to get more accurate results

@@             Coverage Diff             @@
##             main      #48       +/-   ##
===========================================
+ Coverage   61.67%   80.12%   +18.45%     
===========================================
  Files           9        9               
  Lines         467      478       +11     
===========================================
+ Hits          288      383       +95     
+ Misses        179       95       -84
Impacted Files Coverage Δ
src/TensorInference.jl 75.00% <ø> (ø)
src/sampling.jl 98.75% <75.00%> (+98.75%) :arrow_up:
src/Core.jl 53.52% <83.33%> (+15.05%) :arrow_up:
src/mmap.jl 71.18% <100.00%> (ø)
src/utils.jl 86.59% <100.00%> (+3.09%) :arrow_up: