TensorBFS / TensorInference.jl

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

docs: add a section on performance evaluation #74

Closed mroavi closed 1 year ago

mroavi commented 1 year ago

Addresses: https://github.com/TensorBFS/TensorInference.jl/issues/68

codecov[bot] commented 1 year ago

Codecov Report

Merging #74 (c1315dc) into main (eb27f4c) will not change coverage. The diff coverage is n/a.

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

@@           Coverage Diff           @@
##             main      #74   +/-   ##
=======================================
  Coverage   84.61%   84.61%           
=======================================
  Files          10       10           
  Lines         533      533           
=======================================
  Hits          451      451           
  Misses         82       82           
GiggleLiu commented 1 year ago

Can we move the benchmark scripts and data files to another repo and only keep the svg files?

mroavi commented 1 year ago

Can we move the benchmark scripts and data files to another repo and only keep the svg files?

Sure. I moved them to https://github.com/TensorBFS/TensorInferenceBenchmarks.jl/tree/main/scripts/performance-evaluation.

I actually went for the same approach of using Julia to generate the pgfplot graph for consistency, and so that we can easily tweak it if needed.

mroavi commented 1 year ago

I had issues with Julia generating the performance evaluation graph. I generated it offline and uploaded the svg to the docs' assets dir.