the description of read_mtx is not clear
Open
@beyondpie is already working on this.
Since Dec 14, 2023.
- Dominant language
- Python
- Stars
- 323
- Forks
- 43
- PR merge metrics
- No merged PRs in 30d
Description
Hi Kai,
I found that the description of read_mtx is not clear.
- obs_names: File that stores the observation names.
- obs_name_file would be better, otherwise, it's hard to know it's a file.
- it should be row / column of the read_mtx? From my usage, I think it should be row names. But this is usually not the order of 10x_mtx (see https://scanpy.readthedocs.io/en/stable/generated/scanpy.read_10x_mtx.html). And I think people use read_mtx usually because people wants to read 10x_mtx.
- same for var_names.
Maybe we can provide another function like scanpy to directly load mtx from a directory?
Sincerely,
Songpeng
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.