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

Switch NLCD19 Products to Correctly Aligned Layers #3549

Merged
merged 1 commit into from
Jun 15, 2022

Conversation

rajadain
Copy link
Member

@rajadain rajadain commented Jun 13, 2022

Overview

The previously ingested NLCD19 products (2001, 2006, 2011, 2016, and 2019) were all out of alignment with previous layers in the systems, as described here: WikiWatershed/mmw-geoprocessing#102 (comment).

The fixed layers, shown here: WikiWatershed/mmw-geoprocessing#102 (comment), were ingested with uint8raw suffixes. This changes MMW to use the new uint8raw layers, rather than the old byte layers.

Connects WikiWatershed/mmw-geoprocessing#102

Demo

image

Notes

Since this changes geoprocessing results, we need to clear the cache when this is deployed:

redis-cli -h cache.service.mmw.internal -n 1 --raw KEYS ":1:geop*" | xargs redis-cli -h cache.service.mmw.internal -n 1 DEL

Testing Instructions

  • Check out this branch and ./scripts/bundle.sh --debug
  • Clear any saved geoprocessing results:
    vagrant ssh services -c 'redis-cli -n 1 --raw KEYS ":1:geop_*" | xargs redis-cli -n 1 DEL'
  • Go to http://localhost:8000/ and select an east coast HUC-12
    • Ensure all Land Analyses complete correctly
  • Run the Watershed Multi-Year Model
    • Ensure it completes correctly

These are correctly aligned. For details see:
WikiWatershed/mmw-geoprocessing#102
@rajadain rajadain added the GEN Funding Source: General label Jun 13, 2022
Copy link
Contributor

@emilyhu0106 emilyhu0106 left a comment

Choose a reason for hiding this comment

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

Good job in investigating in this alignment issue! The land analysis and multi-year modeling in the HUCs I tested all works smoothly.

Base automatically changed from tt/enable-recent-weather-in-drwi to develop June 15, 2022 15:25
@rajadain rajadain merged commit 2fa873c into develop Jun 15, 2022
@rajadain rajadain deleted the tt/switch-to-correct-nlcd-layers branch June 15, 2022 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GEN Funding Source: General
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants