spatial_neighbour_x should have serializable internals to avoid uns formats
Open
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 598
- Forks
- 121
- Avg merge
- 3d 11h
- Merged PRs (30d)
- 3
Description
These classes would be also ideally serializable so we dont need to rely on a forman under adata.uns which we are scared to change. I will make this as an issue to tackle when the time comes
Originally posted by @selmanozleyen in https://github.com/scverse/squidpy/issues/1147#issuecomment-4430504368
Contributor guide
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.
Research direction
Start by locating the spatial_neighbour_x classes and reviewing how their data is currently stored in adata.uns, including the context from issue 1147. Clarify which internals must be serializable and what format should replace the current adata.uns dependency; done means the classes can be serialized without relying on that format.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- data
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Quiet
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100