developmentseed / developmentseed/zarrista
Explore merging `FixedLengthTensor` and `OptionalFixedLengthTensor` into a single class
- Dominant language
- Python
- Stars
- 45
- Forks
- 2
- Avg merge
- 2d 4h
- Merged PRs (30d)
- 7
Description
After #172 is merged, I would like to explore merging `FixedLengthTensor` and `OptionalFixedLengthTensor` into a single class.
But that could be a bit messy because the `__array__` would sometimes convert to a plain array and other times convert to a masked array.
Contributor guide
No contributing guide indexed for this repository
Research direction
Review the implementations of FixedLengthTensor and OptionalFixedLengthTensor after #172, focusing on their __array__ behavior. Determine whether one class can consistently support conversion to both plain and masked arrays without creating ambiguous behavior; done means the merge decision and its expected behavior are documented clearly.
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