-
Notifications
You must be signed in to change notification settings - Fork 139
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
Gridded routing with lakes fails perfect restart #174
Comments
Updating this with new information... On a longer duration run, this fail becomes more severe with multiple variables in multiple outputs showing diffs in the perfect restart test. Thus, all testing on CI has been extended to 5 days in an attempt to better capture these diffs |
I dug a little deeper on this and found the following:
|
There also appears to be a domain issue, this could be in the files or in hte namelists, need to do more digging. Running v5.1.0 code with v5.0.2 domain yields the same minimal fail of perfect restarts only on the Running v5.1.0 code with dev domain yields numerous fails of perfect restarts
|
Next step is to run v5.0.2 code with dev domain |
v5.0.2 code with dev domain produces many fails. So, the numerour fails in the v5.1.0 code could be because of the dev domain or assosciated namelist json files.
|
The v5.0.2 domain has the following namelist settings for gridded
Wherease the dev domain has these:
Changing the dev domain settings to the same as the v5.0.2 settings results in consistant behavoir for gridded between v5.0.2 and v5.1.0 code with v5.0.2 and dev domains, respectively. Unfortunately, I believe the dev domain settings are correct. |
As far as I know, the gridded routing files are consistent with the NWM full routing, so should be 250 with an agg factor of 4. That is what I have been using for my local tests and appears to match the domain files. |
OK, this change has been applied to bugfix-5.0.x release branch with an updated croton_NY release asset |
Just linking this more recent error report since the variables impacted have changed a bit since the first report. You have to run a few days to get the diffs to show up. @dnyates is digging in. https://travis-ci.org/NCAR/wrf_hydro_nwm_public/builds/430627267 |
Fixed in PR #205 |
As shown in CI testing, Croton gridded routing configuration with lakes fails perfect restart test.
Expected Behavior
Perfect restarts in all vars.
Current Behavior
Per CI testing, lake_inflort does not perfect restart in Croton test case gridded routing configuration. All other vars pass, so this may be just a reporting issue (may not be impacting physics/solution).
Test log here: https://travis-ci.org/NCAR/wrf_hydro_nwm_public/builds/425358497
hydro restart diff:
Variable Group Count ... Range Mean StdDev
0 lake_inflort / 3 ... 3.74963 2.08033 2.13999
The text was updated successfully, but these errors were encountered: