Inconsistent solar zenith parameter naming in decomposition functions
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
- #2858 de @anishmehta24 — fechado sem integrar
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 35/100
Direção de pesquisa
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.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- Python
- Estrelas
- 1.7k
- Forks
- 1.2k
- Merge médio
- 3d 18h
- PRs com merge (30d)
- 13
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de pvlib/pvlib-python
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
pvlib/pvlib-python#2903 · 5 comentários · 2 reações ·
Mantenedores costumam responder em até 1 dia
-
use offset parameter instead of calculating bday in cdromTalvez já em andamento @chenyi6758-source assumiu hoje. Aberta
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 92/100
pvlib/pvlib-python#2902 ·
Mantenedores costumam responder em até 1 dia
-
Add an example or documentation on using 'bhi' from ERA5Talvez livre de novo @anshurajbisoyi98-ctrl assumiu há 15 dias e não há nenhum pull request aberto. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 66/100
pvlib/pvlib-python#2880 · 2 comentários ·
Mantenedores costumam responder em até 1 dia
-
Use "long-wave" rather than "infra-red" to refer to thermal radiation.Talvez livre de novo @Rahi-padwal assumiu há 253 dias e não há nenhum pull request aberto. Abertaapi easy
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 62/100
pvlib/pvlib-python#2653 · 6 comentários ·
Mantenedores costumam responder em até 1 dia
-
question
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 72/100
pvlib/pvlib-python#2637 · 10 comentários ·
Mantenedores costumam responder em até 1 dia
Todas as issues de pvlib/pvlib-python
Issues semelhantes
-
get_page_thumbnail modifies cached page images, affecting later thumbnails and picture exportsAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
docling-project/docling-mcp#147 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 80/100
Mantenedores costumam responder em até 1 dia
-
enhancement
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
autowarefoundation/autoware_carla_scenario#405 ·
Mantenedores costumam responder em até 1 dia
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 79/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
mohitmishra786/readprism#322 ·
Mantenedores costumam responder em até 1 dia