npct / pct-shiny

The Shiny map for Local Authorites
GNU Affero General Public License v3.0
24 stars 14 forks source link

IMD tileserver changes #884

Closed Robinlovelace closed 2 years ago

Robinlovelace commented 2 years ago

From @oobrien, many thanks for the heads-up on this, we can communicate on the issue and solution here.

Hi Robin

Many apologies, but the link to the IMD tiles basemap at https://www.pct.bike/m/ has broken again. Liverpool have unexpectedly closed the firewall on our tile server, and won’t reopen it again as it’s old. As the tileset is legacy, and there’s quite a bit of work on our side needed to modernise it, it’s not something that I can get priority for currently.

Specifically, these are the URLs:

https://cdrc-maps.liv.ac.uk/tiles/shine_labels_cdrc/[Z]/[X]/[Y].png

https://cdrc-maps.liv.ac.uk/tiles/shine_urbanmask_dark/[Z]/[X]/[Y].png

https://cdrc-maps.liv.ac.uk/tiles/imd2015_eng/[Z]/[X]/[Y].png

https://cdrc-maps.liv.ac.uk/tiles/imd2014_wal/[Z]/[X]/[Y].png

The following replacements work:

https://maps.cdrc.ac.uk/tiles/shine_labels_cdrc/[Z]/[X]/[Y].png

https://maps.cdrc.ac.uk/tiles/shine_urbanmask_dark/[Z]/[X]/[Y].png

https://maps.cdrc.ac.uk/tiles/imd2015_eng/[Z]/[X]/[Y].png

https://maps.cdrc.ac.uk/tiles/imd2014_wal/[Z]/[X]/[Y].png

Working examples:

https://maps.cdrc.ac.uk/tiles/shine_labels_cdrc/11/1001/672.png

https://maps.cdrc.ac.uk/tiles/shine_urbanmask_dark/11/1001/672.png

https://maps.cdrc.ac.uk/tiles/imd2014_wal/11/1001/672.png

The link for the IMD tiles that were used on PCT.bike has changed once again (remember back in 2017 they broke) – however looking at the current version of PCT.bike, it looks like you removed them anyway a while back. Is this correct? If so I’ll update our notes here. If not, let me know and I’ll fix it so they are back for you again.

Both the English and the Welsh IMDs have had new versions of the data published, however unfortunately we don’t’ have tiles for those, as these we put our new mapping output on Mapmaker: https://mapmaker.cdrc.ac.uk/#/index-of-multiple-deprivation - ideally you would be able to link to these instead, however they are PBF vector tiles.

Robinlovelace commented 2 years ago

As demonstrated below, the tiles no longer work:

image

I think it's just a case of updating the links to

The following replacements work:

https://maps.cdrc.ac.uk/tiles/shine_labels_cdrc/[Z]/[X]/[Y].png

https://maps.cdrc.ac.uk/tiles/shine_urbanmask_dark/[Z]/[X]/[Y].png

https://maps.cdrc.ac.uk/tiles/imd2015_eng/[Z]/[X]/[Y].png

https://maps.cdrc.ac.uk/tiles/imd2014_wal/[Z]/[X]/[Y].png

oobrien commented 2 years ago

@Robinlovelace Ta I've put a PR on this.

Robinlovelace commented 2 years ago

Update here, does not seem to be working on test server: http://npct0.vs.mythic-beasts.com/m/?r=isle-of-wight

Any ideas why @nikolai-b or anyone?

Robinlovelace commented 2 years ago

I've done some tests with the network inspector in Firefox. It seems that the IMD tile endpoints are not alive, e.g.: https://cdrc-maps.liv.ac.uk/tiles/shine_labels_cdrc/10/512/344.png

Any idea what's going on here @oobrien ?

Robinlovelace commented 2 years ago

In case it helps identify anything obvious, here's the screenshot of the debugging effort. Missing anything? image

oobrien commented 2 years ago

@Robinlovelace That's using the old URL that was blocked - my file replaces it with the new URLs, but maybe there's another file that needs to be configured.

Robinlovelace commented 2 years ago

Ah apologies, my bad.

Robinlovelace commented 2 years ago

It seems the test server is not auto updating. Will try merging to production... Heads-up @nikolai-b and @usr110.

Robinlovelace commented 2 years ago

This is now fixed :tada:

Massive thanks to Ollie on this :pray:

image