preindustrial+concentrations restart: don't change year#700
preindustrial+concentrations restart: don't change year#700blimlim merged 2 commits intodev-preindustrial+concentrationsfrom
Conversation
|
!test repro |
|
❌ The Bitwise Reproducibility Check Failed ❌ When comparing:
Further informationThe experiment can be found on Gadi at The checksums generated by this The checksums compared against are found here https://github.com/ACCESS-NRI/access-esm1.6-configs/tree/aa8e59a579adc78ab433c6b5d21ea03aebabeaf2/testing/checksum Test summary: |
Not expected. I'll try to see what happened |
|
what happened this was just the start date right? not even that, just a year number. |
|
Yeah, only the restart year has changed which is not expected to change results. The checksums are identical for all fields other than |
|
The following shows the column mean ocean age tracer for 1 month runs starting from each restart: MOM appears to completely reset the age tracer when starting from year 1. This might be something we don't want to ever happen, I'll make a separate issue for discussion. |
|
!test repro commit |
|
I'm thinking that whether or not we change the date for this configuration depends on what we want it to represent if we are keeping the picontrol that does have the restart date change in. |
|
❌ The Bitwise Reproducibility Check Failed ❌ When comparing:
🔧 The new checksums will be committed to this PR, if they differ from what is on this branch. Further informationThe experiment can be found on Gadi at The checksums generated by this The checksums compared against are found here https://github.com/ACCESS-NRI/access-esm1.6-configs/tree/aa8e59a579adc78ab433c6b5d21ea03aebabeaf2/testing/checksum Test summary: |
|
@JhanSrbinovsky, @rml599gh, this PR swaps the restart for the preindustrial+concentrations run to a copy of I'll add the MOM namelist change to a separate PR which will let the change be copied to all the configurations |
|
phew, I'm glad my memory wasn't playing tricks on me: That was the restart we used before from: Can you clarify, at the start of the ocean age tracer discussion - there was a comment that I read as "even though the age tracer changes, reproducibility shattered, all other physical characteristics are the same." Was that not the case? |
rml599gh
left a comment
There was a problem hiding this comment.
OK - I think I've got this straight in my mind now. This takes the end of Jhan's picontrol run and puts it into the dev config instead of the test one - which is what we want.
|
Thanks @rml599gh and @JhanSrbinovsky
Yeah that's correct – the ocean age tracer variable changed, but all other aspects of the physical state in the ocean, atmosphere and sea ice weren't affected.
That's right, It's taking the restart from the end of Jhan's run and putting it into the dev config |


1. Summary:
Following further discussion with @JhanSrbinovsky and @rml599gh, we would like to use the same restart from #699 but without the change to the restart year.
This PR updates the restart to use a new copy, without the date change.
2. Issues Addressed:
3. Dependencies (e.g. on payu, or model)
This change requires changes to (note pull request(s) where relevant):
4. Ad-hoc Testing
What ad-hoc testing was done? How are you convinced this change is correct (plots are good)?
5. CI Testing
!test reprohas been run6. Reproducibility
Is this reproducible with the previous commit? (If not, why not?)
!test repro commithas been run.7. Performance
Has the model performance (say, throughput of model-years/wall-day) changed?
If yes, provide the numbers from your testing. Is the performance better or worse?
8. Manifests
Have you changed the executable, the input files and/or the restart files?
If yes, have you updated the manifests?
To update the manifests, run payu setup (in a cloned copy of your feature branch) with reproducibility tests turned off:
Then commit the newly created manifest files (under manifests/) only to the branch for this PR.
9. Documentation
Is the documentation updated?
10. Merge Strategy