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

Simplify rt.conf #352

Closed
climbfuji opened this issue Dec 31, 2020 · 1 comment · Fixed by #353
Closed

Simplify rt.conf #352

climbfuji opened this issue Dec 31, 2020 · 1 comment · Fixed by #353
Assignees
Labels
enhancement New feature or request

Comments

@climbfuji
Copy link
Collaborator

Description

Right now we have a lot of special treatments in rt.conf to run certain tests on some systems but not on others. For many of these tests, the exceptions were introduced in the past when something didn't run on certain systems (because it was currently developed, was missing input data, or the system had problems).

We need to check if these exceptions are still required or if we can remove some of them to cleanup rt.conf and facilitate the overall regression test cleanup that we are planning to do in the near future.

Solution

On a case-by-case basis, remove exceptions to run certain tests only on some platforms or to exclude certain platforms. Create new baselines and verify against them to make sure that the tests not only complete but also reproduce.

Alternatives

Deal with these exceptions later when doing the general overhaul of the global model regression tests.

@climbfuji climbfuji added the enhancement New feature or request label Dec 31, 2020
@climbfuji
Copy link
Collaborator Author

climbfuji commented Dec 31, 2020

@DusanJovic-NOAA I have started working on this and #310 at the same time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
2 participants