chanzuckerberg / cellxgene-census

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

[builder] ongoing refactoring for Dask #1040

Closed bkmartinjr closed 6 months ago

bkmartinjr commented 6 months ago

Third in a series of builder refactoring. Primary changes:

codecov[bot] commented 6 months ago

Codecov Report

Attention: Patch coverage is 49.01961% with 208 lines in your changes are missing coverage. Please review.

Project coverage is 81.31%. Comparing base (311a352) to head (cf252db). Report is 2 commits behind head on main.

Files Patch % Lines
...llxgene_census_builder/build_soma/validate_soma.py 26.81% 191 Missing :warning:
.../cellxgene_census_builder/build_soma/build_soma.py 84.37% 5 Missing :warning:
...cellxgene_census_builder/build_soma/consolidate.py 50.00% 4 Missing :warning:
...ne_census_builder/build_soma/experiment_builder.py 82.60% 4 Missing :warning:
...rc/cellxgene_census_builder/build_soma/__main__.py 66.66% 1 Missing :warning:
...lder/src/cellxgene_census_builder/build_soma/mp.py 93.75% 1 Missing :warning:
...us_builder/src/cellxgene_census_builder/logging.py 96.15% 1 Missing :warning:
...ilder/src/cellxgene_census_builder/process_init.py 95.23% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1040 +/- ## ========================================== + Coverage 76.94% 81.31% +4.37% ========================================== Files 72 73 +1 Lines 5691 5551 -140 ========================================== + Hits 4379 4514 +135 + Misses 1312 1037 -275 ``` | [Flag](https://app.codecov.io/gh/chanzuckerberg/cellxgene-census/pull/1040/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/1040/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=chanzuckerberg) | `81.31% <49.01%> (+4.37%)` | :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.