Refactor duplicated OrderedDict/DataFrame return logic in clearsky models
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 50/100
Rechercherichtung
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.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- Python
- Sterne
- 1.7k
- Forks
- 1.2k
- Ø Merge
- 3 T. 18 Std.
- Gemergte PRs (30 T.)
- 13
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus pvlib/pvlib-python
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
pvlib/pvlib-python#2903 · 5 Kommentare · 2 Reaktionen ·
Maintainer antworten meist innerhalb von 1 Tag
-
use offset parameter instead of calculating bday in cdromEvtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offen
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 92/100
pvlib/pvlib-python#2902 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Add an example or documentation on using 'bhi' from ERA5Evtl. wieder frei @anshurajbisoyi98-ctrl hat das vor 15 Tagen übernommen, und es ist kein Pull Request offen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 66/100
pvlib/pvlib-python#2880 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Use "long-wave" rather than "infra-red" to refer to thermal radiation.Evtl. wieder frei @Rahi-padwal hat das vor 252 Tagen übernommen, und es ist kein Pull Request offen. Offenapi easy
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
pvlib/pvlib-python#2653 · 6 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
question
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 72/100
pvlib/pvlib-python#2637 · 10 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in pvlib/pvlib-python
Ähnliche Issues
-
comp/cli P3 type/security
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
NousResearch/hermes-agent#141420 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
vendor-pitch
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 87/100
Maintainer antworten meist innerhalb von 1 Tag
-
bug llm p2
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
Maintainer antworten meist innerhalb von 1 Tag
-
New tool: ZIPZAX Python Notebook StudioEvtl. vergeben @github-actions hat das heute übernommen. Offensubmission triage
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 68/100
kausalflow/tools#226 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100