-
Notifications
You must be signed in to change notification settings - Fork 25
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
Release 1.4 update regression tests #380
Release 1.4 update regression tests #380
Conversation
I also tried to test with hybrid vertical coordinates enabled. However, I'm not sure this makes sense since the hybrid coordinate setup has changed significantly since |
@TomasTorsvik - In looking at the output of the ERP test for BLOM - it turns out that only 2 fields are different - plon and plat. That's because these two fields actually show differences in the masked values and that makes sense. Can you please verify that you are also just seeing these two differences. If so - I think we should keep this test for now - but work to remove these differences. |
@mvertens
|
I was doing this in noresm2.5 and the file was an hbgcd file. In rerunning the latest test suite it seems that the ERP test is passing (in non-hybrid mode. (ERP_Ld3_P256.T62_tn14.NOINYOC.betzy_intel). |
Regression test: |
@TomasTorsvik - the test is passing now with #377. I've done so much testing today on the different components aux tests that I cannot back track to exactly what the failed test was. However #377 looks okay in terms of regression testing. |
@mvertens - great! I think we can assume that ERP is fine for noresm2.5. I will test for noresm2.1 and noresm2.3 as well (but that will not be on the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
Create regression test suites
aux_blom_noresm
andaux_hamocc_noresm
.I propose to have a separate test suit for iHAMOCC to enable testing of different optional configurations for ocean BGC. For changes to the physics, I think in most cases it will be sufficient to test only the default iHAMOCC setup.
If you agree that this setup is useful, I will include this setup also for
noresm2.1
andnoresm2.3
.I have included a ERP test, testing for bit-reproducible results with different processor count and openmp enabled. This test currently fails at the
COMPARE_base_rest
stage. We have not included answer changing bug fixes in v1.4, so I'm not sure we will correct this for this version of the model. I wonder if it still makes sense to keep it in as a reminder not to rely on this setting, or is it better to take it out?After merging this PR, I would like to create a new release
v1.4.2
to be used innoresm2.0.8
.