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 log fails: Warning: Missing processing time for __ #1458

Open
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

RobHanna-NOAA commented Mar 10, 2025

In post processing.

This warning did not show up anywhere in printed file logs but shows up a very large number of time on screen. This is a known issue where a print statement is issued in a py file that is run by a bash file. The problem occurs in the duration_system.py file when iterating through the 1processing_time_{huc}.txt` files. It should not have any missing files.

When we ran hand 4.6.0.0 BED run, we had 2,153 HUCs but total duration csv only shows 1,967 suggesting that appx 200 HUCs through this error message.

Related.. later in that loops, change it so it does not remove that file from each HUC as it processes it.

But having that information is very helpful and we have never had a full set of valid stats for a BED run. This feature was put in more than three months ago but was never used against a BED run. If we compare our HUC list to this hand 4.6.0.0 list, we might be able to find some HUCs that may have failed and see if we can figure out something on this story.

It is not a must, but a very nice to have for the next BED if we can even find it. It likely has been showing up on screen in many runs regardless of size but with it not showing up on logs and only on screen, it might be easy to find by watching the screen for a bit and likely doesn't even have to be a very big run. Hard to say.

Related:

There is a problem with calculating branch 0 processing times in the total duration processing logic. See 1459.

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