-
Notifications
You must be signed in to change notification settings - Fork 8
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
DT_THERM=6*DT and DIABATIC_FIRST = False #94
Conversation
Revert to 1mo run
@@ -286,7 +286,7 @@ CLOCK_attributes:: | |||
history_ymd = -999 | |||
ice_cpl_dt = 99999 #not used | |||
lnd_cpl_dt = 99999 #not used | |||
ocn_cpl_dt = 3600 | |||
ocn_cpl_dt = 1800 |
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.
It is not necessarily to change the value of ocn_cpl_dt
unless stop_option
is nsteps
. I can do a cherry pick to add comments right after this parameter for all configurations. COSIMA/access-om3#157 (comment)
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.
Thanks @aekiss LGTM!
Automatic Git cherry-picking of commit(s) 58e3b75 2d639fe into 1deg_jra55do_iaf failed. This usually happens when cherry-picking results in a conflic or an empty commit. To manually cherry-pick the commits and open a pull request, please follow these instructions:
git checkout 1deg_jra55do_iaf
git pull
git checkout -b cherry_pick_from_pr94_into_1deg_jra55do_iaf
git cherry-pick 58e3b755e8e09664f4c8eedb40f428ce215a4bfb 2d639fe009f92860169677b18990864317f0a9ca
git push --set-upstream origin cherry_pick_from_pr94_into_1deg_jra55do_iaf
|
Hi @aekiss It seems that your cherry-picks failed because of the conflicts. You might need to follow the steps from #94 (comment) |
OK I'm working on it. The biggest conflict is due to differing |
I was under the impression we weren't going to change these params in our official config branches until we'd done the testing to determine what |
Ah yes, good point. Sorry if I jumped the gun there. I'll wait until we've tested before merging this into |
use DT_THERM=6*DT and DIABATIC_FIRST = False - see COSIMA/access-om3#138