Skip to content

Fix PVSystem module_type docstring default #2713

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.

Sign up for GitHub

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

Merged
cwhanse merged 5 commits into from
Mar 17, 2026
+4 −1
Merged

Fix PVSystem module_type docstring default #2713

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
944ec21
Fix PVSystem module_type docstring default
r0hansaxena Mar 7, 2026
277fbb4
Fix docstring references in v0.15.1 release notes
r0hansaxena Mar 7, 2026
1b32f0c
Update docs/sphinx/source/whatsnew/v0.15.1.rst
r0hansaxena Mar 9, 2026
ed22c4d
Update pvlib/pvsystem.py
r0hansaxena Mar 9, 2026
c266cd4
Apply suggestion from @RDaxini
cwhanse Mar 17, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions docs/sphinx/source/whatsnew/v0.15.1.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,9 @@ Documentation
* Add AI checkbox to PR template, and auto-generate a comment on PRs
from first-time contributors regarding AI and contributing guidelines.
(:issue:`2617`, :pull:`2624`)
* Fix :py:class:`pvlib.pvsystem.PVSystem` docstring incorrectly stating
``module_type`` defaults to ``'glass_polymer'`` instead of ``None``.
(:issue:`2634`, :pull:`2713`)
* Fix a broken docstring reference to ``grounddiffuse`` in
:py:func:`pvlib.irradiance.poa_components`
(:issue:`2089`, :pull:`2708`)
Expand Down
2 changes: 1 addition & 1 deletion pvlib/pvsystem.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -140,7 +140,7 @@ class PVSystem:
module : string, optional
The model name of the modules.

module_type : string, default 'glass_polymer'
module_type : string, default None
Describes the module's construction. Valid strings are 'glass_polymer'
and 'glass_glass'. Used for cell and module temperature calculations.

Expand Down