PEP8ing, fearing the 👻

Author: Martin Journois

folium/features.py

@@ -637,7 +637,7 @@ def _get_self_bounds(self):
         """Computes the bounds of the object itself (not including it's children)
         in the form [[lat_min, lon_min], [lat_max, lon_max]]
         """
-        bounds = [[None,None],[None,None]]
+        bounds = [[None, None], [None, None]]
         for point in iter_points(self.data):
             bounds = [
                 [
@@ -701,7 +701,7 @@ def _get_self_bounds(self):
         """Computes the bounds of the object itself (not including it's children)
         in the form [[lat_min, lon_min], [lat_max, lon_max]]
         """
-        bounds = [[None,None],[None,None]]
+        bounds = [[None, None], [None, None]]
         for point in iter_points(self.data):
             bounds = [
                 [

folium/map.py

@@ -421,7 +421,7 @@ def _get_self_bounds(self):
         """Computes the bounds of the object itself (not including it's children)
         in the form [[lat_min, lon_min], [lat_max, lon_max]]
         """
-        return [[self.location[0],self.location[1]],[self.location[0],self.location[1]]]
+        return [[self.location[0], self.location[1]], [self.location[0], self.location[1]]]
 
 class Popup(Element):
     def __init__(self, html=None, max_width=300):

folium/utilities.py

@@ -449,9 +449,9 @@ def iter_points(x):
     """Iterates over a list representing a feature, and returns a list of points,
     whatever the shape of the array (Point, MultiPolyline, etc).
     """
-    if isinstance(x,list):
+    if isinstance(x, list):
         if len(x):
-            if isinstance(x[0],list):
+            if isinstance(x[0], list):
                 out = []
                 for y in x:
                     out += iter_points(y)
@@ -461,4 +461,4 @@ def iter_points(x):
         else:
             return []
     else:
-        raise ValueError('List type expected {!r}'.format(x))
+        raise ValueError('List type expected. Got {!r}.'.format(x))