openclimatefix / openclimatefix/ocf-data-sampler
Accommodate for satellite data storing coord transform reference as a coordinate
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 20
- Forks
- 43
- PR merge metrics
- No merged PRs in 30d
Description
Satellite data uses geostationary coordinates, which means it needs a reference point (satellite position) to transform to lat lon correctly. We currently store this as an "area" attribute, which leads to some problems (for now only in IODC service, read more in this satip issue).
We will be moving to a more robust way of storing this, as a time-dependent coordinate (see this issue in sat consumer). It will be a bit before this is implemented, but we will need to accommodate for it in code here in ocf_data_sampler/select/geospatial.py. We will keep the attribute method as well for backwards compatibility.
This issue is on hold until the sister issue in sat consumer is completed.
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.
Research direction
Start by reading ocf_data_sampler/select/geospatial.py around lines 123-133, then review the linked satellite-consumer issue to understand the time-dependent coordinate representation. The work is done when geostationary data can use that coordinate reference while retaining support for the existing area attribute; the issue is currently on hold until the sister issue is completed.
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
- 35/100