Consider renaming x to y in bifacial functions
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- Ein halber Tag
- Anfängerfreundlichkeit
- 35/100
Rechercherichtung
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.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- 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 · 2 Kommentare · 2 Reaktionen ·
Maintainer antworten meist innerhalb von 1 Tag
-
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 14 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
-
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 68/100
FavioVazquez/showtime#101 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 83/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
rossant/awesome-math#199 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
nautechsystems/nautilus_trader#5365 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Examples say cnxOffendocumentation embedded
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Maintainer antworten meist innerhalb von 1 Tag