Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

ModelChain disregards diffuse optical losses for non-Sandia modules

Đang mở
#2,509 2 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
22/100
Loại issue
Tính năng
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
python
Lĩnh vực
backend

Hướng nghiên cứu

Start with the effective_irradiance_model function in pvlib/modelchain.py, which applies the FD module parameter to poa_diffuse and defaults to 1. Then read martin_ruiz_diffuse and marion_diffuse in pvlib/iam.py to see what they return. The open question is how to choose a diffuse loss model for non-Sandia modules, which the thread does not settle, so agree on that with maintainers before writing code.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Is your feature request related to a problem? Please describe.
modelchain.ModelChain and modelchain.ModelChain.with_pvwatts only consider optical losses for diffuse irradiance for modules from the Sandia Database, which contain an FD parameter for this. This can be seen in the effective_irradiance_model function in here.

def effective_irradiance_model(self):
def _eff_irrad(module_parameters, total_irrad, spect_mod, aoi_mod):
fd = module_parameters.get('FD', 1.)
return spect_mod * (total_irrad['poa_direct'] * aoi_mod +
fd * total_irrad['poa_diffuse'])

In my opinion, this is relevant for pvwatts approach. Not sure for more detailed approaches like single-diode model, etc.

Describe the solution you'd like
pvlib already contains well-established functions to compute diffuse irradiance losses such as the Martin-Ruiz and Marion's models. Would "just" be a matter of implementing them in the modelchain workflow.

Additional context
Even for Sandia modules, could it make sense to allow users to replace the FD constant by values from the Martin-Ruiz or Marion models?

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

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của pvlib/pvlib-python

Tất cả issue của pvlib/pvlib-python

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.