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

Shippensburg Land Projections: Update Layers #3320

Merged
merged 1 commit into from
Jun 1, 2020

Conversation

rajadain
Copy link
Member

@rajadain rajadain commented May 21, 2020

Overview

Uses 4 of the new Shippensburg layers ingested in #3305 in the Layer Selector. Removes old layers.

Connects #3306

Demo

Screen Shot 2020-05-29 at 11 55 55-fullpage

(the layer selector's height was increased for the purposes of this screenshot)

Notes

There were a total of 6 layers of future land projections ingested:

❯ aws s3 ls tiles.us-east-1.azavea.com/ | grep shippensburg-2100
    PRE shippensburg-2100-centers-30m/
    PRE shippensburg-2100-centers-np-30m/
    PRE shippensburg-2100-centers-osi-30m/
    PRE shippensburg-2100-corridors-30m/
    PRE shippensburg-2100-corridors-np-30m/
    PRE shippensburg-2100-corridors-osi-30m/

We're using the vanilla and -np- ones here. Confirmed with Stroud that these are what they want to show.

Testing Instructions

  • Check out this branch
  • Go to :8000
  • Switch to the Coverage tab in the Layer Selector
    • Ensure you don't see DRB Urban Land 2050 layers
    • Ensure you do see DRB Urban Land 2011 Baseline layer
    • Ensure you do see DRB Urban Land 2100 layers

@rajadain rajadain added the OSI Funding Source: OSI label May 21, 2020
@rajadain rajadain force-pushed the tt/shippensburg-new-future-land-use-layers branch from 7fc1c4b to 5f72c97 Compare May 28, 2020 18:24
@rajadain
Copy link
Member Author

From Claire Jantz:

I think it makes the most sense to put up these four:

centers
centers-np
corridors
corridors-np

I’ll defer to Dave on if we want four up; if we only want to show two, then centers and corridors would be the ones.

From Dave Arscott:

Please add the 4 layers that Claire mentioned. Sounds like you can easily turn the NP layers off if we decide at the last minute to exclude.

here is my suggestion (trying to ensure no text wrapping in the Layers/CoverageGrid window - but probably just one character over wrap width on the longest version)

  • DRB 2011 Urban Baseline [yes, switching the year forward for emphasis, suggesting delete "land" to make room]
  • DRB 2100 Urban Center FX {abbreviation for forecast]
  • DRB 2100 Urban Center-NP FX
  • DRB 2100 Urban Corridor FX [perhaps abbreviate Corridor to Corr or Cor?]
  • DRB 2100 Urban Corridor-NP FX

@rajadain rajadain force-pushed the tt/shippensburg-new-future-land-use-layers branch from 5f72c97 to 7ea7e6b Compare May 29, 2020 15:55
@rajadain rajadain changed the title [WIP] Shippensburg Land Projections: Update Layers Shippensburg Land Projections: Update Layers May 29, 2020
@rajadain rajadain requested a review from mmcfarland May 29, 2020 15:59
@rajadain
Copy link
Member Author

This is now ready for review.

Copy link
Contributor

@mmcfarland mmcfarland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tiles are loading and look good, along with the layer selector and legend components 👍

@mmcfarland mmcfarland assigned rajadain and unassigned mmcfarland Jun 1, 2020
@rajadain rajadain merged commit 36bcf0b into develop Jun 1, 2020
@rajadain rajadain deleted the tt/shippensburg-new-future-land-use-layers branch June 1, 2020 20:55
@rajadain
Copy link
Member Author

rajadain commented Jun 1, 2020

Thanks for taking a look!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OSI Funding Source: OSI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants