Kyle Barron

Results 1635 comments of Kyle Barron

This needs a little more thought because `GeometryArray` holds more than just one dimension. Do we take in a `GeometryArray` and then pick the dimension with data? What about an...

Indeed we should add a section in `DEVELOP.md` suggesting to use `uv run pre-commit install` to install pre-commit hooks locally. > 1. **Unify Hooks:** Extend the existing `.pre-commit-config.yaml` to include...

As I understand it, this problem is universal regardless of what tool you use to run pre-commit checks. Assuming that lefthook uses isolated environments, it won't have Python dependencies available...

I see; so lefthook doesn't use isolated environments like pre-commit does. I don't have a strong opinion here. I'd be fine with just documenting the existing state. But also open...

It's hard to say from that traceback what the error is. Is it from auth? Or are you calling HEAD on a directory?

> However, I'm a bit confused about the workaround you're referring to. Oh I had a typo in my original code. I meant `pyarrow.array(external_scalar)[0]`. So that uses the implementation of...

That's awesome! > 1. Aside from file object interactions, there's not a whole lot that's likely to break unexpectedly in the absence of the GIL, right? I don't really know....

That's really cool, and something I've been curious about for a while: https://github.com/kylebarron/arrow-js-ffi/issues/43 Looking forward to seeing that merged and seeing more examples of it in use.

There's been a lot of discussion about this before. https://github.com/georust/geo/pull/797, https://github.com/georust/geo/issues/1150. Note that the new [`geo_traits`](https://docs.rs/geo-traits/latest/geo_traits/) crate supports Z, M, and ZM dimensions. So we could have an alternate backend...

I think it would take at least a year to get enough consensus to add support for multiple dimensions to `geo-types`. I'd encourage you to explore the use of `geo-traits`...