Joris Van den Bossche

Results 844 comments of Joris Van den Bossche

Yes, then it's best to check each partitions bounds (which can still be done with my snippet above if you want (or parse each file's metadata yourself), although you will...

The build passed, but because of the message in the build output: ``` -- Performing post-build validation The following files were built for an incorrect architecture: /usr/local/share/vcpkg/packages/gdal_arm64-osx-dynamic-release/lib/libgdal.30.dylib Expected arm64, but...

And as a starter, `delocate` didn't include the dylibs in the wheel for the arm64 case ..

> For instance, sqlite does not list `arm64-osx`: The same is true for gdal, I think that's because "arm64-osx" is not yet an "official" triplet, but only a "community" triplet...

@brendan-ward @martinfleis there are arm64 MacOS wheels to test! -> https://github.com/geopandas/pyogrio/suites/8673510505/artifacts/390667508

So indeed, just update this PR with the latest changes from main and https://github.com/geopandas/pyogrio/pull/161 (to update the GDAL versions / vcpkg baseline so GDAL is using CMake), _seems_ to have...

That is great to hear! So I am planning to merge the other PR then soon (https://github.com/geopandas/pyogrio/pull/161), so we can update and merge this one as well.

Cool, thanks a lot for testing!

For the case of x / y columns, we have an example in the docs like: ``` import dask.dataframe as dd import dask_geopandas ddf = dd.read_csv('...') ddf = ddf.set_geometry( dask_geopandas.points_from_xy(ddf,...

@aprilmay Ah, that's a good point, thanks for noticing this! That's certainly something we will need to fix / workaround in some way. I opened an issue about supporting more...