chanzuckerberg / cellxgene-census

CZ CELLxGENE Discover Census
https://chanzuckerberg.github.io/cellxgene-census/
MIT License
72 stars 18 forks source link

[python] Support custom obs encoders #1191

Open ebezzi opened 3 weeks ago

ebezzi commented 3 weeks ago

Add support for custom obs encoders. Example use case: when multiple obs columns need to be batched together before the tensor creation.

codecov[bot] commented 6 days ago

Codecov Report

Attention: Patch coverage is 95.10490% with 7 lines in your changes missing coverage. Please review.

Project coverage is 91.30%. Comparing base (f775282) to head (a0e424e). Report is 1 commits behind head on main.

Files Patch % Lines
...s/src/cellxgene_census/experimental/ml/encoders.py 89.70% 7 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1191 +/- ## ========================================== + Coverage 91.19% 91.30% +0.11% ========================================== Files 77 80 +3 Lines 5971 6256 +285 ========================================== + Hits 5445 5712 +267 - Misses 526 544 +18 ``` | [Flag](https://app.codecov.io/gh/chanzuckerberg/cellxgene-census/pull/1191/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=chanzuckerberg) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/chanzuckerberg/cellxgene-census/pull/1191/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=chanzuckerberg) | `91.30% <95.10%> (+0.11%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=chanzuckerberg#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.