Closed ivirshup closed 2 months ago
Attention: Patch coverage is 84.84848%
with 10 lines
in your changes missing coverage. Please review.
Project coverage is 91.15%. Comparing base (
f775282
) to head (629af82
). Report is 2 commits behind head on main.
Files | Patch % | Lines |
---|---|---|
...us/src/cellxgene_census/experimental/ml/pytorch.py | 84.84% | 10 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Building on top of #1169
This PR removes the new argument
method
(which was undocumented and had no tests atm) and instead usingreturn_sparse_X
to control conversion. Previouslyreturn_sparse_X
controlled whether densification happened in a wrapping iterator, while now a dense array is created without having to make a CSR intermediate.