Skip to content

WIP: Update DRVI and align with the latest scvi-tools version#33

Merged
moinfar merged 8 commits into
mainfrom
update-scvi-version
Nov 16, 2025
Merged

WIP: Update DRVI and align with the latest scvi-tools version#33
moinfar merged 8 commits into
mainfrom
update-scvi-version

Conversation

@moinfar

@moinfar moinfar commented May 22, 2025

Copy link
Copy Markdown
Collaborator

Steps:

  • Relax all dependencies
  • Align with the latest scvi-tools
  • Automatic tests passed
  • Check backward compatibility
  • Test and make sure metrics match
  • Test to see if additional features (new in scvi-tools) work with no problem.
  • Put limits on dependencies with respect to the newest packages.

Goal: solve #22

@codecov

codecov Bot commented Jul 10, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 59.59596% with 40 lines in your changes missing coverage. Please review.
✅ Project coverage is 68.95%. Comparing base (90a5c73) to head (533aeb7).
⚠️ Report is 28 commits behind head on main.

Files with missing lines Patch % Lines
src/drvi/scvi_tools_based/module/_constants.py 32.35% 23 Missing ⚠️
...c/drvi/scvi_tools_based/model/base/_archesmixin.py 72.00% 7 Missing ⚠️
src/drvi/scvi_tools_based/module/_drvi.py 80.00% 4 Missing ⚠️
...i/scvi_tools_based/model/base/_generative_mixin.py 50.00% 3 Missing ⚠️
src/drvi/scvi_tools_based/model/_drvi.py 80.00% 2 Missing ⚠️
src/drvi/nn_modules/prior.py 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #33      +/-   ##
==========================================
- Coverage   69.57%   68.95%   -0.62%     
==========================================
  Files          49       50       +1     
  Lines        2524     2580      +56     
==========================================
+ Hits         1756     1779      +23     
- Misses        768      801      +33     
Files with missing lines Coverage Δ
src/drvi/scvi_tools_based/nn/_base_components.py 84.06% <100.00%> (-0.05%) ⬇️
...i/utils/tools/interpretability/_latent_traverse.py 93.75% <100.00%> (ø)
src/drvi/nn_modules/prior.py 89.13% <50.00%> (+0.11%) ⬆️
src/drvi/scvi_tools_based/model/_drvi.py 69.62% <80.00%> (-6.44%) ⬇️
...i/scvi_tools_based/model/base/_generative_mixin.py 76.28% <50.00%> (+0.24%) ⬆️
src/drvi/scvi_tools_based/module/_drvi.py 80.87% <80.00%> (+0.10%) ⬆️
...c/drvi/scvi_tools_based/model/base/_archesmixin.py 72.10% <72.00%> (+0.25%) ⬆️
src/drvi/scvi_tools_based/module/_constants.py 32.35% <32.35%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@moinfar moinfar changed the title Update DRVI and align with the latest scvi-tools version WIP: Update DRVI and align with the latest scvi-tools version Nov 14, 2025
@moinfar moinfar merged commit 5794bcb into main Nov 16, 2025
8 of 9 checks passed
@moinfar moinfar deleted the update-scvi-version branch November 16, 2025 17:00
@moinfar moinfar restored the update-scvi-version branch November 16, 2025 17:01
moinfar added a commit that referenced this pull request Nov 16, 2025
moinfar added a commit that referenced this pull request Nov 16, 2025
@moinfar moinfar deleted the update-scvi-version branch November 16, 2025 17:11
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.

1 participant