Skip to content

expand PVSystem repr #1027

@wholmgren

Description

@wholmgren

The PVSystem repr is

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.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions