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

[5pt] Total huc processing duration for branch 0 is incorrect #1459

Open
RobHanna-NOAA opened this issue Mar 10, 2025 · 0 comments
Open

[5pt] Total huc processing duration for branch 0 is incorrect #1459

RobHanna-NOAA opened this issue Mar 10, 2025 · 0 comments
Labels
bug Something isn't working Med Priority
Milestone

Comments

@RobHanna-NOAA
Copy link
Contributor

There is a bug in calculating the branch 0 timing.
The intent:

  • HUC duration: Total processing for the unit, branch 0 and all of it's branches included: Very top to very bottom of run_unit_wb.sh. This too is correct.

  • Branch 0 processing by itself: In run_unit_wb.sh from the time it starts processing branch 0, appx line 130, to appx line 298, after usgs crosswalk. This one is incorrect.

  • Remaining branches combined: in run_unit_wb.sh, appx line 314, to line 329. This one appears to be correct in logic.

It was detected in hand_4_6_0_0. It likely has been there since the beginning but was missed on testing.

Also.. see a likely related card : 1458: Missing processing time.

@RobHanna-NOAA RobHanna-NOAA changed the title [5pt] total huc processing duration for branch 0 is incorrect [5pt] Total huc processing duration for branch 0 is incorrect Mar 10, 2025
@RobHanna-NOAA RobHanna-NOAA added bug Something isn't working Med Priority labels Mar 10, 2025
@RobHanna-NOAA RobHanna-NOAA modified the milestone: FIM v4.6 Mar 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Med Priority
Projects
None yet
Development

No branches or pull requests

1 participant