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

Add test for Anomaly forcing and rename test for bias correction #1396

Closed
ekluzek opened this issue Jun 10, 2021 · 2 comments · Fixed by #1771
Closed

Add test for Anomaly forcing and rename test for bias correction #1396

ekluzek opened this issue Jun 10, 2021 · 2 comments · Fixed by #1771
Labels
bfb bit-for-bit simple easy for anyone to work on tests additions or changes to tests
Milestone

Comments

@ekluzek
Copy link
Collaborator

ekluzek commented Jun 10, 2021

Right now our test that's labeled "af" for anomaly forcing is actually a bias correction test (test mod af_bias_v7). This should be changed to be labeled "bc" for bias correction. And a new test added that's actually testing the anomaly forcing.

@ekluzek ekluzek added tag: simple bfb tests additions or changes to tests labels Jun 10, 2021
@ekluzek ekluzek changed the title Test Anomaly forcing rather and rename test for bias correction Add test for Anomaly forcing and rename test for bias correction Jun 10, 2021
@ekluzek
Copy link
Collaborator Author

ekluzek commented Jun 10, 2021

Note, #1386 relates to this.

@billsacks
Copy link
Member

One idea: put these tests in a cdeps test list (since they're actually testing cdeps code), but then also have copies in our clm_science test list.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bfb bit-for-bit simple easy for anyone to work on tests additions or changes to tests
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants