Skip to content

Adds faiman_rad and ross models to get_cell_temperature().#2631

Merged
cwhanse merged 29 commits into
pvlib:mainfrom
ramaroesilva:pvsystem-temp-models
Apr 2, 2026
Merged

Adds faiman_rad and ross models to get_cell_temperature().#2631
cwhanse merged 29 commits into
pvlib:mainfrom
ramaroesilva:pvsystem-temp-models

Conversation

@ramaroesilva
Copy link
Copy Markdown
Contributor

@ramaroesilva ramaroesilva commented Dec 18, 2025

Differently from what was originally discussed with @cwhanse and @echedey-ls, I decided not to include a verification on whether both k and noct are provided within get_cell_temperature. imo, this type of verification should be (and is) done within the temperature.ross function itself and we see that the opposite situation - expected parameters not being provided - is never verified within get_cell_temperature as I imagine it is done within each temperature model function.

@cwhanse about wind and ross model, if that's okay I still included a small note in the wind_speed description to alert ross users that providing a None is enough.

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

pvlib.pvsystem.Array.get_cell_temperature does not accept ross and faiman_rad models

5 participants