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

fixes display on #1929 - matches correct gage #321

Merged

mattanger merged 3 commits into development from bugfix/1929_beta-gage-matches-summary-gage

Feb 3, 2021

Member

ryangroth5 commented Feb 1, 2021

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
fixes display on #1929 - matches correct gage

What is the current behavior? (You can also link to an open issue here)
uses a zero-index gage in the reach gages to display flow on beta, though the rest of the logic uses a gage match

What is the new behavior (if this is a feature change)?
uses correct gage

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
no

Other information:
none

ryangroth5 self-assigned this

ryangroth5 added 3 commits

February 1, 2021 13:14


          fixes display on #1929 - matches correct gage

35c1901


          type check on parameter

025347e


          type check

5533f12

ryangroth5 requested a review from ngottlieb

February 3, 2021 14:02

mattanger merged commit 21e94a6 into development

mattanger deleted the bugfix/1929_beta-gage-matches-summary-gage branch

February 3, 2021 17:40

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet