chanzuckerberg / cellxgene-census

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

Identify tiledbsoma version that causes builder unit tests to fail #1167

Closed prathapsridharan closed 1 month ago

prathapsridharan commented 1 month ago

This PR shows the tiledbsoma version that causes census builder unit tests to fail

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.12%. Comparing base (d771466) to head (edec044). Report is 2 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1167 +/- ## ======================================= Coverage 91.11% 91.12% ======================================= Files 77 77 Lines 5886 5902 +16 ======================================= + Hits 5363 5378 +15 - Misses 523 524 +1 ``` | [Flag](https://app.codecov.io/gh/chanzuckerberg/cellxgene-census/pull/1167/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/1167/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=chanzuckerberg) | `91.12% <ø> (+<0.01%)` | :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.

prathapsridharan commented 1 month ago

A fix for this has been identified by tiledbsoma maintainers. The issue is fixed by tiledbsoma at commit 3869abc and verified by passing builder unit tests as can be seen in the CI. The issue is tracked here:

https://github.com/single-cell-data/TileDB-SOMA/issues/2663