-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Description
The PVSystem repr is
pvlib-python/pvlib/pvsystem.py
Lines 239 to 243 in 27872b8
def __repr__(self): | |
attrs = ['name', 'surface_tilt', 'surface_azimuth', 'module', | |
'inverter', 'albedo', 'racking_model'] | |
return ('PVSystem: \n ' + '\n '.join( | |
('{}: {}'.format(attr, getattr(self, attr)) for attr in attrs))) |
The main issue that I have is that the repr doesn't give me enough information about the temperature model settings. It's relatively important because temperature_model_params
(not printed) may be inferred from module_type
(not printed) and racking_model
(printed). So I'd like to add both temperature_model_params
and module_type
.
We also don't include module_parameters
, inverter_parameters
, and losses_parameters
in the repr. If I recall correctly, we decided against including these because they can be relatively long. I still think that's reasonable. We could add something like if len(module_parameters): 'Set. See PVSystem.module_parameters'; else: {}
, but I don't know if that's worth the effort.