Skip to content

Fix error message modelchain.infer_temperature_model error to be more informative #1977

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
merged 7 commits into from
Mar 11, 2024
Merged

Fix error message modelchain.infer_temperature_model error to be more informative #1977

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
b582472
Update error message in infer_temperature_model
matsuobasho Feb 22, 2024
0f8da57
Updated whats new rst doc
matsuobasho Feb 22, 2024
fe1e676
Minor wording update to bug fix description about error message update
matsuobasho Feb 22, 2024
2f36d7c
Fix ValuerError message formatting in infer_temperature_model to addr…
matsuobasho Mar 3, 2024
ecfa727
Capitalize beginning of error message in modelchain.infer_temperature…
matsuobasho Mar 4, 2024
3cd6e63
Update pvlib/modelchain.py
matsuobasho Mar 5, 2024
53c5517
Update test_inconsistent_array_params and test_temperature_model_not_…
matsuobasho Mar 10, 2024
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
18 changes: 9 additions & 9 deletions pvlib/modelchain.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1128,15 +1128,15 @@ def infer_temperature_model(self):
elif {'noct', 'module_efficiency'} <= params:
return self.noct_sam_temp
else:
raise ValueError(f'could not infer temperature model from '
f'ModelChain.system. '
f'If Arrays are used to construct the PVSystem, '
f'check that all Arrays in '
f'ModelChain.system.arrays '
f'have parameers for the same temperature model. '
f'If Arrays are not used, check that the PVSystem '
f'attributes `racking_model` and `module_type` '
f'are valid.')
raise ValueError('could not infer temperature model from '
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
raise ValueError('could not infer temperature model from '
raise ValueError('Could not infer temperature model from '

Copy link
Member

Choose a reason for hiding this comment

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

@matsuobasho this capitalization change will also require the tests to be updated. See the test failures here: https://github.com/pvlib/pvlib-python/actions/runs/8146219415/job/22264244062?pr=1977#step:9:134

'ModelChain.system. '
'If Arrays are used to construct the PVSystem, '
'check that all Arrays in '
'ModelChain.system.arrays '
'have parameers for the same temperature model. '
'If Arrays are not used, check that the PVSystem '
'attributes `racking_model` and `module_type` '
'are valid.')

def _set_celltemp(self, model):
"""Set self.results.cell_temperature using the given cell
Expand Down