Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: support st_read_meta (duckdb geospatial extension) #10377

Open
1 task done
cboettig opened this issue Oct 26, 2024 · 2 comments
Open
1 task done

feat: support st_read_meta (duckdb geospatial extension) #10377

cboettig opened this issue Oct 26, 2024 · 2 comments
Labels
duckdb The DuckDB backend feature Features or general enhancements

Comments

@cboettig
Copy link

Is your feature request related to a problem?

knowing the CRS (coordinate reference system) of our data is often a crucial bit of information. This information is already embedded inside all the spatial formats we read with read_geo() method in ibis. If we read these data with geopandas directly, the CRS is preserved and attached to the geopandas object as expected. But if we read these data via ibis / duckdb spatial (e.g. because perhaps the file is really large and we want to filter it first), .execute() gives us a geopandas object but with empty CRS attribute.

What is the motivation behind your request?

No response

Describe the solution you'd like

The duckdb spatial extension provides st_read_meta() for this and related metadata. Ideally, ibis read_geo could attempt to call st_read_meta() in some way as well, such that when the user materializes the data with execute(), it comes with .crs already correctly attached.

A simpler a probably necessary solution anyway would just be to have ibis bindings to the st_read_meta() method in duckdb (which can access additional metadata elements as well).

What version of ibis are you running?

9.5

What backend(s) are you using, if any?

duckdb + spatial extension

Code of Conduct

  • I agree to follow this project's Code of Conduct
@cboettig cboettig added the feature Features or general enhancements label Oct 26, 2024
@cboettig
Copy link
Author

Maybe an aside, but there seems to be an issue (at least in my testing) in trying to call st_read_meta when using the virtual filesystem interface duckdb/duckdb_spatial#269 (comment)

@cpcloud
Copy link
Member

cpcloud commented Nov 2, 2024

Thanks for the issue, this seems like a nice addition to the DuckDB backend!

@cpcloud cpcloud added the duckdb The DuckDB backend label Nov 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duckdb The DuckDB backend feature Features or general enhancements
Projects
Status: backlog
Development

No branches or pull requests

2 participants