Skip to content

Commit 920f18e

Browse files
authored
Merge pull request #2771 from galenlynch/fix/remove-six
Remove Python 2 compatibility shims
2 parents eb28ce1 + e03133e commit 920f18e

72 files changed

Lines changed: 119 additions & 191 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

allensdk/api/queries/brain_observatory_api.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,6 @@
3838
from pathlib import Path
3939

4040
import pandas as pd
41-
from six import string_types
4241

4342
import allensdk.brain_observatory.stimulus_info as stimulus_info
4443

@@ -785,7 +784,7 @@ def dataframe_query_string(self, filters):
785784
"""
786785

787786
def _quote_string(v):
788-
if isinstance(v, string_types):
787+
if isinstance(v, str):
789788
return "'%s'" % (v)
790789
else:
791790
return str(v)

allensdk/api/queries/image_download_api.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,6 @@
3535
#
3636
from .rma_template import RmaTemplate
3737
from allensdk.api.warehouse_cache.cache import cacheable
38-
from six import string_types
39-
4038

4139
class ImageDownloadApi(RmaTemplate):
4240
'''HTTP Client to download whole or partial two-dimensional images from the Allen Institute
@@ -384,7 +382,7 @@ def download_image(self,
384382
colormap_filter = kwargs.get('colormap', None)
385383

386384
if colormap_filter is not None:
387-
if isinstance(colormap_filter, string_types):
385+
if isinstance(colormap_filter, str):
388386
params.append('colormap=%s' % (colormap_filter))
389387
else:
390388
lower_threshold = colormap_filter[0]

allensdk/api/queries/mouse_connectivity_api.py

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,6 @@
3838
from allensdk.api.warehouse_cache.cache import cacheable, Cache
3939
import numpy as np
4040
import nrrd
41-
import six
42-
4341

4442
class MouseConnectivityApi(ReferenceSpaceApi, GridDataApi):
4543
'''
@@ -282,7 +280,7 @@ def experiment_source_search(self, **kwargs):
282280
`service::mouse_connectivity_injection_structure <http://help.brain-map.org/display/api/Connected+Services+and+Pipes#ConnectedServicesandPipes-service%3A%3Amouseconnectivityinjectionstructure>`_.
283281
284282
'''
285-
tuples = [(k, v) for k, v in six.iteritems(kwargs)]
283+
tuples = [(k, v) for k, v in kwargs.items()]
286284
return self.service_query('mouse_connectivity_injection_structure', parameters=tuples)
287285

288286
def experiment_spatial_search(self, **kwargs):
@@ -318,7 +316,7 @@ def experiment_spatial_search(self, **kwargs):
318316
319317
'''
320318

321-
tuples = [(k, v) for k, v in six.iteritems(kwargs)]
319+
tuples = [(k, v) for k, v in kwargs.items()]
322320
return self.service_query('mouse_connectivity_target_spatial', parameters=tuples)
323321

324322
def experiment_injection_coordinate_search(self, **kwargs):
@@ -349,7 +347,7 @@ def experiment_injection_coordinate_search(self, **kwargs):
349347
`service::mouse_connectivity_injection_coordinate <http://help.brain-map.org/display/api/Connected+Services+and+Pipes#ConnectedServicesandPipes-service%3A%3Amouseconnectivityinjectioncoordinate>`_.
350348
351349
'''
352-
tuples = [(k, v) for k, v in six.iteritems(kwargs)]
350+
tuples = [(k, v) for k, v in kwargs.items()]
353351
return self.service_query('mouse_connectivity_injection_coordinate', parameters=tuples)
354352

355353
def experiment_correlation_search(self, **kwargs):
@@ -384,7 +382,7 @@ def experiment_correlation_search(self, **kwargs):
384382
`service::mouse_connectivity_correlation <http://help.brain-map.org/display/api/Connected+Services+and+Pipes#ConnectedServicesandPipes-service%3A%3Amouseconnectivitycorrelation>`_.
385383
386384
'''
387-
tuples = sorted(six.iteritems(kwargs))
385+
tuples = sorted(kwargs.items())
388386
return self.service_query('mouse_connectivity_correlation',
389387
parameters=tuples)
390388

allensdk/api/queries/reference_space_api.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,8 +39,6 @@
3939
import allensdk.core.sitk_utilities as sitk_utilities
4040
import numpy as np
4141
import nrrd
42-
import six
43-
4442

4543
class ReferenceSpaceApi(RmaApi):
4644

allensdk/brain_observatory/circle_plots.py

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -36,10 +36,6 @@
3636

3737
import math
3838

39-
try:
40-
xrange
41-
except:
42-
from past.builtins import xrange
4339

4440
import numpy as np
4541
import pandas as pd
@@ -50,7 +46,6 @@
5046
import matplotlib.transforms as mxfms
5147
import matplotlib.colors as mcolors
5248
import skimage.transform
53-
from six import iteritems
5449

5550

5651
DEFAULT_COLOR_MAP = LinearSegmentedColormap.from_list('default', [[.7,0,.7,0.0],[.7,0,0,1]])
@@ -178,7 +173,7 @@ def build_hex_pack(n):
178173
mag = 1
179174
sign = 1
180175

181-
for j in xrange(N-1):
176+
for j in range(N-1):
182177
x += v[0] * mag * sign
183178
y += v[1] * mag * sign
184179
mag += 1
@@ -277,7 +272,7 @@ def make_pincushion_plot(data, trials, on, nrows, ncols, clim=None, color_map=No
277272
color_map = LSN_ON_COLOR_MAP if on else LSN_OFF_COLOR_MAP
278273

279274
ax = plt.gca()
280-
for (col,row,on_state), sweeps in iteritems(trials):
275+
for (col,row,on_state), sweeps in trials.items():
281276
if on_state != on:
282277
continue
283278

@@ -477,7 +472,7 @@ def plot(self, category_data,
477472
df = pd.DataFrame({ 'category': category_data })
478473
gb = df.groupby(['category'])
479474

480-
for category, trials in iteritems(gb.groups):
475+
for category, trials in gb.groups.items():
481476
idx = self.cat_idx_map[category]
482477
order = np.argsort(data[trials])[::-1]
483478
trial_order = np.array(trials)[order]
@@ -714,7 +709,7 @@ def plot(self,
714709

715710
gb = df.groupby(['group', 'angle', 'r'])
716711

717-
for (group, angle, r), trials in iteritems(gb.groups):
712+
for (group, angle, r), trials in gb.groups.items():
718713
responses = np.sort(data[trials])[::-1]
719714

720715
circles = spiral_trials_polar(self.r_radius_map[r],

allensdk/brain_observatory/ecephys/stimulus_analysis/drifting_gratings.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import numpy as np
22
import pandas as pd
3-
from six import string_types
43
import scipy.ndimage as ndi
54
import scipy.stats as st
65
from scipy.signal import welch

allensdk/brain_observatory/ecephys/stimulus_analysis/flashes.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import numpy as np
22
import pandas as pd
3-
from six import string_types
3+
44
import scipy.ndimage as ndi
55
import scipy.stats as st
66
from scipy.optimize import curve_fit

allensdk/brain_observatory/ecephys/stimulus_analysis/natural_movies.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import numpy as np
22
import pandas as pd
3-
from six import string_types
3+
44
import scipy.ndimage as ndi
55
import scipy.stats as st
66
from scipy.optimize import curve_fit

allensdk/brain_observatory/ecephys/stimulus_analysis/static_gratings.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
from six import string_types
21
import numpy as np
32
import pandas as pd
43
from scipy.optimize import curve_fit

allensdk/brain_observatory/ecephys/stimulus_analysis/stimulus_analysis.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
from six import string_types
21
import numpy as np
32
import pandas as pd
43
import scipy.stats as st
@@ -25,7 +24,7 @@ def __init__(self, ecephys_session, trial_duration=None, **kwargs):
2524
# TODO: Create a set of a class methods.
2625
if isinstance(ecephys_session, EcephysSession):
2726
self._ecephys_session = ecephys_session
28-
elif isinstance(ecephys_session, string_types):
27+
elif isinstance(ecephys_session, str):
2928
nwb_version = kwargs.get('nwb_version', 2)
3029
self._ecephys_session = EcephysSession.from_nwb_path(
3130
path=ecephys_session, nwb_version=nwb_version)
@@ -172,7 +171,7 @@ def stim_table(self):
172171

173172
self._stim_table = self.ecephys_session.get_stimulus_table(
174173
[self._stimulus_key] if isinstance(self._stimulus_key,
175-
string_types) else
174+
str) else
176175
self._stimulus_key
177176
)
178177

0 commit comments

Comments
 (0)