Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade Basemap #3208

Merged
merged 1 commit into from Dec 11, 2019
Merged

Upgrade Basemap #3208

merged 1 commit into from Dec 11, 2019

Conversation

@rajadain
Copy link
Member

rajadain commented Dec 11, 2019

Overview

Mapbox is increasing its Raster Tiles API pricing, causing us to use one of the newer APIs which has newly styled tiles. The tiles have been styled to generally match the previous cartography, although minor differences will persist.

This new Static Tiles API will be cheaper and more sustainably priced than the increased Raster Tiles API.

Connects #3204

Demo

2019-12-11 14 12 20

Testing Instructions

  • Check out this branch
  • Go to :8000/
    • Ensure you see the new basemap
    • Ensure it maintains the look and feel of the previous basemap
Mapbox is increasing its Raster Tiles API pricing, causing
us to use one of the newer APIs which has newly styled tiles.
The tiles have been styled to generally match the previous
cartography, although minor differences will persist.

This new Static Tiles API will be cheaper and more sustainably
priced than the increased Raster Tiles API.
@rajadain rajadain added the NSF label Dec 11, 2019
@rajadain rajadain requested a review from caseycesari Dec 11, 2019
Copy link
Member

caseycesari left a comment

Looks similar, but better!

image

image

@caseycesari caseycesari assigned rajadain and unassigned caseycesari Dec 11, 2019
@rajadain rajadain merged commit 0454eec into develop Dec 11, 2019
2 checks passed
2 checks passed
default Build finished.
Details
model-my-watershed-pull-requests Build #4116 succeeded in 10 min
Details
@rajadain rajadain deleted the tt/upgrade-basemap branch Dec 11, 2019
@rajadain

This comment has been minimized.

Copy link
Member Author

rajadain commented Dec 11, 2019

Thanks for taking a look!

@rajadain rajadain mentioned this pull request Dec 17, 2019
rajadain added a commit that referenced this pull request Jan 31, 2020
In c74f8f0, #3204, #3208, and #3213  we switched to a new
basemap, as we had been alerted by Mapbox that the API we
were previously using (the Raster Tiles API) would undergo
a price increase, and were encouraged to switch to the newer
Static Tiles API.

Unfortunately, the Static Tiles API is even more expensive
than the increased-in-price Raster Tiles API. So for now
we're switching back to the Raster Tiles API. Going forward
we may look into other basemap providers which are more
reasonably priced.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.