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

Plot velocity and depth time series for problematic spots when the model have multiple domains #10

Closed
mohamedmorsy opened this issue Aug 16, 2019 · 2 comments
Closed

Plot velocity and depth time series for problematic spots when the model have multiple domains #10

mohamedmorsy opened this issue Aug 16, 2019 · 2 comments

Comments

@mohamedmorsy
Copy link

Please check if the code loop through all the model domains (not only one domain) and identify any high velocities.
@johnwallx
Copy link
Contributor

This is still being carried out in the QAQC-FluvialTest notebook, but does not get executed for pluvial. Not sure why this is the case. Notes within velCheckMain suggest something was not executing under a particular circumstance.

Pinging @slawler for insight.

@slawler
Copy link
Member

slawler commented Aug 20, 2019

Resolved in PR #15

@slawler slawler closed this as completed Aug 20, 2019
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
3 participants
@johnwallx @mohamedmorsy @slawler