TensorBFS / TensorInference.jl

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

Return `Samples` type instead of a matrix #60

Closed GiggleLiu closed 1 year ago

GiggleLiu commented 1 year ago

Returning samples as a matrix is not intuitive, so I returned the a Samples instance instead. Since this is an API change, I bumped the middle version number.

I will merge and tagged it directly if the tests pass, since I need to deliver a lecture on Tursday.

codecov[bot] commented 1 year ago

Codecov Report

Merging #60 (439af41) into main (499316d) will decrease coverage by 0.05%. The diff coverage is 82.35%.

@@            Coverage Diff             @@
##             main      #60      +/-   ##
==========================================
- Coverage   81.66%   81.61%   -0.05%     
==========================================
  Files          10       10              
  Lines         518      533      +15     
==========================================
+ Hits          423      435      +12     
- Misses         95       98       +3     
Files Changed Coverage Δ
src/TensorInference.jl 100.00% <ø> (ø)
src/generictensornetworks.jl 90.90% <66.66%> (-9.10%) :arrow_down:
src/sampling.jl 97.59% <80.00%> (-1.16%) :arrow_down:
src/Core.jl 51.66% <100.00%> (+5.37%) :arrow_up:
GiggleLiu commented 1 year ago

I need to get it merged for tomorrows' tutorial.