scverse / scverse/spatialdata-io
In `.geojson` parsed, the index should be parsed from `.geojson`
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 103
- Forks
- 65
- Avg merge
- 1h 8m
- Merged PRs (30d)
- 3
Description
The .geojson parsed (currently in spatialdata.models.ShapesModels.parse(), which will be moved to spatialdata-io, doesn't try to parse the index of the object from the .geojson file.
This behavior should be implemented. For instance in the incoming seqFISH reader #227 the index is assumed to match the index from the table (which is reasonable and expected), but more robust parser would read it from the .geojson.
CC @LLehner @quentinblampey
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 reading spatialdata.models.ShapesModels.parse(), which currently parses the .geojson and is planned to move to spatialdata-io. Compare its behavior with the incoming seqFISH reader in issue #227, where the index is taken from the table. Done means the parser obtains each object's index from the .geojson rather than assuming it matches the table index.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- data
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100