Consider updating pvlib.tracking.singleaxis() to return zeros (or similar) for nighttime instead of NaNs
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
The NaN output comes from the night-time branch of singleaxis in pvlib/tracking.py, around the line the issue links. Read the function and the tests in pvlib/tests/test_tracking.py to see which outputs are NaN after sunset. Done means the maintainers have agreed on zero versus NaN for each output, or an opt-in parameter, with tests and docs updated. The design choice is still open, so settle it in the thread before coding.
索引モデルが issue の本文から書いたものです。
説明
Is your feature request related to a problem? Please describe.
pvlib.tracking.singleaxis returns NaNs for intervals where the sun is below the horizon. This can result in NaNs propogating to lots of other pvlib outputs and can cause confusion, other issues, etc. See https://groups.google.com/g/pvlib-python/c/ZeVmTC7eddY as an example.
It appears to happen here:
Describe the solution you'd like
Perhaps change the default to be zero for surface_tilt and tracker_theta (and whatever is most appropriate for aoi and surface_azimuth). Or have an optional input for pvlib.tracking.singleaxis to specify whether it returns NaNs or numbers at night.
Describe alternatives you've considered
Applying .fillna(0) (or similar) to the output of pvlib.tracking.singleaxis is what I typically do.
Additional context
It looks like this zero vs nan situation was briefly discussed in https://github.com/pvlib/pvlib-python/issues/569. It seems it would be worth further discussion. Is there a significant reason that NaN is the output at night?
fillna(0) is used in the gallery here and here, but not here.
Tagging @cwhanse.
- 主要言語
- 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 · コメント 2 件 · リアクション 2 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 92/100
pvlib/pvlib-python#2902 ·
メンテナーはふだん 1 日以内に返信
-
Add an example or documentation on using 'bhi' from ERA5再び着手できるかも @anshurajbisoyi98-ctrl が 14 日前に担当しましたが、オープン中のプルリクエストはありません。 オープン
難易度 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
-
難易度 1/5 1時間未満 初心者へのやさしさ 65/100
521xueweihan/HelloGitHub#4025 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
aws-samples/sample-per-model-bedrock#321 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1〜3時間 初心者へのやさしさ 68/100
FavioVazquez/showtime#101 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
koush/ha_scrypted#81 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
google/osv-scanner-action#147 ·