Support pmtile based admin layers; COUNTRY=global #1417
Conversation
|
Build succeeded and deployed at https://prism-1417.surge.sh |
gislawill
changed the title
|
Heads up @ericboucher and @wadhwamatic, this is ready for review. I believe all features but the Analysis Layers are functional. I did run into some performance issues with the loadBoundaryRelations functionality that is required for the Alerts module. We've have over 20000 admin level 2 regions in the dataset and that much data caused a slowdown in start up time. Are any deployments still using the alerts module?? |
…ithub.com/WFP-VAM/prism-app into feature/support-pmtile-based-admin-layers
| <Source | ||
| id={`source-${layer.id}`} | ||
| type="vector" | ||
| url={`pmtiles://${layer.path}`} | ||
| > |
There was a problem hiding this comment.
Note: we might get some performance benefits by rendering one source per PMTiles even when we're pulling multiple layers from the same file.
@gislawill great! Look forward to digging into this later. No, there are no active deployments that are using alerts. But it's a feature that we need functioning in the next month for upcoming deployments. |
|
@wadhwamatic sounds good. @ericboucher it'd be helpful to get your thoughts on loadBoundaryRelations (in frontend/src/components/Common/BoundaryDropdown/utils.ts) and if there's any way it could be simplified while still supporting alerts. That function alone is taking about 22 seconds to run on my computer 😅 |
Description
This addresses #1404 by adding support PMTiles (vector tiles) for admin boundaries. This branch does not yet support running analysis (zonal stats) on these PMTiles. We will add support in a following PR that involves querying a S3-hosted geoparquet.
How to test the feature:
Checklist - did you ...
Test your changes with
REACT_APP_COUNTRY=rbd yarn startREACT_APP_COUNTRY=cambodia yarn startREACT_APP_COUNTRY=mozambique yarn startScreenshot/video of feature:
https://www.loom.com/share/a7c80f04dbbb43a5b4cd68334886d088?sid=cda64d68-f6b3-4307-8b9e-3bf827a3faac