possible typo in modelchain.ModelChain input naming
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
- Issue-Typ
- Dokumentation
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Ruhig
- Tech-Stack
- python
- Bereich
- api, documentation
Rechercherichtung
Start with the ModelChain docstring that documents the aoi_model parameter, then compare it with the parameter name in the constructor to see whether docs and code disagree. The open question is whether to rename the parameter to iam_model or fix the docs, and that rename would affect the public API, so confirm the maintainers' choice before editing. Done when the name is consistent and the maintainers have agreed on the direction.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Describe the bug
Was discussing modelchain.ModelChain with @cbcrespo and noticed that in the documentation of modelchain.ModelChain the input for defining the IAM model is named aoi_model. Find below the description:
aoi_model (str, or function, optional) – If not specified, the model will be inferred from the parameters that are common to all of system.arrays[i].module_parameters. Valid strings are ‘physical’, ‘ashrae’, ‘sapm’, ‘martin_ruiz’, ‘interp’ and ‘no_loss’. The ModelChain instance will be passed as the first argument to a user-defined function.
Expected behavior
Is this naming intended or should it be iam_model?
- 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 · 5 Kommentare · 2 Reaktionen ·
Maintainer antworten meist innerhalb von 1 Tag
-
use offset parameter instead of calculating bday in cdromEvtl. vergeben @chenyi6758-source hat das vor 1 Tag übernommen. Offen
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 16 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 253 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
-
good first issue tools
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Maintainer antworten meist innerhalb von 1 Tag
-
bug deferred domain:ci P1
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
Maintainer antworten meist innerhalb von 1 Tag
-
get_page_thumbnail modifies cached page images, affecting later thumbnails and picture exportsOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
docling-project/docling-mcp#147 ·
-
issue: A pipe model's Valves button ignores the Allow Chat Valves permissionEvtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offenbug confirmed issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
open-webui/open-webui#32771 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Scheduled Security Scan is failing on mainEvtl. vergeben @pkupt hat das heute übernommen. Offensecurity-scan-failure
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 74/100
semantica-agi/semantica#2045 ·
Maintainer antworten meist innerhalb von 2 Tagen