Kyle Barron

Results 1059 comments of Kyle Barron

See also https://github.com/PyO3/pyo3/discussions/2340#discussioncomment-2646525

good to know about that serde option but we should be checking if all fields are null and not even writing the metadata key in that case

This is solved now. Helped by https://github.com/geoarrow/geoarrow-rs/pull/719 but solved separately before that I think.

It should maybe also be possible to pass in a list of slices? ``` geo_table[0:3, 3:6] ``` though I'm not overly excited about that syntax because it feels too much...

I don't like this API. It should be clear with methods. Regardless, these general rechunk/take methods are implemented in arro3 now.

Also need to check `#[allow(clippy::large_enum_variant)]` on `OwnedGeometry`

ref: https://pyodide.org/en/stable/development/building-and-testing-packages.html, https://blog.pyodide.org/posts/rust-pyo3-support-in-pyodide/, https://github.com/pola-rs/polars/issues/3672#issuecomment-1188613095

To simplify development overhead and maintenance, we're moving to only top-level functions anyways, so hopefully pyodide should be on the roadmap

ref https://github.com/cjriley9/gpkg-rs