Skip to content
This repository has been archived by the owner on Oct 5, 2018. It is now read-only.

map_type fixes. mapnik->standard, hide osmarender #11

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

harry-wood
Copy link

@harry-wood harry-wood commented Jan 15, 2017

The map_type 'mapnik' value is badly named. Create a new alias 'standard'
which is now the new default (while the old 'mapnik' value continues to work as
a deprecated value)

'osmarender' has been (badly) mapped to MapQuest open tiles for a while now, so
broken because these were discontinued (as was the original 'osmarender' layer)
Fix maptype=osmarender so it works, by mapping to osm standard, but obviously
this value is also deprecated.

Changes to the intro page to reflect this.

Fixes https://github.com/dfacts/staticmaplite/issues/3

Deployment note: If there's a large cache built up under ../cache/maps/mapnik that will no longer get used, so you may want to rename that directory as ../cache/maps/standard (or maybe you just delete the cache periodically anyway?)

Harry Wood added 2 commits January 15, 2017 01:39
The map_type 'mapnik' value is badly named. Create a new alias 'standard'
which is now the new default (which the old 'mapnik' value continues to work as
a deprecated value)

'osmarender' has been (badly) mapped to MapQuest open tiles for a while now, so
broken because these were discontinued (as was the original 'osmarender' layer)
Fix maptype=osmarender so it works, by mapping to osm standard, but obviously
this value is also deprecated.

Changes to the intro page to reflect this.

Fixes https://github.com/dfacts/staticmaplite/issues/3
Define 'mapnik' and 'osmrender' explcitly as aliases of 'standard' rather than
just setting to the same tile URL. That approach would mean the aliases will
have their own cache directory (which is unnecessary). Now they will all use
the 'standard' cache directory.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant