earthdaily / earthdaily/earthdaily-python-client
Mask to support intersects=None, bbox=None
- Dominant language
- Python
- Stars
- 31
- Forks
- 10
- PR merge metrics
- No merged PRs in 30d
Description
The default args are `intersects=None, bbox=None` yet one of either [must be passed](https://github.com/earthdaily/earthdaily-python-client/blob/e9f98be38ac0d1f019fdc37921ef18825f42b053/earthdaily/earthdatastore/mask/__init__.py#L159). However in the case you want to use the `scl` (or another cloud mask) over the entire image, it should be possible to not pass these args and just default to the whole image
Contributor guide
Research direction
Read earthdaily/earthdatastore/mask/__init__.py around line 159, where the requirement for intersects or bbox is enforced. Check how the scl or other cloud mask is applied when no spatial arguments are supplied; done means allowing both defaults to select the whole image without breaking existing intersects or bbox usage.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- data
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100