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.

Sign up for GitHub

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

Jump to bottom

Add "NLCD to MapShed land class" lookup table #9

Open
aufdenkampe opened this issue Aug 10, 2018 · 3 comments
Open

Add "NLCD to MapShed land class" lookup table #9

aufdenkampe opened this issue Aug 10, 2018 · 3 comments

Comments

@aufdenkampe
Copy link
Member

To make the mapping and underlying assumptions more explicit.
Add to the new "Technical Documentation" tab (#7).

This is item 3 in the list for issue #5.
@aufdenkampe aufdenkampe mentioned this issue Aug 10, 2018
Open
4 tasks
@aufdenkampe
Copy link
Member Author

aufdenkampe commented Oct 3, 2018 •
edited
Loading

Existing documentation include:

@aufdenkampe
Copy link
Member Author

aufdenkampe commented Oct 3, 2018 •
edited
Loading

Explicit Mapshed to NLCD mapping for MMW occurs in the code here:

Curve numbers for NLCD classes are in the code here:

@aufdenkampe aufdenkampe reopened this Oct 3, 2018
@aufdenkampe
Copy link
Member Author

aufdenkampe commented Oct 3, 2018 •
edited
Loading

Commit ff03e26 only partially resolves this issue, because the added table is not used as a lookup.

Also, uses MMW v1.23 mapping of "NLCD 21 Developed, Open Space", described in WikiWatershed/model-my-watershed#2937.

Need to upgrade with MMW release 1.24

@aufdenkampe aufdenkampe mentioned this issue Oct 3, 2018
Open
aufdenkampe added a commit that referenced this issue Oct 25, 2018
@aufdenkampe
Add co-author Reid Christianson; Add forward compatibility for Ld_Mixed
6d6fe37 
- Reid Christianson was the original developer of the Clean Water Optimization Tool when he was at the Center for Watershed Protection, and much of his work is built into this tool.
- Added forward compatibility for handling the outputs for "Low-Density Open" that will come with MMW v1.24 as described in #9 & #10. Although this implementation doesn't explicitly show the new mapping of "NLCD 21 Developed, Open Space", it will still give the correct answer.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@aufdenkampe