Skip to content

Conversation

@ramaroesilva
Copy link
Contributor

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.


temperature_cell = func(poa_global, temp_air, wind_speed,
*required, **optional)
if model == 'ross':
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Although wind speed was kept as an input requested by get_cell_temperature, here such a differentiation is needed since ross is apparently the only model not using wind.

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

Successfully merging this pull request may close these issues.

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

1 participant