Skip to content

fix: ensure read_10x_mtx returns CSR format #4017

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
ernestprovo23 wants to merge 2 commits into
base: main
Choose a base branch
from
+61 −5

fix: read COO→CSC directly to avoid double conversion

d3f4030
Select commit
Loading
Failed to load commit list.
Open

fix: ensure read_10x_mtx returns CSR format #4017

fix: read COO→CSC directly to avoid double conversion
d3f4030
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Mar 30, 2026 in 1s

78.27% (target 75.00%)

View this Pull Request on Codecov

78.27% (target 75.00%)

Details

Codecov Report

❌ Patch coverage is 57.69231% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.27%. Comparing base (2ddfcc7) to head (d3f4030).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/scanpy/readwrite.py 57.69% 11 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #4017      +/-   ##
==========================================
- Coverage   78.34%   78.27%   -0.07%     
==========================================
  Files         117      117              
  Lines       12782    12807      +25     
==========================================
+ Hits        10014    10025      +11     
- Misses       2768     2782      +14
Files with missing lines Coverage Δ
src/scanpy/readwrite.py 76.69% <57.69%> (-1.97%) ⬇️
View more details on Codecov