Consider renaming x to y in bifacial functions
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
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.
由索引模型根据 Issue 内容生成。
描述
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.
- 主要语言
- Python
- 星标
- 1.7k
- 派生
- 1.2k
- 平均合并
- 3 天 18 小时
- 30 天内合并 PR
- 13
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
pvlib/pvlib-python 的其他 Issue
-
bug
难度 2/5 1-3 小时 新手友好度 78/100
pvlib/pvlib-python#2903 · 2 条评论 · 2 个 reaction ·
维护者通常 1 天内回复
-
难度 1/5 1 小时以内 新手友好度 92/100
pvlib/pvlib-python#2902 ·
维护者通常 1 天内回复
-
Add an example or documentation on using 'bhi' from ERA5可能重新可做 @anshurajbisoyi98-ctrl 于 13 天前认领，目前没有进行中的 PR。 未关闭
难度 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 于 251 天前认领，目前没有进行中的 PR。 未关闭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 小时 新手友好度 82/100
sierra-research/tau2-bench#635 ·
维护者通常 1 天内回复
-
The add-ssm-model skill requires a Cython simulator; the contributing guide allows Python ones可能已有人在做 @fmuia 今天认领。 未关闭documentation skill
难度 2/5 1-3 小时 新手友好度 83/100
lnccbrown/ssm-simulators#406 ·
维护者通常 4 天内回复
-
难度 1/5 1 小时以内 新手友好度 95/100
btclib-org/btclib-ecc#309 ·
维护者通常 1 天内回复
-
bug
难度 2/5 1-3 小时 新手友好度 68/100
jbacalso24/mimry#459 · 1 条评论 ·
维护者通常 1 天内回复
-
auth bug high-priority
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 1 天内回复