Merge pull request #1231 from georust/mkirk/unify-densify

unify Densify trait (adds support for Geodesic, Rhumb)
georust · Oct 22, 2024 · cf37789 · cf37789
2 parents f683c7a + 44dd231
commit cf37789
Show file tree

Hide file tree

Showing 11 changed files with 513 additions and 343 deletions.
diff --git a/geo/CHANGES.md b/geo/CHANGES.md
@@ -53,6 +53,20 @@
   line_string.length::<Haversine>();
   ```
   * <https://github.com/georust/geo/pull/1228>
+* Deprecated `DensifyHaversine`
+* BREAKING: `Densify::densify` is no longer strictly Euclidean, and now accepts a generic line measure parameter.
+   ```
+  // Before
+  line_string.densify();
+  line_string.densify_haversine();
+  // After
+  line_string.densify::<Euclidean>();
+  line_string.densify::<Haversine>();
+
+  // Additional measures are now supported
+  line_string.densify::<Geodesic>();
+  line_string.densify::<Rhumb>();
+   ```
 * Change IntersectionMatrix::is_equal_topo to now consider empty geometries as equal.
   * <https://github.com/georust/geo/pull/1223>
 * Fix `(LINESTRING EMPTY).contains(LINESTRING EMPTY)` and `(MULTIPOLYGON EMPTY).contains(MULTIPOINT EMPTY)` which previously

diff --git a/geo/src/algorithm/densify.rs b/geo/src/algorithm/densify.rs