From 8375a11f80437bd20ade33c873f073689523f429 Mon Sep 17 00:00:00 2001
From: Kyle Barron <kylebarron2@gmail.com>
Date: Tue, 26 Nov 2024 22:10:39 -0500
Subject: [PATCH 1/2] Remove infallible accessor for `Point::coord`

---
 rust/geoarrow/src/algorithm/native/map_coords.rs | 6 +++---
 rust/geoarrow/src/scalar/point/scalar.rs         | 4 ----
 2 files changed, 3 insertions(+), 7 deletions(-)

diff --git a/rust/geoarrow/src/algorithm/native/map_coords.rs b/rust/geoarrow/src/algorithm/native/map_coords.rs
index 2cca36c7..20d2b042 100644
--- a/rust/geoarrow/src/algorithm/native/map_coords.rs
+++ b/rust/geoarrow/src/algorithm/native/map_coords.rs
@@ -10,7 +10,7 @@ use crate::trait_::ArrayAccessor;
 use crate::NativeArray;
 use geo_traits::{
     CoordTrait, GeometryCollectionTrait, GeometryTrait, GeometryType, LineStringTrait,
-    MultiLineStringTrait, MultiPointTrait, MultiPolygonTrait, PolygonTrait, RectTrait,
+    MultiLineStringTrait, MultiPointTrait, MultiPolygonTrait, PointTrait, PolygonTrait, RectTrait,
 };
 
 /// Note: this will currently always create a _two-dimensional_ output array because it returns a [`geo::Coord`].
@@ -52,7 +52,7 @@ impl MapCoords for Point<'_> {
         F: Fn(&crate::scalar::Coord) -> std::result::Result<geo::Coord, E> + Sync,
         GeoArrowError: From<E>,
     {
-        Ok(geo::Point(map_op(&self.coord())?))
+        Ok(geo::Point(map_op(&self.coord().unwrap())?))
     }
 }
 
@@ -229,7 +229,7 @@ impl MapCoords for PointArray {
         );
         for maybe_geom in self.iter() {
             if let Some(geom) = maybe_geom {
-                let result = geom.coord().try_map_coords(&map_op)?;
+                let result = geom.coord().unwrap().try_map_coords(&map_op)?;
                 builder.push_coord(Some(&result));
             } else {
                 builder.push_null()
diff --git a/rust/geoarrow/src/scalar/point/scalar.rs b/rust/geoarrow/src/scalar/point/scalar.rs
index 288e5dd9..918882d7 100644
--- a/rust/geoarrow/src/scalar/point/scalar.rs
+++ b/rust/geoarrow/src/scalar/point/scalar.rs
@@ -19,10 +19,6 @@ impl<'a> Point<'a> {
         Point { coords, geom_index }
     }
 
-    pub fn coord(&self) -> Coord {
-        self.coords.value(self.geom_index)
-    }
-
     pub fn into_owned_inner(self) -> (CoordBuffer, usize) {
         let coords = self.coords.owned_slice(self.geom_index, 1);
         (coords, self.geom_index)

From f364e785be6618ef86bceae8a241b01efd1b99a7 Mon Sep 17 00:00:00 2001
From: Kyle Barron <kylebarron2@gmail.com>
Date: Tue, 26 Nov 2024 22:25:04 -0500
Subject: [PATCH 2/2] fix doctest

---
 rust/geoarrow/src/trait_.rs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/rust/geoarrow/src/trait_.rs b/rust/geoarrow/src/trait_.rs
index 99b2b097..b4528a26 100644
--- a/rust/geoarrow/src/trait_.rs
+++ b/rust/geoarrow/src/trait_.rs
@@ -540,8 +540,8 @@ pub trait ArrayAccessor<'a>: ArrayBase {
     /// let point = geo::point!(x: 1., y: 2.);
     /// let array: PointArray = (vec![point].as_slice(), Dimension::XY).into();
     /// let value = array.value(0); // geoarrow::scalar::Point
-    /// assert_eq!(value.coord().x(), 1.);
-    /// assert_eq!(value.coord().y(), 2.);
+    /// assert_eq!(value.coord().unwrap().x(), 1.);
+    /// assert_eq!(value.coord().unwrap().y(), 2.);
     /// ```
     ///
     /// # Panics