Skip to content

expand PVSystem repr #1027

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.

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

Closed
wholmgren opened this issue Aug 24, 2020 · 0 comments · Fixed by #1031
Closed

expand PVSystem repr #1027

wholmgren opened this issue Aug 24, 2020 · 0 comments · Fixed by #1031
Milestone

Comments

@wholmgren
Copy link
Member

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant