codecov[bot]
commented
4 months ago Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 82.34%. Comparing base (
c6cf312) to head (deb2dbf).
Additional details and impacted files
```diff @@ Coverage Diff @@ ## main #1070 +/- ## ======================================= Coverage 82.34% 82.34% ======================================= Files 74 74 Lines 5714 5714 ======================================= Hits 4705 4705 Misses 1009 1009 ``` | [Flag](https://app.codecov.io/gh/chanzuckerberg/cellxgene-census/pull/1070/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/1070/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=chanzuckerberg) | `82.34% <ø> (ø)` | | 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.
Notebooks for the new embedding API usage.
Notebook 1 preview Notebook 2 preview
Notes:
Loading embeddings into a Scipy matrices and NumPy arrayspart from Notebook 1, and I replaced it withLoad an embedding into a dense NumPy array. Using sparse matrices isn't straightforward with the new API, since we don't expose direct access to the underlying SparseNDArray, and I don't think the example was particularly useful in the first place, since all the embeddings we have are essentially dense.