Deprecate usage of SolarPosition(...).
Open
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 402
- Forks
- 134
- PR merge metrics
- No merged PRs in 30d
Description
Remove usage of SolarPosition inside the atlite code and favour direct usage of cutout.data variables.
By this version everyone who has not migrated yet to cutouts with integrate solar positions will be forced to upgrade or stay with an earlier version.
See #199 .
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 searching the atlite codebase for SolarPosition usage and compare those call sites with the cutout.data variables described in the issue. Review issue #199 for the migration context. Done means SolarPosition is no longer used inside atlite and the affected functionality continues to use the direct cutout.data values.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- data
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100