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

More thoroughly investigate answer changes in ctsm1.0.dev001 #690

Closed
billsacks opened this issue Apr 23, 2019 · 10 comments
Closed

More thoroughly investigate answer changes in ctsm1.0.dev001 #690

billsacks opened this issue Apr 23, 2019 · 10 comments
Assignees
Labels
investigation Needs to be verified and more investigation into what's going on.

Comments

@billsacks
Copy link
Member

ctsm1.0.dev001 included the following answer changes:

  • Larger than roundoff-level changes expected from the following:
    • Include qflx_h2osfc_surf in BGC code: expected to be same climate
    • Two changes just affecting VIC
    • A change just affecting the QOVER diagnostic field
  • Additional changes believed to be roundoff-level

June 28, 2018, @olyson ran the ensemble consistency test against release-clm5.0.01 (I'm not sure if this was the 9-timestep F compset UF-ECT, or the 1-year B compset ECT).

Should we do more thorough testing on this tag to ensure that it hasn't changed answers relative to release-clm5.0.01? e.g., should we do longer runs and run the diagnostics package on this, or should we run the 1-year ECT if only the UF-ECT was run?

@billsacks billsacks added the investigation Needs to be verified and more investigation into what's going on. label Apr 23, 2019
@billsacks
Copy link
Member Author

@dlawrenncar @olyson feel free to mark this as a wontfix if you feel like this has been tested sufficiently.

@olyson
Copy link
Contributor

olyson commented Apr 24, 2019

I ran the 9step test on this just now and it passes overall but fails one PC for one ensemble member.

ctsm1 0 dev001_statistical_verification

@billsacks
Copy link
Member Author

@olyson what was the baseline tag for this?

@olyson
Copy link
Contributor

olyson commented Apr 24, 2019

It is supposed to be pointing to this:
/gpfs/fs1/p/cesmdata/cseg/inputdata/validation/uf_ensembles/cesm2.0

@olyson
Copy link
Contributor

olyson commented Apr 24, 2019

There is an attribute added to the netcdf files that I uploaded that include the model version:

model_version=cesm2.0.0

I used release-cesm2.0.0 but with ctsm1.0.dev001 to run the simulations required.

@billsacks
Copy link
Member Author

Huh, that's strange. cesm2.0.0 uses release-clm5.0.01, which is what you used as a baseline last June (I think). But you're getting different conclusions now vs. before?

@olyson
Copy link
Contributor

olyson commented Apr 24, 2019

Yes, I replaced release-clm5.0.01 in my checkout of release-cesm2.0.0 with ctsm1.0.dev001
The difference is limited to 1 PC score for 1 run. Maybe due to compiler or module changes? I had to update the modules to get this to work.

@billsacks
Copy link
Member Author

Oh, okay: now that I look more closely, I see that the overall result is a pass, like before.

So I guess the question remains, for you and/or @dlawrenncar , whether this is sufficient or we should do some additional confirmation.

@dlawrenncar
Copy link
Contributor

dlawrenncar commented Apr 25, 2019 via email

@billsacks
Copy link
Member Author

Thanks, @dlawrenncar . I'll close this issue then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
investigation Needs to be verified and more investigation into what's going on.
Projects
None yet
Development

No branches or pull requests

3 participants