acteng / atip

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

Adjust boundary data #264

Closed dabreegster closed 11 months ago

dabreegster commented 1 year ago

A few related tasks:

The missing LAD problem: Screenshot from 2023-07-11 12-42-45 ATIP is using https://github.com/acteng/atip/blob/main/assets/authorities.geojson. This file comes from https://github.com/acteng/schema/blob/main/boundaries/lads_tas_combined.geojson, processed by https://github.com/acteng/abstreet-to-atip/tree/main/fix_boundaries. Checking another repo, I see https://github.com/acteng/boundaries/blob/main/lads.geojson, which looks complete to me. Should we use that file along with https://github.com/acteng/boundaries/blob/main/transport_authorities.geojson and ignore that old combined GJ file?

dabreegster commented 1 year ago

Revisiting https://github.com/acteng/abstreet-to-atip/issues/4#issuecomment-1419158941, checking different versions of the LPAs from OS...

BFC version (https://geoportal.statistics.gov.uk/datasets/ons::local-planning-authorities-april-2022-uk-bfc/explore?location=50.835262%2C-1.072888%2C15.49) has the same problem: Screenshot from 2023-07-11 12-54-42

BGC also won't work: https://geoportal.statistics.gov.uk/datasets/ons::local-planning-authorities-april-2022-uk-bgc-3/explore?location=50.806798%2C-0.966544%2C12.44

Or BUC: https://geoportal.statistics.gov.uk/datasets/ons::local-planning-authorities-april-2022-uk-buc-3/explore?location=50.838586%2C-1.066440%2C12.95

But BFE (https://geoportal.statistics.gov.uk/datasets/ons::local-planning-authorities-april-2022-uk-bfe-3/explore?location=50.830575%2C-1.076023%2C13.60) looks good; it's just one polygon for Portsmouth! I think we need to switch to this geometry

robinlovelace-ate commented 1 year ago

Big +1 to this. My first impression is that just being able to view the boundaries would be enough.

robinlovelace-ate commented 1 year ago

But BFE (https://geoportal.statistics.gov.uk/datasets/ons::local-planning-authorities-april-2022-uk-bfe-3/explore?location=50.830575%2C-1.076023%2C13.60) looks good; it's just one polygon for Portsmouth! I think we need to switch to this geometry

Big +1 to that and well done on finding it!

robinlovelace-ate commented 1 year ago

Checking another repo, I see https://github.com/acteng/boundaries/blob/main/lads.geojson, which looks complete to me. Should we use that file along with https://github.com/acteng/boundaries/blob/main/transport_authorities.geojson and ignore that old combined GJ file?

Yes but the file needs to be updated based on this newer LAD level data. Everything can be built from LADs.

robinlovelace-ate commented 1 year ago

https://geoportal.statistics.gov.uk/datasets/ons::local-authority-districts-may-2023-uk-bfc-2/explore

dabreegster commented 1 year ago

After we change the boundaries file, I'll regenerate all route-snapper and route-info files. It's a slightly costly process, so let's batch together any changes and confirm them before proceeding. One concern is about backwards compatibility: any authority names defined by ATIP files built up to this point are based on the old boundaries. So if the newer files change anything substantial, we should understand it.

I'm still trying to detangle all the definitions of things from the chat yesterday... today we have:

And Chris linked to LPAs; the BFE version of that fixes the multi-polygon problem. Today we don't have LPAs; do we want them as well?

I can't find MCAs defined on that geoportal site either.

robinlovelace-ate commented 1 year ago

Today we don't have LPAs; do we want them as well?

Yes I think so, a relevant boundary dataset.

dabreegster commented 1 year ago

Reading through https://geoportal.statistics.gov.uk/documents/boundary-dataset-guidance-2011-to-2021/explore and https://www.ons.gov.uk/methodology/geography/geographicalproducts/digitalboundaries closely... we want "extent of the realm" to avoid things like the Portsmouth issue. But the only variation available with this is BFE, which is full resolution. Just LADS with trimmed coordinate precision winds up 90MB. Do we have to make our own generalized extent of the realm variations?

dabreegster commented 11 months ago

If we want to revisit this and also fix #345, here's a new writeup. Ignore the rest of this issue; it was confusing other work for the browse page. This is just about the boundaries used on the start page for the sketch tool. It has:

There are a few problems:

1) LADs that happen to share the same name with a TA are skipped. I'm not sure that's appropriate; we should include all of them? Or if the boundaries are exactly the same for those cases, we could mark it in the geojson as "both" with some property. 2) In London and Portsmouth and other places, the boundary is clipped to the coastline in ways that break the route tool. We need one simple polygon covering the whole area. Previously we've tried to fix this by taking a convex hull of the input, but it makes the boundaries overlap with each other and look messy, and it doesn't work for the Thames in London. I think we want "extent of the realm" variations, but also to keep resolution low or down-scale in order to keep file sizes small.

dabreegster commented 11 months ago

Consolidating in #345