geoarrow / geoarrow-rs

GeoArrow in Rust, Python, and JavaScript (WebAssembly) with vectorized geometry operations
http://geoarrow.org/geoarrow-rs/
Apache License 2.0
259 stars 17 forks source link

nth_unchecked in coordinate lookup should not consult inner dimension #867

Open kylebarron opened 2 days ago

kylebarron commented 2 days ago

E.g. for the separated coordinates it should just assume that the nth coordinate buffer is valid and has the given length

(maybe add a debug_assert in there for the length)

kylebarron commented 2 days ago

This is already true, but we should remember to switch to using get_unchecked in the next version of the traits.