search

meyer-lab / tensorpack

A collection of tensor methods from the Meyer lab.
MIT License
2 stars 2 forks source link

Change CMTF to SVD imputation with a set starting seed #32

Closed aarmey closed 2 years ago

aarmey commented 2 years ago

This should make the factorization process deterministic.

codecov[bot] commented 2 years ago

Codecov Report

Merging #32 (c756db6) into main (f2563e7) will increase coverage by 1.19%. The diff coverage is 97.82%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
+ Coverage   80.03%   81.22%   +1.19%     
==========================================
  Files           9       10       +1     
  Lines         626      671      +45     
==========================================
+ Hits          501      545      +44     
- Misses        125      126       +1
Flag Coverage Δ
unittests 81.22% <97.82%> (+1.19%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
tensorpack/SVD_impute.py 97.72% <97.72%> (ø)
tensorpack/cmtf.py 97.00% <100.00%> (+0.01%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f2563e7...c756db6. Read the comment docs.