reprs for PVSystem and Location

Author: wholmgren

pvlib/location.py

@@ -84,9 +84,9 @@ def __init__(self, latitude, longitude, tz='UTC', altitude=0,
         # super(Location, self).__init__(**kwargs)
 
     def __repr__(self):
-        return ('{}: latitude={}, longitude={}, tz={}, altitude={}'
-                .format(self.name, self.latitude, self.longitude,
-                        self.tz, self.altitude))
+        attrs = ['name', 'latitude', 'longitude', 'altitude', 'tz']
+        return ('Location: \n  ' + '\n  '.join(
+            (attr + ': ' + str(getattr(self, attr)) for attr in attrs)))
 
 
     @classmethod

pvlib/modelchain.py

@@ -325,8 +325,7 @@ def __repr__(self):
         attrs = [
             'orientation_strategy', 'clearsky_model', 'transposition_model',
             'solar_position_method', 'airmass_model', 'dc_model', 'ac_model',
-            'aoi_model', 'spectral_model', 'temp_model', 'losses_model'
-            ]
+            'aoi_model', 'spectral_model', 'temp_model', 'losses_model']
 
         def getmcattr(self, attr):
             """needed to avoid recursion in property lookups"""

pvlib/pvsystem.py

@@ -144,10 +144,10 @@ def __init__(self,
         super(PVSystem, self).__init__(**kwargs)
 
     def __repr__(self):
-        return ('PVSystem with tilt:' + str(self.surface_tilt) +
-                ' and azimuth: ' + str(self.surface_azimuth) +
-                ' with Module: ' + str(self.module) +
-                ' and Inverter: ' + str(self.inverter))
+        attrs = ['surface_tilt', 'surface_azimuth', 'module', 'inverter',
+                 'albedo', 'racking_model']
+        return ('PVSystem: \n  ' + '\n  '.join(
+            (attr + ': ' + str(getattr(self, attr)) for attr in attrs)))
 
     def get_aoi(self, solar_zenith, solar_azimuth):
         """Get the angle of incidence on the system.