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

Do not use watershed groups for upstream analysis #52

Open
smnorris opened this issue Jun 23, 2021 · 3 comments
Open

Do not use watershed groups for upstream analysis #52

smnorris opened this issue Jun 23, 2021 · 3 comments
Labels
bug Something isn't working
Milestone

Comments

@smnorris
Copy link
Owner

Watershed codes / watershed groups / assessment watersheds are not quite consistent enough to use for pre-aggregated upstream summaries.

For example, looking for area upstream of this fundamental watershed:
watershed_feature_id = 8924583
wscode=100.190442.244975
localcode=100.190442.244975.541514

If we look upstream using the fundamental watersheds, almost of the Nicola River / NICL watershed group is included.
If we look upstream using assessment watersheds or watershed groups, the NICL watershed group is excluded because the watershed group is coded with local code 100.190442.244975.539934 - there is a tiny bit of river polygon with that code present in the group.

Do not use watershed groups (or assessment watersheds, that are also based on group boundaries) for upstream/downstream queries.

FWA_WatershedAtMeasure and fwa_watersheds_upstream_area.sql will have to be fixed.

@smnorris smnorris added the bug Something isn't working label Jun 23, 2021
@smnorris smnorris changed the title upstream area - assessment watersheds / watershed groups as pre-aggregated shapes Do not use watershed groups for upstream analysis Jun 23, 2021
@smnorris
Copy link
Owner Author

smnorris commented Jun 23, 2021

Also - because they are not consistent enough to produce results equivalent to using fundamental watersheds, dropping the watershed codes from the watershed groups might be a good idea.

@smnorris
Copy link
Owner Author

fixed for fwa_watersheds_upstream_area with 6f03ce6

@smnorris smnorris added this to the v0.2.0 milestone Sep 28, 2021
@smnorris
Copy link
Owner Author

When watershed codes are removed from fwa_watershed_groups_poly :

  • how does fwa_basins_poly get generated? Just copy/paste watershed group codes/ids manually into the query?
  • is there any other convenient aggregation that could replace watershed groups in fwa_watershedatmeasure()?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant