-
Notifications
You must be signed in to change notification settings - Fork 105
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
Add IFI to regression tests (and put regression tests in repository) #1119
Conversation
Before this is merged, I'd like other people to confirm rt.sh works for them on Hera, Hercules, and Orion. Presently, it won't run for non-epic users, except on Hera, due to permission problems discussed in |
@SamuelTrahanNOAA I just attempted to run on Hercules and Orion, and I'm getting an error in nohup.out:
|
That error means this directory doesn't exist:
The I updated |
You can find the locations in export machine=HERA
export homedir=${homedir:-"/scratch2/BMC/wrfruc/Samuel.Trahan/UPP/test_suite"}
...
export machine=ORION
export homedir=${homedir:-"/work2/noaa/wrfruc/strahan/upp-ifi-regtest/upp-data"}
...
export machine=HERCULES
export homedir=${homedir:-"/work2/noaa/wrfruc/strahan/upp-ifi-regtest/upp-data"} I put Hercules and Orion in the same directory because that's what epic did in their project area.
The simplest way:
I don't expect any other tests to change output. |
@gspetro-NOAA I have no comments on this PR. We may start the final testing after you sync baselines from Sam's account to the EPIC's account. |
I have updated baselines on all systems, and the RTs pass on Hera and Orion with no changes (from the new baselines). I will test on Hercules once it's back up. Then I'll be able to approve. |
@gspetro-NOAA Great! |
@WenMeng-NOAA @gspetro-NOAA - I reverted the changes to test_suite paths. They now point to whatever they did before my commits.
|
The testing with ifi code involved on Hera: ./rt.sh -a ovp
The results look good to me. |
This is a minor thing, but I noticed some runtimes are longer than the expected runtime. It's only a small amount, but perhaps the expected runtimes should be updated eventually. |
The testing without ifi code involved on Hera: ./rt.sh -a ovp -d
The results look good to me. |
@SamuelTrahanNOAA We could address that in the future. |
My regression test results on Hera using the latest updates to
|
I have wondered about this, too. On Orion/Hercules, in particular, the |
The UPP RTs have been completed on WCOSS2 with no baseline changes. |
This PR is ready for merging. |
This adds four regression tests to the UPP test suite related to the closed-source library IFI, which is generally not accessible to users. These tests are enabled by the presence of a sorc/libIFI.fd/src directory in the UPP clone.
Presently, the ifi_standalone_hrrr test is disabled because the output doesn't match post_hrrr_ifi.
In pursuant to that, this PR also:
Also, emacs autosave files
*~
are added back to the .gitignore. That gitignore line was removed, likely by accident, in prior PRs. If you'd prefer me to take that change out, please let me know and I'll open another PR for it.Documentation Required
Users should know how to build IFI's executables via compile_upp.sh. It's the
-B
option.Blocking Dependencies
Presently, only epic group members can run rt.sh on MSU HPC platforms due to permission problems.