Fixing flake8 warnings, except unused folium.features.GeoJsonPopup import and _version.py newline error

Author: pmains

folium/__init__.py

@@ -22,8 +22,8 @@
     CustomIcon,
     DivIcon,
     GeoJson,
-    GeoJsonTooltip,
     GeoJsonPopup,
+    GeoJsonTooltip,
     LatLngPopup,
     RegularPolygonMarker,
     TopoJson,

folium/elements.py

@@ -1,4 +1,4 @@
-from branca.element import Figure, Element, JavascriptLink, CssLink
+from branca.element import CssLink, Element, Figure, JavascriptLink
 
 
 class JSCSSMixin(Element):

folium/features.py

@@ -3,28 +3,28 @@
 
 """
 
-import json
-import warnings
 import functools
+import json
 import operator
+import warnings
 
 from branca.colormap import LinearColormap, StepColormap
 from branca.element import (Element, Figure, JavascriptLink, MacroElement)
 from branca.utilities import color_brewer
 
 from folium.elements import JSCSSMixin
 from folium.folium import Map
-from folium.map import (FeatureGroup, Icon, Layer, Marker, Tooltip, Popup)
+from folium.map import (FeatureGroup, Icon, Layer, Marker, Popup, Tooltip)
 from folium.utilities import (
-    validate_locations,
     _parse_size,
+    camelize,
     get_bounds,
+    get_obj_in_upper_tree,
     image_to_url,
     none_max,
     none_min,
-    get_obj_in_upper_tree,
     parse_options,
-    camelize
+    validate_locations,
 )
 from folium.vector_layers import Circle, CircleMarker, PolyLine, path_options
 
@@ -583,16 +583,16 @@ def find_identifier(self):
         """
         feats = self.data['features']
         # Each feature has an 'id' field with a unique value.
-        unique_ids = set(feat.get('id', None) for feat in feats)
+        unique_ids = {feat.get('id', None) for feat in feats}
         if None not in unique_ids and len(unique_ids) == len(feats):
             return 'feature.id'
         # Each feature has a unique string or int property.
         if all(isinstance(feat.get('properties', None), dict) for feat in feats):
             for key in feats[0]['properties']:
-                unique_values = set(
+                unique_values = {
                     feat['properties'].get(key, None) for feat in feats
                     if isinstance(feat['properties'].get(key, None), (str, int))
-                )
+                }
                 if len(unique_values) == len(feats):
                     return 'feature.properties.{}'.format(key)
         # We add an 'id' field with a unique value to the data.

folium/folium.py

@@ -13,9 +13,9 @@
 from folium.raster_layers import TileLayer
 from folium.utilities import (
     _parse_size,
+    parse_options,
     temp_html_filepath,
     validate_location,
-    parse_options,
 )
 
 from jinja2 import Environment, PackageLoader, Template

folium/map.py

@@ -3,13 +3,12 @@
 
 """
 
-from collections import OrderedDict
-
 import warnings
+from collections import OrderedDict
 
 from branca.element import Element, Figure, Html, MacroElement
 
-from folium.utilities import validate_location, camelize, parse_options
+from folium.utilities import camelize, parse_options, validate_location
 
 from jinja2 import Template
 
@@ -305,6 +304,7 @@ def render(self):
             raise ValueError("{} location must be assigned when added directly to map.".format(self._name))
         super(Marker, self).render()
 
+
 class Popup(Element):
     """Create a Popup instance that can be linked to a Layer.
 

folium/plugins/__init__.py

@@ -7,7 +7,6 @@
 """
 
 from folium.plugins.antpath import AntPath
-from folium.plugins.polyline_offset import PolyLineOffset
 from folium.plugins.beautify_icon import BeautifyIcon
 from folium.plugins.boat_marker import BoatMarker
 from folium.plugins.draw import Draw
@@ -25,6 +24,7 @@
 from folium.plugins.minimap import MiniMap
 from folium.plugins.mouse_position import MousePosition
 from folium.plugins.pattern import CirclePattern, StripePattern
+from folium.plugins.polyline_offset import PolyLineOffset
 from folium.plugins.polyline_text_path import PolyLineTextPath
 from folium.plugins.scroll_zoom_toggler import ScrollZoomToggler
 from folium.plugins.search import Search

folium/plugins/antpath.py

@@ -1,5 +1,5 @@
 from folium.elements import JSCSSMixin
-from folium.vector_layers import path_options, BaseMultiLocation
+from folium.vector_layers import BaseMultiLocation, path_options
 
 from jinja2 import Template
 

folium/plugins/dual_map.py

@@ -1,4 +1,4 @@
-from branca.element import MacroElement, Figure
+from branca.element import Figure, MacroElement
 
 from folium.elements import JSCSSMixin
 from folium.folium import Map

folium/plugins/heat_map.py

@@ -3,10 +3,10 @@
 from folium.elements import JSCSSMixin
 from folium.map import Layer
 from folium.utilities import (
+    if_pandas_df_convert_to_numpy,
     none_max,
     none_min,
     parse_options,
-    if_pandas_df_convert_to_numpy,
     validate_location,
 )
 

folium/plugins/marker_cluster.py

@@ -1,6 +1,6 @@
 from folium.elements import JSCSSMixin
 from folium.map import Layer, Marker
-from folium.utilities import validate_locations, parse_options
+from folium.utilities import parse_options, validate_locations
 
 from jinja2 import Template