scvi-tools icon indicating copy to clipboard operation
scvi-tools copied to clipboard

Likelihood params bugfix

Open tommycelsius opened this issue 1 year ago • 1 comments

Bug: When using gene_likelihood!="zinb", the function get_likelihood_parameters() would fail.

To reproduce:

model = SCVI(adata, gene_likelihood="nb")
model.get_likelihood_parameters()

Fix: Added if self.module.gene_likelihood == "zinb": to a few lines.

tommycelsius avatar Jul 28 '22 22:07 tommycelsius

Codecov Report

Base: 91.39% // Head: 91.40% // Increases project coverage by +0.01% :tada:

Coverage data is based on head (93d7009) compared to base (f017b50). Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1618      +/-   ##
==========================================
+ Coverage   91.39%   91.40%   +0.01%     
==========================================
  Files         116      116              
  Lines        9190     9193       +3     
==========================================
+ Hits         8399     8403       +4     
+ Misses        791      790       -1     
Impacted Files Coverage Δ
scvi/model/base/_rnamixin.py 93.36% <100.00%> (+0.62%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov[bot] avatar Jul 28 '22 22:07 codecov[bot]