WikiWatershed / rapid-watershed-delineation

Rapid Watershed Delineation Code for MMW2
Apache License 2.0
12 stars 6 forks source link

Reduce storage and fix internally draining gaps. #54

Closed dtarb closed 7 years ago

dtarb commented 7 years ago

This addresses the issue #53

kdeloach commented 7 years ago

Consider splitting this up into 2 pull requests. One to reduce storage, and one to fix internally draining gaps.

Updating the Python code is a relatively quick process, but updating the underlying data is very time consuming. We'd prefer to wait a little longer before applying the data updates, just in case we discover more updates that need to made-- so we can apply them all at once.

dtarb commented 7 years ago

The fix to internally draining gaps involves data updates as it requires the new regions data that I have provided. The flow directions data to fix the gaps are not in the currently deployed data.

I could separate out python code to just the watershed attribute changes associated with reduced storage without the files being deleted, but this seems hardly worthwhile.

kdeloach commented 7 years ago

We were considering not deleting any data (only updating), in case we later decide to use those attributes. From that point of view, it isn't likely that we will need these changes to reduce storage.

In any case, we can still merge the code in but make a note that we did not necessarily run these scripts on production.

kdeloach commented 7 years ago

Using the latest code and data I was able to generate watersheds for areas outlined in the screenshot from #53.

For future reference, this area is "Rick Husband Amarillo International Airport" which is located East of Amarillo, TX.

Here is a watershed I generated at (roughly) LatLng(35.21169, -101.69701):

screenshot 2017-01-30 at 18 08 38

kdeloach commented 7 years ago

This worked without any code changes so I will merge this in tomorrow morning then begin the process to cycle these changes onto the staging environment.