search

tpcolson / npmap-species

Automating tile generation and display for National Park Service geospatial species data.
13 stars 14 forks source link

Upgrade to Park Tile 3.0 #122

Closed tpcolson closed 3 years ago

tpcolson commented 8 years ago

@nateirwin would like to do a merge/pull on the Denver/Full code to upgrade the base map to Park Tiles 3.0 to maintain compatibility with the rest of the NPS Centennial redesign. @huang-utk do you approve?

jrecipice commented 8 years ago

@jduggan1, do you anticipate any issues if we go up to Park Tiles 3.0?

tpcolson commented 8 years ago

@nateirwin can you make a branch, make the change, and we'll test before merging?

tpcolson commented 8 years ago

Just thought of this:

PT 3.0 uses vector tile which uses campsites, etc...as you zoom in. @nateirwin one part of the user experience we wanted was for folks to be able to turn those vector features on or off regardless of zoom level. We might have to either go with a custom PT 3.0 css, or re-visit how we turn those features on and off at user GUI. Either way, let's still test.

evilkermit commented 8 years ago

Depends on what it looks like. The beta version I found was only viewable on the NPS intranet; is there somewhere else I can see it?

nateirwin commented 8 years ago

https://github.com/nationalparkservice/npmap.js/blob/master/src/preset/baselayers.json#L263

The vector overlay is optional, so using the Mapbox ids captured in the linked JSON will not bring the POIs in. Your custom POIs will continue to work.

evilkermit commented 8 years ago

Thanks! If you all want to look at it before updating the live site here's the URL: http://seelab.eecs.utk.edu/species_mapper/full/

I don't think there will be any issues, but I will defer to you all.

nateirwin commented 8 years ago

That looks great! One other minor suggestion: If you update NPMap.js to 3.0.17 (just change the URL path), you'll get a fix that addresses a few minor Safari issues.

Other than that, no recommendations... great job!

tpcolson commented 8 years ago

Was this done? Can we close this?

evilkermit commented 8 years ago

It is updated in the repo, but it looks like the version running on nps.gov hasn't been updated with this change yet.

KendraStraub commented 5 years ago

@tpcolson - can this be closed?

auxiliary commented 5 years ago

This should be checked and if needed changed to version 4.0