acteng / atip

Active Travel Infrastructure Platform
https://acteng.github.io/atip/
Apache License 2.0
22 stars 4 forks source link

Use gzipped scheme data #494

Closed dabreegster closed 6 months ago

dabreegster commented 6 months ago

As fate would have it, we're forced to look at #493 immediately, because we're now over the 32MB app engine response limit. If we gzip that file, it's down to about 7MB and loads much faster on slow connections anyway!

Do gzip --keep all_schemes_output.geojson after running the combine script, and upload that to the same private_layers/v1/ folder. Please test in the dev environment first, deploy this PR manually (backend/README has instructions), make sure everthing works end-to-end, then push to test env

dabreegster commented 6 months ago

I'll be offline until about 2, please do merge as soon as dev env is ready for it

On Wed, Apr 3, 2024, 10:52 AM Pete-Y-CS @.***> wrote:

@.**** approved this pull request.

— Reply to this email directly, view it on GitHub https://github.com/acteng/atip/pull/494#pullrequestreview-1976231604, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAMWLF4WJBIBDWDYPMBWAA3Y3PGNTAVCNFSM6AAAAABFU6II6CVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTSNZWGIZTCNRQGQ . You are receiving this because you authored the thread.Message ID: @.***>