Merge pull request #257 from martinfleis/mbc

pysal · Aug 2, 2023 · 42e9663 · 42e9663
2 parents c7f180c + 17cf55a
commit 42e9663
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/esda/shape.py b/esda/shape.py
@@ -181,8 +181,8 @@ def minimum_bounding_circle_ratio(collection):
     (1963)
     """
     ga = _cast(collection)
-    mbc = shapely.minimum_bounding_circle(ga)
-    return shapely.area(ga) / shapely.area(mbc)
+    mbca = (shapely.minimum_bounding_radius(ga) ** 2) * numpy.pi
+    return shapely.area(ga) / mbca
 
 
 def radii_ratio(collection):

diff --git a/esda/tests/test_shape.py b/esda/tests/test_shape.py
@@ -79,7 +79,7 @@ def test_nmi():
 
 def test_mbc():
     observed = esda.shape.minimum_bounding_circle_ratio(shape)
-    testing.assert_allclose(observed, 0.437571, atol=ATOL)
+    testing.assert_allclose(observed, 0.434765, atol=ATOL)
 
 
 def test_reflexive_angle_ratio():