Inconsistent solar zenith parameter naming in decomposition functions
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
- #2858 của @anishmehta24 — đã đóng, không merge
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 35/100
Hướng nghiên cứu
Review the decomposition functions named in the issue: erbs, erbs_driesse, orgill_hollands, campbell_norman, dirindex, louche, disc, dirint, and boland. Check the related discussion in #1403 and the closed pull request #2858 before making changes. Done means the functions use a consistent solar_zenith parameter name and their relevant tests pass.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Describe the bug
Among the decomposition models, solar zenith assumes two parameter names depending on the function:
zenith(erbs,erbs_driesse,orgill_hollands,campbell_norman,dirindex)solar_zenith(louche,disc,dirint,boland).
This demands adjustments in the code when a user wants to run different decomposition models for comparison.
Expected behavior
Assume a harmonized parameter naming across all functions, solar_zenith seems good to me :-)
Additional context
Relates to #1403. Can be a good issue for first-timers.
- Ngôn ngữ chính
- Python
- Star
- 1.7k
- Fork
- 1.2k
- Merge trung bình
- 3 ngày 18 giờ
- Pull request đã merge (30 ngày)
- 13
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của pvlib/pvlib-python
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
pvlib/pvlib-python#2903 · 2 bình luận · 2 reaction ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 92/100
pvlib/pvlib-python#2902 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Add an example or documentation on using 'bhi' from ERA5Có thể làm lại được @anshurajbisoyi98-ctrl đã nhận 14 ngày trước và không có pull request nào đang mở. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
pvlib/pvlib-python#2880 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Use "long-wave" rather than "infra-red" to refer to thermal radiation.Có thể làm lại được @Rahi-padwal đã nhận 252 ngày trước và không có pull request nào đang mở. Đang mởapi easy
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
pvlib/pvlib-python#2653 · 6 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
question
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 72/100
pvlib/pvlib-python#2637 · 10 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của pvlib/pvlib-python
Issue tương tự
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 68/100
FavioVazquez/showtime#101 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 83/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
rossant/awesome-math#199 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
nautechsystems/nautilus_trader#5365 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Examples say cnxĐang mởdocumentation embedded
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 1 ngày