Refactor duplicated OrderedDict/DataFrame return logic in clearsky models
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 50/100
Piste de recherche
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.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- Python
- Étoiles
- 1.7k
- Forks
- 1.2k
- Merge moyen
- 3 j 18 h
- PR mergées (30 j)
- 16
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de pvlib/pvlib-python
-
Possibility of beam irradiance from behind plane of arrayPeut-être à nouveau libre Une pull request pour cette issue a été fermée sans être fusionnée. Ouvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
pvlib/pvlib-python#2903 · 5 commentaires · 2 réactions ·
Les mainteneurs répondent en général sous 1 jour
-
use offset parameter instead of calculating bday in cdromPeut-être pris @chenyi6758-source l’a pris il y a 2 jours. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 92/100
pvlib/pvlib-python#2902 ·
Les mainteneurs répondent en général sous 1 jour
-
Add an example or documentation on using 'bhi' from ERA5Peut-être à nouveau libre @anshurajbisoyi98-ctrl l’a pris il y a 17 jours, et aucune pull request n’est ouverte. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
pvlib/pvlib-python#2880 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Use "long-wave" rather than "infra-red" to refer to thermal radiation.Peut-être à nouveau libre @Rahi-padwal l’a pris il y a 255 jours, et aucune pull request n’est ouverte. Ouverteapi easy
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
pvlib/pvlib-python#2653 · 6 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
question
Difficulté 1/5 1-3 heures Accessibilité débutants 72/100
pvlib/pvlib-python#2637 · 10 commentaires ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de pvlib/pvlib-python
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
zach-blumenfeld/aip#13 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 80/100
nathanmarlor/foxess_modbus#1287 ·
Les mainteneurs répondent en général sous 4 jours
-
bug good first issue help wanted
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
OpenLakeForge/openlakeforge#498 ·
Les mainteneurs répondent en général sous 1 jour
-
good first issue hacktoberfest help wanted python
Difficulté 2/5 1-3 heures Accessibilité débutants 73/100
Les mainteneurs répondent en général sous 1 jour
-
Exact onnxruntime==1.27.0 core pin makes Pipecat-based apps uninstallable from the dashboardOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 69/100
pollen-robotics/reachy_mini#1493 ·
Les mainteneurs répondent en général sous 1 jour