Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include bifaciality effect in ML module model #241

Closed
timorichert opened this issue Feb 28, 2019 · 4 comments
Closed

Include bifaciality effect in ML module model #241

timorichert opened this issue Feb 28, 2019 · 4 comments
Assignees

Comments

@timorichert
Copy link

Is your feature request related to a problem? Please describe.
Currently, the ML model does not consider bifaciality effects.

Describe the solution you'd like
The backside irradiance should be considered in the effective irradiance sum considering the bifaciality factor, as already implemented in the CEC module model.

@caseyzak24
Copy link
Contributor

@timorichert this is an important feature for us as well. Are you already working on it? If not I will start but I don't want to duplicate efforts.

@caseyzak24
Copy link
Contributor

@timorichert just for reference this would be addressed by a commit I submitted as part of PR #314

@timorichert
Copy link
Author

@caseyzak24 great work and thanks for the effort! I'm currently not working on anything SAM-related, but love to see the current developments!

@allenlawrence94
Copy link
Contributor

Addressed here: #622

@janinefreeman janinefreeman self-assigned this Oct 8, 2021
@janinefreeman janinefreeman added this to the SAM Fall 2021 Release milestone Oct 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants