-
Notifications
You must be signed in to change notification settings - Fork 34
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
[Feature Request]: make use of pmtiles for admin boundaries #1404
Comments
Heads up @wadhwamatic and @valpesendorfer, we're planning to approach this in the following way:
|
Thanks @gislawill. The plan sounds good. One thing I'm thinking about is 'interactions with the admin boundaries directly will need to be updated'. My hope is that basic map interaction will 'just work'. I would expect that we'll have more challenges when we run zonal stats through PRISM's analysis features. I don't know enough about PMTiles for this type of use. Might need to consider alternatives in that scenario, while still keeping boundaries in sync between approaches. cc @ericboucher |
@wadhwamatic, you're 100% right — that "zonal stats through PRISM's analysis features" is a really great callout. One approach: Within PRISM's server, we could decode the PMTiles into a geojson that we can then use for zonal stats (as opposed to sending the geojson with the request, as we do now). It's moderately complicated functionality: Decoding only part of a global PMTiles file is definitely do-able but it'll be important write it correctly and maintain it well to avoid loading unnecessary data. Alternatively, if the PMTiles are directly associated with a geoparquet in S3 (the direction we're moving in), our prism server can selectively query the relevant features and easily load just the necessary data. That query would be harder to screw up and easier to maintain. Plus, this better maintains a "one source of truth" approach. I don't think this needs to be a blocker on this ticket it's is a good time to revisit if the PRISM api should run zonal stats or if we should move it to a shared service (@ericboucher and @valpesendorfer) |
Sorry for the late follow up @gislawill. I think the geoparquet approach is more inline with where we'd want to go with this but let's hear from @valpesendorfer with his thoughts |
Agreed, I don't think |
Provide a clear and concise description of what you want to happen.
We are unable to load larger admin boundary files. Rather than attempt to load large geojson files which can contain more data than necessary, we should try to only load relevant data based on the users' zoom level and in-view bbox. We've determined that pmtiles will provide a good solution for this.
It's critical that we continue to maintain support for admin boundaries in the form of geojson, while adding support for admin boundaries using pmtiles
The change would touch many areas and that the following features still work using either admin boundary approach (json or pmtiles):
Is there anything else you can add about the proposal? You might want to link to related issues here, if you haven't already.
No response
The text was updated successfully, but these errors were encountered: