From e5495cbfcba37d2c84fa36c2753dd3fad9ef56f8 Mon Sep 17 00:00:00 2001 From: Cliff Hansen Date: Mon, 11 Nov 2024 15:03:35 -0800 Subject: [PATCH 01/10] describe racking model choices for SAPM temperature models --- pvlib/pvsystem.py | 21 ++++++++++++++------- pvlib/temperature.py | 24 ++++++++++++++++++++++++ 2 files changed, 38 insertions(+), 7 deletions(-) diff --git a/pvlib/pvsystem.py b/pvlib/pvsystem.py index 76872248a4..77b8f855a3 100644 --- a/pvlib/pvsystem.py +++ b/pvlib/pvsystem.py @@ -170,9 +170,11 @@ class PVSystem: inverter_parameters : dict or Series, optional Inverter parameters as defined by the SAPM, CEC, or other. - racking_model : string, default 'open_rack' - Valid strings are 'open_rack', 'close_mount', and 'insulated_back'. - Used to identify a parameter set for the SAPM cell temperature model. + racking_model : string, optional + Valid strings are ``'open_rack'``, ``'close_mount'``, and + ``'insulated_back'``. Used to identify a parameter set for the SAPM + cell temperature model. See :py:func:`pvlib.temperature.sapm_module` + for definitions. losses_parameters : dict or Series, optional Losses parameters as defined by PVWatts or other. @@ -1395,8 +1397,10 @@ class FixedMount(AbstractMount): West=270. [degrees] racking_model : str, optional - Valid strings are 'open_rack', 'close_mount', and 'insulated_back'. - Used to identify a parameter set for the SAPM cell temperature model. + Valid strings are ``'open_rack'``, ``'close_mount'``, and + ``'insulated_back'``. Used to identify a parameter set for the SAPM + cell temperature model. See :py:func:`pvlib.temperature.sapm_module` + for definitions. module_height : float, optional The height above ground of the center of the module [m]. Used for @@ -1472,8 +1476,11 @@ class SingleAxisTrackerMount(AbstractMount): `cross_axis_tilt`. [degrees] racking_model : str, optional - Valid strings are 'open_rack', 'close_mount', and 'insulated_back'. - Used to identify a parameter set for the SAPM cell temperature model. + Valid strings are ``'open_rack'``, ``'close_mount'``, and + ``'insulated_back'`. Used to identify a parameter set for the SAPM + cell temperature model. ``'open_rack'`` should be used for systems + with single-axis trackers. See :py:func:`pvlib.temperature.sapm_module` + for definitions. module_height : float, optional The height above ground of the center of the module [m]. Used for diff --git a/pvlib/temperature.py b/pvlib/temperature.py index 6269014969..756d60c969 100644 --- a/pvlib/temperature.py +++ b/pvlib/temperature.py @@ -125,6 +125,18 @@ def sapm_cell(poa_global, temp_air, wind_speed, a, b, deltaT, | glass/polymer | insulated back | -2.81 | -0.0455 | 0 | +---------------+----------------+-------+---------+---------------------+ + Mounting cases can be described in terms of air flow across and around the + rear-facing surface of the module: + + * "open rack" refers to mounting that allows relatively free air flow. + This case is typical of ground-mounted systems on fixed racking or + single axis trackers. + * "close roof" refers to limited or restricted air flow. This case is + typical of roof-mounted systems with some gap behind the module. + * "insulated back" refers to systems with no air flow contacting the rear + surface of the module. This case is typical of building-integrated PV + systems, or systems laid flat on a ground surface. + References ---------- .. [1] King, D. et al, 2004, "Sandia Photovoltaic Array Performance @@ -206,6 +218,18 @@ def sapm_module(poa_global, temp_air, wind_speed, a, b): | glass/polymer | insulated back | -2.81 | -0.0455 | 0 | +---------------+----------------+-------+---------+---------------------+ + Mounting cases can be described in terms of air flow across and around the + rear-facing surface of the module: + + * "open rack" refers to mounting that allows relatively free air flow. + This case is typical of ground-mounted systems on fixed racking or + single axis trackers. + * "close roof" refers to limited or restricted air flow. This case is + typical of roof-mounted systems with some gap behind the module. + * "insulated back" refers to systems with no air flow contacting the rear + surface of the module. This case is typical of building-integrated PV + systems, or systems laid flat on a ground surface. + References ---------- .. [1] King, D. et al, 2004, "Sandia Photovoltaic Array Performance From 1356d0578c9bb840f2331b82e2156e9f63331b45 Mon Sep 17 00:00:00 2001 From: Cliff Hansen Date: Mon, 11 Nov 2024 15:09:23 -0800 Subject: [PATCH 02/10] whatsnew --- docs/sphinx/source/whatsnew/v0.11.2.rst | 2 ++ 1 file changed, 2 insertions(+) diff --git a/docs/sphinx/source/whatsnew/v0.11.2.rst b/docs/sphinx/source/whatsnew/v0.11.2.rst index a07f4bf43d..bdee21cbe3 100644 --- a/docs/sphinx/source/whatsnew/v0.11.2.rst +++ b/docs/sphinx/source/whatsnew/v0.11.2.rst @@ -33,6 +33,8 @@ Documentation page, using the sphinx glossary directive. (:issue:`1421`, :pull:`2234`) * Explained how to write docstrings for new functions in :ref:`example-docstring` (:discussion:`2081`, :pull:`2254`) +* Clarify mounting cases for parameters for :py:func:`~pvlib.temperature.sapm_module` + and :py:func:`~pvlib.temperature.cell`. (:issue:`1323`, :pull:`2293`) * Added the following variables to the :ref:`nomenclature` page: - `spectra` and `spectra_components` (:issue:`2150`, :pull:`2264`) From 4d5b6405a23e5919868c921dd6ef777213113f94 Mon Sep 17 00:00:00 2001 From: Cliff Hansen Date: Mon, 11 Nov 2024 15:17:35 -0800 Subject: [PATCH 03/10] doc edits --- pvlib/pvsystem.py | 8 ++++---- pvlib/temperature.py | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/pvlib/pvsystem.py b/pvlib/pvsystem.py index 77b8f855a3..df6d4dc1c1 100644 --- a/pvlib/pvsystem.py +++ b/pvlib/pvsystem.py @@ -173,7 +173,7 @@ class PVSystem: racking_model : string, optional Valid strings are ``'open_rack'``, ``'close_mount'``, and ``'insulated_back'``. Used to identify a parameter set for the SAPM - cell temperature model. See :py:func:`pvlib.temperature.sapm_module` + cell temperature model. See :py:func:`~pvlib.temperature.sapm_module` for definitions. losses_parameters : dict or Series, optional @@ -1399,7 +1399,7 @@ class FixedMount(AbstractMount): racking_model : str, optional Valid strings are ``'open_rack'``, ``'close_mount'``, and ``'insulated_back'``. Used to identify a parameter set for the SAPM - cell temperature model. See :py:func:`pvlib.temperature.sapm_module` + cell temperature model. See :py:func:`~pvlib.temperature.sapm_module` for definitions. module_height : float, optional @@ -1479,8 +1479,8 @@ class SingleAxisTrackerMount(AbstractMount): Valid strings are ``'open_rack'``, ``'close_mount'``, and ``'insulated_back'`. Used to identify a parameter set for the SAPM cell temperature model. ``'open_rack'`` should be used for systems - with single-axis trackers. See :py:func:`pvlib.temperature.sapm_module` - for definitions. + with single-axis trackers. + See :py:func:`~pvlib.temperature.sapm_module` for definitions. module_height : float, optional The height above ground of the center of the module [m]. Used for diff --git a/pvlib/temperature.py b/pvlib/temperature.py index 756d60c969..2ce57001d6 100644 --- a/pvlib/temperature.py +++ b/pvlib/temperature.py @@ -127,7 +127,7 @@ def sapm_cell(poa_global, temp_air, wind_speed, a, b, deltaT, Mounting cases can be described in terms of air flow across and around the rear-facing surface of the module: - + * "open rack" refers to mounting that allows relatively free air flow. This case is typical of ground-mounted systems on fixed racking or single axis trackers. @@ -220,7 +220,7 @@ def sapm_module(poa_global, temp_air, wind_speed, a, b): Mounting cases can be described in terms of air flow across and around the rear-facing surface of the module: - + * "open rack" refers to mounting that allows relatively free air flow. This case is typical of ground-mounted systems on fixed racking or single axis trackers. From 34f98a8e9a5b12566a0584d208e9e659cfc5dd7c Mon Sep 17 00:00:00 2001 From: Cliff Hansen Date: Wed, 11 Dec 2024 08:43:52 -0700 Subject: [PATCH 04/10] add freestanding back in --- pvlib/pvsystem.py | 22 ++++++++++++---------- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/pvlib/pvsystem.py b/pvlib/pvsystem.py index 3aca55512a..057be99aa0 100644 --- a/pvlib/pvsystem.py +++ b/pvlib/pvsystem.py @@ -168,9 +168,10 @@ class PVSystem: Inverter parameters as defined by the SAPM, CEC, or other. racking_model : string, optional - Valid strings are ``'open_rack'``, ``'close_mount'``, and - ``'insulated_back'``. Used to identify a parameter set for the SAPM - cell temperature model. See :py:func:`~pvlib.temperature.sapm_module` + Valid strings are ``'open_rack'``, ``'close_mount'``, + ``'freestanding'``, and ``'insulated_back'`. Used to identify a + parameter set for the SAPM cell temperature model. + See :py:func:`~pvlib.temperature.sapm_module` for definitions. losses_parameters : dict or Series, optional @@ -1394,9 +1395,10 @@ class FixedMount(AbstractMount): West=270. [degrees] racking_model : str, optional - Valid strings are ``'open_rack'``, ``'close_mount'``, and - ``'insulated_back'``. Used to identify a parameter set for the SAPM - cell temperature model. See :py:func:`~pvlib.temperature.sapm_module` + Valid strings are ``'open_rack'``, ``'close_mount'``, + ``'freestanding'``, and ``'insulated_back'`. Used to identify a + parameter set for the SAPM cell temperature model. + See :py:func:`~pvlib.temperature.sapm_module` for definitions. module_height : float, optional @@ -1473,10 +1475,10 @@ class SingleAxisTrackerMount(AbstractMount): `cross_axis_tilt`. [degrees] racking_model : str, optional - Valid strings are ``'open_rack'``, ``'close_mount'``, and - ``'insulated_back'`. Used to identify a parameter set for the SAPM - cell temperature model. ``'open_rack'`` should be used for systems - with single-axis trackers. + Valid strings are ``'open_rack'``, ``'close_mount'``, + ``'freestanding'``, and ``'insulated_back'`. Used to identify a + parameter set for the SAPM cell temperature model. + ``'open_rack'`` should be used for systems with single-axis trackers. See :py:func:`~pvlib.temperature.sapm_module` for definitions. module_height : float, optional From ce8b75c75869e43c3ddea7becbe24c5a4ec8a083 Mon Sep 17 00:00:00 2001 From: Cliff Hansen Date: Wed, 11 Dec 2024 09:00:31 -0700 Subject: [PATCH 05/10] definitions for pvsyst racking --- pvlib/temperature.py | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+) diff --git a/pvlib/temperature.py b/pvlib/temperature.py index 2ce57001d6..1cde37d113 100644 --- a/pvlib/temperature.py +++ b/pvlib/temperature.py @@ -300,6 +300,18 @@ def sapm_cell_from_module(module_temperature, poa_global, deltaT, | glass/polymer | insulated back | -2.81 | -0.0455 | 0 | +---------------+----------------+-------+---------+---------------------+ + Mounting cases can be described in terms of air flow across and around the + rear-facing surface of the module: + + * "open rack" refers to mounting that allows relatively free air flow. + This case is typical of ground-mounted systems on fixed racking or + single axis trackers. + * "close roof" refers to limited or restricted air flow. This case is + typical of roof-mounted systems with some gap behind the module. + * "insulated back" refers to systems with no air flow contacting the rear + surface of the module. This case is typical of building-integrated PV + systems, or systems laid flat on a ground surface. + References ---------- .. [1] King, D. et al, 2004, "Sandia Photovoltaic Array Performance @@ -384,6 +396,16 @@ def pvsyst_cell(poa_global, temp_air, wind_speed=1.0, u_c=29.0, u_v=0.0, | insulated | 15.0 | 0.0 | +--------------+---------------+---------------+ + Mounting cases can be described in terms of air flow across and around the + rear-facing surface of the module: + + * "freestanding" refers to mounting that allows relatively free air + circulation around the modules. This case is typical of ground-mounted + systems on tilted, fixed racking or single axis trackers. + * "insluated" refers to mounting with air flow across only the front + surface. This case is typical of roof-mounted systems with no gap + behind the module. + References ---------- .. [1] "PVsyst 7 Help", [Online]. Available: From 285cc1ace745c8dac94f9beffb2fd6bce627d97f Mon Sep 17 00:00:00 2001 From: Cliff Hansen Date: Wed, 11 Dec 2024 09:02:32 -0700 Subject: [PATCH 06/10] add link to pvsyst_cell --- pvlib/pvsystem.py | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/pvlib/pvsystem.py b/pvlib/pvsystem.py index 057be99aa0..ca9b578cea 100644 --- a/pvlib/pvsystem.py +++ b/pvlib/pvsystem.py @@ -171,8 +171,8 @@ class PVSystem: Valid strings are ``'open_rack'``, ``'close_mount'``, ``'freestanding'``, and ``'insulated_back'`. Used to identify a parameter set for the SAPM cell temperature model. - See :py:func:`~pvlib.temperature.sapm_module` - for definitions. + See :py:func:`~pvlib.temperature.sapm_module` and + :py:func:`~pvlib.temperature.pvsyst_cell` for definitions. losses_parameters : dict or Series, optional Losses parameters as defined by PVWatts or other. @@ -1398,8 +1398,8 @@ class FixedMount(AbstractMount): Valid strings are ``'open_rack'``, ``'close_mount'``, ``'freestanding'``, and ``'insulated_back'`. Used to identify a parameter set for the SAPM cell temperature model. - See :py:func:`~pvlib.temperature.sapm_module` - for definitions. + See :py:func:`~pvlib.temperature.sapm_module` and + :py:func:`~pvlib.temperature.pvsyst_cell` for definitions. module_height : float, optional The height above ground of the center of the module [m]. Used for @@ -1479,7 +1479,8 @@ class SingleAxisTrackerMount(AbstractMount): ``'freestanding'``, and ``'insulated_back'`. Used to identify a parameter set for the SAPM cell temperature model. ``'open_rack'`` should be used for systems with single-axis trackers. - See :py:func:`~pvlib.temperature.sapm_module` for definitions. + See :py:func:`~pvlib.temperature.sapm_module` and + :py:func:`~pvlib.temperature.pvsyst_cell` for definitions. module_height : float, optional The height above ground of the center of the module [m]. Used for From 46ae6a75fa5c8a6bac51b151f20ece1f35963c19 Mon Sep 17 00:00:00 2001 From: Cliff Hansen Date: Wed, 11 Dec 2024 09:04:08 -0700 Subject: [PATCH 07/10] update whatsnew --- docs/sphinx/source/whatsnew/v0.11.2.rst | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/docs/sphinx/source/whatsnew/v0.11.2.rst b/docs/sphinx/source/whatsnew/v0.11.2.rst index 7be4387657..02b7f6d16d 100644 --- a/docs/sphinx/source/whatsnew/v0.11.2.rst +++ b/docs/sphinx/source/whatsnew/v0.11.2.rst @@ -48,7 +48,8 @@ Documentation * Explained how to write docstrings for new functions in :ref:`example-docstring` (:discussion:`2081`, :pull:`2254`) * Clarify mounting cases for parameters for :py:func:`~pvlib.temperature.sapm_module` - and :py:func:`~pvlib.temperature.cell`. (:issue:`1323`, :pull:`2293`) + :py:func:`~pvlib.temperature.sapm_cell` and :py:func:`~pvlib.temperature.pvsyst_cell`. + (:issue:`1323`, :pull:`2293`) * Added the following variables to the :ref:`nomenclature` page: - `spectra` and `spectra_components` (:issue:`2150`, :pull:`2264`) From c5c56e2293597996b9e49e1c5e6b4f3adc6ffe01 Mon Sep 17 00:00:00 2001 From: Cliff Hansen Date: Wed, 11 Dec 2024 10:05:41 -0700 Subject: [PATCH 08/10] review comments --- docs/sphinx/source/whatsnew/v0.11.2.rst | 4 ++-- pvlib/pvsystem.py | 17 ++++++++++------- pvlib/temperature.py | 14 +++++++------- 3 files changed, 19 insertions(+), 16 deletions(-) diff --git a/docs/sphinx/source/whatsnew/v0.11.2.rst b/docs/sphinx/source/whatsnew/v0.11.2.rst index 02b7f6d16d..4434e10097 100644 --- a/docs/sphinx/source/whatsnew/v0.11.2.rst +++ b/docs/sphinx/source/whatsnew/v0.11.2.rst @@ -47,9 +47,9 @@ Documentation page, using the sphinx glossary directive. (:issue:`1421`, :pull:`2234`) * Explained how to write docstrings for new functions in :ref:`example-docstring` (:discussion:`2081`, :pull:`2254`) -* Clarify mounting cases for parameters for :py:func:`~pvlib.temperature.sapm_module` +* Clarify mounting cases for parameters for :py:func:`~pvlib.temperature.sapm_module`, :py:func:`~pvlib.temperature.sapm_cell` and :py:func:`~pvlib.temperature.pvsyst_cell`. - (:issue:`1323`, :pull:`2293`) + (:issue:`1323`, :pull:`2293`) * Added the following variables to the :ref:`nomenclature` page: - `spectra` and `spectra_components` (:issue:`2150`, :pull:`2264`) diff --git a/pvlib/pvsystem.py b/pvlib/pvsystem.py index ca9b578cea..05d909e95e 100644 --- a/pvlib/pvsystem.py +++ b/pvlib/pvsystem.py @@ -169,8 +169,9 @@ class PVSystem: racking_model : string, optional Valid strings are ``'open_rack'``, ``'close_mount'``, - ``'freestanding'``, and ``'insulated_back'`. Used to identify a - parameter set for the SAPM cell temperature model. + ``'insulated_back'`, ``'freestanding'`` and ``'insulated'``. + Used to identify a parameter set for the SAPM or PVsyst cell + temperature model. See :py:func:`~pvlib.temperature.sapm_module` and :py:func:`~pvlib.temperature.pvsyst_cell` for definitions. @@ -1396,8 +1397,9 @@ class FixedMount(AbstractMount): racking_model : str, optional Valid strings are ``'open_rack'``, ``'close_mount'``, - ``'freestanding'``, and ``'insulated_back'`. Used to identify a - parameter set for the SAPM cell temperature model. + ``'insulated_back'`, ``'freestanding'`` and ``'insulated'``. + Used to identify a parameter set for the SAPM or PVsyst cell + temperature model. See :py:func:`~pvlib.temperature.sapm_module` and :py:func:`~pvlib.temperature.pvsyst_cell` for definitions. @@ -1476,9 +1478,10 @@ class SingleAxisTrackerMount(AbstractMount): racking_model : str, optional Valid strings are ``'open_rack'``, ``'close_mount'``, - ``'freestanding'``, and ``'insulated_back'`. Used to identify a - parameter set for the SAPM cell temperature model. - ``'open_rack'`` should be used for systems with single-axis trackers. + ``'insulated_back'`, ``'freestanding'`` and ``'insulated'``. + Used to identify a parameter set for the SAPM or PVsyst cell + temperature model. ``'open_rack'`` should be used for systems with + single-axis trackers. See :py:func:`~pvlib.temperature.sapm_module` and :py:func:`~pvlib.temperature.pvsyst_cell` for definitions. diff --git a/pvlib/temperature.py b/pvlib/temperature.py index 1cde37d113..ab31ffaf56 100644 --- a/pvlib/temperature.py +++ b/pvlib/temperature.py @@ -118,7 +118,7 @@ def sapm_cell(poa_global, temp_air, wind_speed, a, b, deltaT, +===============+================+=======+=========+=====================+ | glass/glass | open rack | -3.47 | -0.0594 | 3 | +---------------+----------------+-------+---------+---------------------+ - | glass/glass | close roof | -2.98 | -0.0471 | 1 | + | glass/glass | close mount | -2.98 | -0.0471 | 1 | +---------------+----------------+-------+---------+---------------------+ | glass/polymer | open rack | -3.56 | -0.075 | 3 | +---------------+----------------+-------+---------+---------------------+ @@ -131,7 +131,7 @@ def sapm_cell(poa_global, temp_air, wind_speed, a, b, deltaT, * "open rack" refers to mounting that allows relatively free air flow. This case is typical of ground-mounted systems on fixed racking or single axis trackers. - * "close roof" refers to limited or restricted air flow. This case is + * "close mount" refers to limited or restricted air flow. This case is typical of roof-mounted systems with some gap behind the module. * "insulated back" refers to systems with no air flow contacting the rear surface of the module. This case is typical of building-integrated PV @@ -211,7 +211,7 @@ def sapm_module(poa_global, temp_air, wind_speed, a, b): +===============+================+=======+=========+=====================+ | glass/glass | open rack | -3.47 | -0.0594 | 3 | +---------------+----------------+-------+---------+---------------------+ - | glass/glass | close roof | -2.98 | -0.0471 | 1 | + | glass/glass | close mount | -2.98 | -0.0471 | 1 | +---------------+----------------+-------+---------+---------------------+ | glass/polymer | open rack | -3.56 | -0.075 | 3 | +---------------+----------------+-------+---------+---------------------+ @@ -224,7 +224,7 @@ def sapm_module(poa_global, temp_air, wind_speed, a, b): * "open rack" refers to mounting that allows relatively free air flow. This case is typical of ground-mounted systems on fixed racking or single axis trackers. - * "close roof" refers to limited or restricted air flow. This case is + * "close mount" refers to limited or restricted air flow. This case is typical of roof-mounted systems with some gap behind the module. * "insulated back" refers to systems with no air flow contacting the rear surface of the module. This case is typical of building-integrated PV @@ -293,7 +293,7 @@ def sapm_cell_from_module(module_temperature, poa_global, deltaT, +===============+================+=======+=========+=====================+ | glass/glass | open rack | -3.47 | -0.0594 | 3 | +---------------+----------------+-------+---------+---------------------+ - | glass/glass | close roof | -2.98 | -0.0471 | 1 | + | glass/glass | close mount | -2.98 | -0.0471 | 1 | +---------------+----------------+-------+---------+---------------------+ | glass/polymer | open rack | -3.56 | -0.075 | 3 | +---------------+----------------+-------+---------+---------------------+ @@ -306,7 +306,7 @@ def sapm_cell_from_module(module_temperature, poa_global, deltaT, * "open rack" refers to mounting that allows relatively free air flow. This case is typical of ground-mounted systems on fixed racking or single axis trackers. - * "close roof" refers to limited or restricted air flow. This case is + * "close mount" refers to limited or restricted air flow. This case is typical of roof-mounted systems with some gap behind the module. * "insulated back" refers to systems with no air flow contacting the rear surface of the module. This case is typical of building-integrated PV @@ -402,7 +402,7 @@ def pvsyst_cell(poa_global, temp_air, wind_speed=1.0, u_c=29.0, u_v=0.0, * "freestanding" refers to mounting that allows relatively free air circulation around the modules. This case is typical of ground-mounted systems on tilted, fixed racking or single axis trackers. - * "insluated" refers to mounting with air flow across only the front + * "insulated" refers to mounting with air flow across only the front surface. This case is typical of roof-mounted systems with no gap behind the module. From ae64732cfb8a64039ffdcc2b82732c505bfe40ce Mon Sep 17 00:00:00 2001 From: Cliff Hansen Date: Wed, 11 Dec 2024 10:15:41 -0700 Subject: [PATCH 09/10] pvsyst for trackers --- pvlib/pvsystem.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pvlib/pvsystem.py b/pvlib/pvsystem.py index 05d909e95e..4a5ac105e4 100644 --- a/pvlib/pvsystem.py +++ b/pvlib/pvsystem.py @@ -1480,8 +1480,8 @@ class SingleAxisTrackerMount(AbstractMount): Valid strings are ``'open_rack'``, ``'close_mount'``, ``'insulated_back'`, ``'freestanding'`` and ``'insulated'``. Used to identify a parameter set for the SAPM or PVsyst cell - temperature model. ``'open_rack'`` should be used for systems with - single-axis trackers. + temperature model. ``'open_rack'`` or ``'freestanding'`` should + be used for systems with single-axis trackers. See :py:func:`~pvlib.temperature.sapm_module` and :py:func:`~pvlib.temperature.pvsyst_cell` for definitions. From a8e719bcf2a46f6d93a074b606388904d831fcbd Mon Sep 17 00:00:00 2001 From: Cliff Hansen Date: Wed, 11 Dec 2024 10:59:40 -0700 Subject: [PATCH 10/10] Apply suggestions from code review Co-authored-by: Kevin Anderson --- pvlib/pvsystem.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/pvlib/pvsystem.py b/pvlib/pvsystem.py index 4a5ac105e4..f99f0275af 100644 --- a/pvlib/pvsystem.py +++ b/pvlib/pvsystem.py @@ -169,7 +169,7 @@ class PVSystem: racking_model : string, optional Valid strings are ``'open_rack'``, ``'close_mount'``, - ``'insulated_back'`, ``'freestanding'`` and ``'insulated'``. + ``'insulated_back'``, ``'freestanding'`` and ``'insulated'``. Used to identify a parameter set for the SAPM or PVsyst cell temperature model. See :py:func:`~pvlib.temperature.sapm_module` and @@ -1397,7 +1397,7 @@ class FixedMount(AbstractMount): racking_model : str, optional Valid strings are ``'open_rack'``, ``'close_mount'``, - ``'insulated_back'`, ``'freestanding'`` and ``'insulated'``. + ``'insulated_back'``, ``'freestanding'`` and ``'insulated'``. Used to identify a parameter set for the SAPM or PVsyst cell temperature model. See :py:func:`~pvlib.temperature.sapm_module` and @@ -1478,7 +1478,7 @@ class SingleAxisTrackerMount(AbstractMount): racking_model : str, optional Valid strings are ``'open_rack'``, ``'close_mount'``, - ``'insulated_back'`, ``'freestanding'`` and ``'insulated'``. + ``'insulated_back'``, ``'freestanding'`` and ``'insulated'``. Used to identify a parameter set for the SAPM or PVsyst cell temperature model. ``'open_rack'`` or ``'freestanding'`` should be used for systems with single-axis trackers.