possible typo in modelchain.ModelChain input naming
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 半日
- 初心者へのやさしさ
- 35/100
- issue の種類
- ドキュメント
- 明瞭さ
- おおむね明確
- 活発さ
- 静か
- 技術スタック
- python
- 領域
- api, documentation
調査の方向性
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.
索引モデルが issue の本文から書いたものです。
説明
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?
- 主要言語
- Python
- スター
- 1.7k
- フォーク
- 1.2k
- 平均マージ
- 3日 18時間
- マージ済み PR（30日）
- 13
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
pvlib/pvlib-python のほかの issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
pvlib/pvlib-python#2903 · コメント 5 件 · リアクション 2 件 ·
メンテナーはふだん 1 日以内に返信
-
use offset parameter instead of calculating bday in cdrom対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 1/5 1時間未満 初心者へのやさしさ 92/100
pvlib/pvlib-python#2902 ·
メンテナーはふだん 1 日以内に返信
-
Add an example or documentation on using 'bhi' from ERA5再び着手できるかも @anshurajbisoyi98-ctrl が 15 日前に担当しましたが、オープン中のプルリクエストはありません。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
pvlib/pvlib-python#2880 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
Use "long-wave" rather than "infra-red" to refer to thermal radiation.再び着手できるかも @Rahi-padwal が 252 日前に担当しましたが、オープン中のプルリクエストはありません。 オープンapi easy
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
pvlib/pvlib-python#2653 · コメント 6 件 ·
メンテナーはふだん 1 日以内に返信
-
question
難易度 1/5 1〜3時間 初心者へのやさしさ 72/100
pvlib/pvlib-python#2637 · コメント 10 件 ·
メンテナーはふだん 1 日以内に返信
pvlib/pvlib-python の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
SwiftKickMobile/skai#84 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 82/100
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
jxxghp/MoviePilot#7070 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
flop-labs/technocore-chat#997 ·
メンテナーはふだん 1 日以内に返信
-
tunneld on Windows: Wi-Fi devices are not discovered because uvicorn runs on a ProactorEventLoopオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
doronz88/pymobiledevice3#2051 ·
メンテナーはふだん 1 日以内に返信