Fix backwards compatibility for TOTALVI #1502
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report
@@ Coverage Diff @@
## master #1502 +/- ##
=======================================
Coverage 91.06% 91.07%
=======================================
Files 112 112
Lines 8809 8813 +4
=======================================
+ Hits 8022 8026 +4
Misses 787 787
Continue to review full report at Codecov.
|
adamgayoso
requested changes
Apr 8, 2022
justjhong
force-pushed
the
jhong/fixtotalvicompat
branch
from
2b32a49
to
0bb7d8b
Compare
adamgayoso
approved these changes
Apr 8, 2022
nrclaudio
pushed a commit
to nrclaudio/scvi-tools-tune
that referenced
this pull request
Jun 21, 2022
* run checks * fix backwards compatibility for totalvi * fix release note Co-authored-by: Adam Gayoso <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Since TOTALVI.setup_anndata was modified to remove the unnecessary labels key, issues arose when a legacy totalvi model was loaded then saved, then loaded again.