Skip to content

refactor(model): refactor out common minified mode methods#2883

Merged
canergen merged 26 commits intomainfrom
minified-mode
Nov 29, 2024
Merged

refactor(model): refactor out common minified mode methods#2883
canergen merged 26 commits intomainfrom
minified-mode

Conversation

@martinkim0
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Jul 3, 2024

Codecov Report

Attention: Patch coverage is 92.85714% with 5 lines in your changes missing coverage. Please review.

Project coverage is 84.60%. Comparing base (bcd7e67) to head (906160e).
Report is 72 commits behind head on main.

Files with missing lines Patch % Lines
src/scvi/model/base/_base_model.py 87.50% 3 Missing ⚠️
src/scvi/model/utils/_minification.py 91.66% 1 Missing ⚠️
src/scvi/module/_vae.py 90.90% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2883      +/-   ##
==========================================
+ Coverage   84.53%   84.60%   +0.06%     
==========================================
  Files         178      178              
  Lines       15067    15052      -15     
==========================================
- Hits        12737    12734       -3     
+ Misses       2330     2318      -12     
Files with missing lines Coverage Δ
src/scvi/data/_constants.py 100.00% <100.00%> (ø)
src/scvi/model/_scanvi.py 95.91% <100.00%> (+4.69%) ⬆️
src/scvi/model/_scvi.py 95.83% <100.00%> (+5.42%) ⬆️
src/scvi/model/base/_log_likelihood.py 84.78% <100.00%> (+2.73%) ⬆️
src/scvi/model/base/_vaemixin.py 87.95% <ø> (ø)
src/scvi/module/base/_base_module.py 88.34% <100.00%> (+0.04%) ⬆️
src/scvi/train/_trainingplans.py 94.60% <100.00%> (+0.01%) ⬆️
src/scvi/utils/_decorators.py 100.00% <100.00%> (ø)
src/scvi/model/utils/_minification.py 93.33% <91.66%> (+1.02%) ⬆️
src/scvi/module/_vae.py 94.92% <90.90%> (+0.43%) ⬆️
... and 1 more

@martinkim0 martinkim0 added the cuda tests Run test suite on CUDA label Jul 11, 2024
@canergen
Copy link
Member

@martinkim0 Was this one ready to merge or are things missing?

@canergen canergen added cuda tests Run test suite on CUDA on-merge: backport to 1.2.x and removed cuda tests Run test suite on CUDA labels Jul 30, 2024
@ori-kron-wis ori-kron-wis added this to the scvi-tools 1.2 milestone Nov 6, 2024
@ori-kron-wis ori-kron-wis self-assigned this Nov 6, 2024
@ori-kron-wis ori-kron-wis removed the cuda tests Run test suite on CUDA label Nov 19, 2024
@canergen canergen merged commit 7c96323 into main Nov 29, 2024
meeseeksmachine pushed a commit to meeseeksmachine/scvi-tools that referenced this pull request Nov 29, 2024
canergen pushed a commit that referenced this pull request Nov 29, 2024
…on minified mode methods) (#3060)

Backport PR #2883: refactor(model): refactor out common minified mode
methods

Co-authored-by: Martin Kim <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants