Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

vmap False by default #3146

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

vmap False by default #3146

wants to merge 4 commits into from

Conversation

PierreBoyeau
Copy link
Contributor

Fixes #3145

@canergen canergen added on-merge: backport to 1.2.x on-merge: backport to 1.2.x draft labels Jan 14, 2025
Copy link

codecov bot commented Jan 14, 2025

Codecov Report

Attention: Patch coverage is 0% with 15 lines in your changes missing coverage. Please review.

Project coverage is 82.72%. Comparing base (820c987) to head (d685dbb).
Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
src/scvi/external/mrvi/_model.py 0.00% 15 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #3146       +/-   ##
===========================================
+ Coverage   53.30%   82.72%   +29.41%     
===========================================
  Files         181      181               
  Lines       15433    15446       +13     
===========================================
+ Hits         8227    12777     +4550     
+ Misses       7206     2669     -4537     
Files with missing lines Coverage Δ
src/scvi/external/mrvi/_model.py 7.66% <0.00%> (-82.52%) ⬇️

... and 124 files with indirect coverage changes

@canergen
Copy link
Member

canergen commented Feb 9, 2025

@PierreBoyeau please provide an update about the use case with 1k samples?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
draft on-merge: backport to 1.2.x on-merge: backport to 1.2.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adjust MrVI running hyperparameters for 1000s of samples
3 participants