Closed ivirshup closed 1 month ago
Attention: Patch coverage is 97.67442%
with 1 lines
in your changes are missing coverage. Please review.
Project coverage is 91.16%. Comparing base (
e36addb
) to head (f2db37f
).
Files | Patch % | Lines |
---|---|---|
...lxgene_census/src/cellxgene_census/_get_anndata.py | 93.75% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
get_obs
and get_var
to me seems consistent when put together with anndata
related functions. And the same applies to helper functions needed in service of get_anndata
, get_obs
, get_var
.
I think we can shift things around later if needed but for now this organization seems fine to me
Thanks for the catch! Applied your changes.
Fixes https://github.com/chanzuckerberg/cellxgene-census/issues/1037
TODO:
get_var