Refactor duplicated OrderedDict/DataFrame return logic in clearsky models
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 50/100
Direzione di ricerca
Start in clearsky.py at the TODO inside bird(), then read ineichen and simplified_solis to compare the repeated OrderedDict-or-DataFrame return code. Design a small decorator for that pattern and apply it to the three functions. Done means the boilerplate is gone, the return types for both input kinds are unchanged, and the existing clearsky tests still pass. The design is open, so agree on the decorator's shape with maintainers first.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
While reading clearsky.py, I noticed this TODO comment in bird():
TODO: be DRY, use decorator to wrap methods that need to return either
OrderedDict or DataFrame instead of repeating this boilerplate code
A few clearsky model functions (ineichen, simplified_solis, bird) repeat the same pattern of returning either an OrderedDict or a DataFrame depending on the input type.
It might be cleaner to introduce a small decorator to handle this logic and remove the duplicated boilerplate.
Happy to work on this if it sounds good.
- Lingua principale
- Python
- Stelle
- 1.7k
- Fork
- 1.2k
- Merge medio
- 3g 18h
- PR unite (30g)
- 13
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di pvlib/pvlib-python
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
pvlib/pvlib-python#2903 · 2 commenti · 2 reazioni ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
pvlib/pvlib-python#2902 ·
I maintainer di solito rispondono entro 1 giorno
-
Add an example or documentation on using 'bhi' from ERA5Forse di nuovo libera @anshurajbisoyi98-ctrl l’ha presa 14 giorni fa e non c’è nessuna pull request aperta. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
pvlib/pvlib-python#2880 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Use "long-wave" rather than "infra-red" to refer to thermal radiation.Forse di nuovo libera @Rahi-padwal l’ha presa 251 giorni fa e non c’è nessuna pull request aperta. Apertaapi easy
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
pvlib/pvlib-python#2653 · 6 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
question
Difficoltà 1/5 1-3 ore Idoneità per principianti 72/100
pvlib/pvlib-python#2637 · 10 commenti ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di pvlib/pvlib-python
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
NousResearch/hermes-agent#140958 ·
I maintainer di solito rispondono entro 1 giorno
-
Bisect issue233186Apertabisect
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
dtcxzyw/llvm-bisect-service#548 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
sallowayma-git/IELTS-practice#248 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 65/100
anthropics/claude-plugins-official#6763 · 10 commenti · 10 reazioni ·
I maintainer di solito rispondono entro 3 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
I maintainer di solito rispondono entro 1 giorno