-
Notifications
You must be signed in to change notification settings - Fork 346
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SCANVI bug fixes #1458
SCANVI bug fixes #1458
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1458 +/- ##
==========================================
+ Coverage 90.97% 91.04% +0.06%
==========================================
Files 111 111
Lines 8743 8762 +19
==========================================
+ Hits 7954 7977 +23
+ Misses 789 785 -4
Continue to review full report at Codecov.
|
"Unknown", | ||
batch_key="batch", | ||
categorical_covariate_keys=["cat1", "cat2"], | ||
continuous_covariate_keys=["cont1", "cont2"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
arches won't actually work as intended with extra categorical covariates. We should not test extra categorical covariates and instead add an error message in load_query_data if they are detected. Extra continuous are fine.
if ( | ||
allow_missing_labels | ||
and self.attr_key is not None | ||
and self.attr_key not in adata_target.obs | ||
): | ||
# Fill in original .obs attribute with unlabeled_category values. | ||
warnings.warn( | ||
f"Missing labels key {self.attr_key}. Filling in with unlabeled category {self._unlabeled_category}." | ||
) | ||
_set_data_in_registry( | ||
adata_target, | ||
self._unlabeled_category, | ||
self.attr_name, | ||
self._original_attr_key, | ||
) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice
Co-authored-by: Justin Hong <[email protected]>
* address issue 1449 * fix scanvi covariate handling * fix tests * raise error on extra cat covs * release note
Fixes #1449
Fixes #1450