Skip to content

Commit e5ba5ff

Browse files
author
Rob Story
committed
STY: PEP8 cleanups and update gitignore
1 parent 6fd5be4 commit e5ba5ff

File tree

2 files changed

+32
-20
lines changed

2 files changed

+32
-20
lines changed

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,3 +43,4 @@ nosetests.xml
4343

4444
#Virtualenv
4545
ENV
46+
.env

folium/folium.py

Lines changed: 31 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -10,13 +10,16 @@
1010
from __future__ import absolute_import
1111
from __future__ import print_function
1212
from __future__ import division
13+
1314
import codecs
14-
import json
1515
import functools
16+
import json
1617
import os
18+
from uuid import uuid4
19+
1720
from jinja2 import Environment, PackageLoader
1821
from pkg_resources import resource_string, resource_filename
19-
from uuid import uuid4
22+
2023
from folium import utilities
2124
from folium.six import text_type, binary_type, iteritems
2225

@@ -173,7 +176,8 @@ def __init__(self, location=None, width=960, height=500,
173176
if isinstance(attr, binary_type):
174177
attr = text_type(attr, 'utf8')
175178
self.template_vars['attr'] = attr
176-
self.tile_types.update({'Custom': {'template': tiles, 'attr': attr}})
179+
self.tile_types.update({'Custom': {'template': tiles,
180+
'attr': attr}})
177181

178182
self.added_layers = []
179183
self.template_vars.setdefault('wms_layers', [])
@@ -205,7 +209,8 @@ def add_tile_layer(self, tile_name=None,tile_url=None,active=False):
205209

206210

207211
@iter_obj('simple')
208-
def add_wms_layer(self, wms_name=None,wms_url=None,wms_format=None,wms_layers=None,wms_transparent=True):
212+
def add_wms_layer(self, wms_name=None, wms_url=None, wms_format=None,
213+
wms_layers=None, wms_transparent=True):
209214
'''adds a simple tile layer
210215
Parameters
211216
----------
@@ -218,12 +223,12 @@ def add_wms_layer(self, wms_name=None,wms_url=None,wms_format=None,wms_layers=No
218223
wms_name = wms_name.replace (" ", "_")
219224
wms_temp = self.env.get_template('wms_layer.js')
220225

221-
wms = wms_temp.render({'wms_name': wms_name,
222-
'wms_url': wms_url,
223-
'wms_format': wms_format,
224-
'wms_layer_names':wms_layers,
225-
'wms_transparent':str(wms_transparent).lower()
226-
})
226+
wms = wms_temp.render({
227+
'wms_name': wms_name,
228+
'wms_url': wms_url,
229+
'wms_format': wms_format,
230+
'wms_layer_names':wms_layers,
231+
'wms_transparent':str(wms_transparent).lower()})
227232

228233
self.template_vars.setdefault('wms_layers', []).append((wms))
229234

@@ -237,7 +242,7 @@ def add_layers_to_map(self):
237242
layers_temp = self.env.get_template('add_layers.js')
238243

239244
data_string = ''
240-
for i, layer in enumerate(self.added_layers):
245+
for i, layer in enumerate(self.added_layers):
241246
name = layer.keys()[0]
242247
data_string+='\"'
243248
data_string+=name
@@ -247,13 +252,15 @@ def add_layers_to_map(self):
247252
if i < len(self.added_layers)-1:
248253
data_string+=",\n"
249254
else:
250-
data_string+="\n"
255+
data_string+="\n"
251256

252257
data_layers = layers_temp.render({'layers': data_string})
253258
self.template_vars.setdefault('data_layers', []).append((data_string))
254259

255260
@iter_obj('simple')
256-
def simple_marker(self, location=None, popup='Pop Text', popup_on=True,marker_color='blue',marker_icon='info-sign',clustered_marker=False,icon_angle=0):
261+
def simple_marker(self, location=None, popup='Pop Text', popup_on=True,
262+
marker_color='blue', marker_icon='info-sign',
263+
clustered_marker=False, icon_angle=0):
257264
'''Create a simple stock Leaflet marker on the map, with optional
258265
popup text or Vincent visualization.
259266
@@ -271,7 +278,8 @@ def simple_marker(self, location=None, popup='Pop Text', popup_on=True,marker_co
271278
marker_icon
272279
icon from (http://getbootstrap.com/components/) you want on the marker
273280
clustered_marker
274-
boolean of whether or not you want the marker clustered with other markers
281+
boolean of whether or not you want the marker clustered with
282+
other markers
275283
276284
Returns
277285
-------
@@ -299,7 +307,7 @@ def simple_marker(self, location=None, popup='Pop Text', popup_on=True,marker_co
299307

300308
#Get marker and popup
301309
marker = mark_temp.render({'marker': 'marker_' + str(count),
302-
'lat': location[0],
310+
'lat': location[0],
303311
'lon': location[1],
304312
'icon': add_line
305313
})
@@ -308,7 +316,7 @@ def simple_marker(self, location=None, popup='Pop Text', popup_on=True,marker_co
308316
count=count,
309317
popup_on=popup_on)
310318

311-
319+
312320

313321
if clustered_marker:
314322
add_mark = 'clusteredmarkers.addLayer(marker_{0})'.format(count)
@@ -661,16 +669,19 @@ def geo_json(self, geo_path=None, geo_str=None, data_out='data.json',
661669
662670
Example
663671
-------
664-
>>>map.geo_json(geo_path='us-states.json', line_color='blue', line_weight=3)
665-
>>>map.geo_json(geo_path='geo.json', data=df, columns=['Data 1', 'Data 2'],
672+
>>>map.geo_json(geo_path='us-states.json', line_color='blue',
673+
line_weight=3)
674+
>>>map.geo_json(geo_path='geo.json', data=df,
675+
columns=['Data 1', 'Data 2'],
666676
key_on='feature.properties.myvalue', fill_color='PuBu',
667677
threshold_scale=[0, 20, 30, 40, 50, 60])
668678
>>>map.geo_json(geo_path='countries.json', topojson='objects.countries')
669679
'''
670680

671681
if reset:
672-
reset_vars = ['json_paths', 'func_vars', 'color_scales', 'geo_styles',
673-
'gjson_layers', 'map_legends', 'topo_convert']
682+
reset_vars = ['json_paths', 'func_vars', 'color_scales',
683+
'geo_styles', 'gjson_layers', 'map_legends',
684+
'topo_convert']
674685
for var in reset_vars:
675686
self.template_vars.update({var: []})
676687
self.mark_cnt['geojson'] = 1

0 commit comments

Comments
 (0)