Skip to content

Commit 07296b0

Browse files
authored
Merge pull request #252 from PyPSA/gpd-version-check
use packaging.version instead of deprecated distutils.StrictVersion
2 parents b3dac6d + 2791e7b commit 07296b0

File tree

2 files changed

+4
-7
lines changed

2 files changed

+4
-7
lines changed

scripts/build_industrial_distribution_key.py

+2-4
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,7 @@
55
import geopandas as gpd
66

77
from itertools import product
8-
from distutils.version import StrictVersion
9-
10-
gpd_version = StrictVersion(gpd.__version__)
8+
from packaging.version import Version, parse
119

1210

1311
def locate_missing_industrial_sites(df):
@@ -73,7 +71,7 @@ def prepare_hotmaps_database(regions):
7371

7472
gdf = gpd.GeoDataFrame(df, geometry='coordinates', crs="EPSG:4326")
7573

76-
kws = dict(op="within") if gpd_version < '0.10' else dict(predicate="within")
74+
kws = dict(op="within") if parse(gpd.__version__) < Version('0.10') else dict(predicate="within")
7775
gdf = gpd.sjoin(gdf, regions, how="inner", **kws)
7876

7977
gdf.rename(columns={"index_right": "bus"}, inplace=True)

scripts/cluster_gas_network.py

+2-3
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,8 @@
88

99
from shapely import wkt
1010
from pypsa.geo import haversine_pts
11-
from distutils.version import StrictVersion
11+
from packaging.version import Version, parse
1212

13-
gpd_version = StrictVersion(gpd.__version__)
1413

1514
def concat_gdf(gdf_list, crs='EPSG:4326'):
1615
"""Concatenate multiple geopandas dataframes with common coordinate reference system (crs)."""
@@ -34,7 +33,7 @@ def build_clustered_gas_network(df, bus_regions, length_factor=1.25):
3433

3534
gdf = gpd.GeoDataFrame(geometry=df[f"point{i}"], crs="EPSG:4326")
3635

37-
kws = dict(op="within") if gpd_version < '0.10' else dict(predicate="within")
36+
kws = dict(op="within") if parse(gpd.__version__) < Version('0.10') else dict(predicate="within")
3837
bus_mapping = gpd.sjoin(gdf, bus_regions, how="left", **kws).index_right
3938
bus_mapping = bus_mapping.groupby(bus_mapping.index).first()
4039

0 commit comments

Comments
 (0)