Inconsistent solar zenith parameter naming in decomposition functions
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
- #2858 de @anishmehta24 — cerrado sin fusionar
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Review the decomposition functions named in the issue: erbs, erbs_driesse, orgill_hollands, campbell_norman, dirindex, louche, disc, dirint, and boland. Check the related discussion in #1403 and the closed pull request #2858 before making changes. Done means the functions use a consistent solar_zenith parameter name and their relevant tests pass.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Describe the bug
Among the decomposition models, solar zenith assumes two parameter names depending on the function:
zenith(erbs,erbs_driesse,orgill_hollands,campbell_norman,dirindex)solar_zenith(louche,disc,dirint,boland).
This demands adjustments in the code when a user wants to run different decomposition models for comparison.
Expected behavior
Assume a harmonized parameter naming across all functions, solar_zenith seems good to me :-)
Additional context
Relates to #1403. Can be a good issue for first-timers.
- Lenguaje dominante
- Python
- Estrellas
- 1.7k
- Forks
- 1.2k
- Merge medio
- 3 d 18 h
- PR fusionados (30 d)
- 13
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de pvlib/pvlib-python
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
pvlib/pvlib-python#2903 · 2 comentarios · 2 reacciones ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
pvlib/pvlib-python#2902 ·
Los mantenedores suelen responder en 1 día
-
Add an example or documentation on using 'bhi' from ERA5Quizá libre de nuevo @anshurajbisoyi98-ctrl la tomó hace 13 días y no hay ningún pull request abierto. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
pvlib/pvlib-python#2880 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Use "long-wave" rather than "infra-red" to refer to thermal radiation.Quizá libre de nuevo @Rahi-padwal la tomó hace 251 días y no hay ningún pull request abierto. Abiertoapi easy
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
pvlib/pvlib-python#2653 · 6 comentarios ·
Los mantenedores suelen responder en 1 día
-
question
Dificultad 1/5 1-3 horas Aptitud para principiantes 72/100
pvlib/pvlib-python#2637 · 10 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de pvlib/pvlib-python
Issues similares
-
`lvm_compact` has no self-hosted twin, and `docs/158` §5 says every language-face component mustAbiertobug sev-3 vm
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
sierra-research/tau2-bench#635 ·
Los mantenedores suelen responder en 1 día
-
The add-ssm-model skill requires a Cython simulator; the contributing guide allows Python onesPosiblemente ocupada @fmuia la tomó hoy. Abiertodocumentation skill
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
lnccbrown/ssm-simulators#406 ·
Los mantenedores suelen responder en 4 días
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 95/100
btclib-org/btclib-ecc#309 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
autowarefoundation/autoware_carla_scenario#281 ·
Los mantenedores suelen responder en 1 día