Consider renaming x to y in bifacial functions
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- Meio dia
- Facilidade para iniciantes
- 35/100
Direção de pesquisa
Start with the bifacial coordinate-system description and the function at pvlib/bifacial/utils.py around line 95, which takes a parameter x that means a distance along the y axis. Check the other bifacial modules for the same parameter. The issue does not settle the new name, so confirm the naming with maintainers before editing. Done means the parameter names and docstrings match the coordinate convention and the bifacial tests still pass.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
The reference and local coordinate systems for bifacial functions align x to the East (reference) or along a tracker axis (local), and y to the North (reference) or perpendicular to the tracker axis and parallel to the ground. z is always Up.
Some bifacial functions, e.g., https://github.com/pvlib/pvlib-python/blob/6553710e8e87f0931846eba2abc6f21c19ebdadf/pvlib/bifacial/utils.py#L95, require a parameter x defined as "Position on the ground between two rows", or "Position on the row's slant length". In the coordinate systems, x corresponds to a distance in the y direction.
- 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 há 1 dia. 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á 16 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
-
good first issue tools
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
Mantenedores costumam responder em até 1 dia
-
bug deferred domain:ci P1
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
Mantenedores costumam responder em até 1 dia
-
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 ·
-
issue: A pipe model's Valves button ignores the Allow Chat Valves permissionTalvez já em andamento Um pull request vinculado a esta issue está aberto ou já foi mesclado. Abertabug confirmed issue
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
open-webui/open-webui#32771 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
Scheduled Security Scan is failing on mainTalvez já em andamento @pkupt assumiu hoje. Abertasecurity-scan-failure
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 74/100
semantica-agi/semantica#2045 ·
Mantenedores costumam responder em até 2 dias