pvlib / pvlib-python

A set of documented functions for simulating the performance of photovoltaic energy systems.
https://pvlib-python.readthedocs.io
BSD 3-Clause "New" or "Revised" License
1.21k stars 1.01k forks source link

Update `racking_model` options in PVSystem and FixedMount classes #2232

Closed RDaxini closed 2 days ago

RDaxini commented 1 month ago
RDaxini commented 1 month ago

I have limited these changes to PVSystem and FixedMount only as these are the two classes mentioned in #1942 I could make further edits, e.g. more formatting, update/add default values for the same (and other) parameters in the other docstrings, etc. either in this PR if recommended, or in a follow up PR if this would become too cluttered (please advise)

RDaxini commented 4 days ago

Good to go?

kandersolar commented 2 days ago

One more thing -- can we copy the new racking_model description to SingleAxisTrackerMount as well?

RDaxini commented 2 days ago

One more thing -- can we copy the new racking_model description to SingleAxisTrackerMount as well?

I think I did originally somewhere in a past commit, but then after some discussion about scope and what was mentioned in the original issue, I reverted that... I'll go ahead and redo!

RDaxini commented 2 days ago

Thanks both for your input and reviewing !