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

CICE does not set date correctly when use_restart_time = .false. #38

Closed
nichannah opened this issue Aug 14, 2019 · 2 comments · Fixed by #39
Closed

CICE does not set date correctly when use_restart_time = .false. #38

nichannah opened this issue Aug 14, 2019 · 2 comments · Fixed by #39

Comments

@nichannah
Copy link
Contributor

As mentioned in COSIMA/libaccessom2#22 it looks like the driver part of CICE is not setting the year correctly when use_restart_time = .false..

The current year is being set using the forcing date rather than the experiment date.

@aekiss
Copy link
Contributor

aekiss commented Dec 20, 2019

Reopening this issue.

Commit e3200e8 (merged in ab47343) is not conditional on use_restart_time = .false. and prevents restarting with use_restart_time = .true., so was reverted in d3e8bdf.

So as yet we don't have a neat way to deal with multiple forcing cycles (issue COSIMA/access-om2#149) or setting date correctly with use_restart_time = .false..

Should the first argument passed to input_data here

call input_data(accessom2%get_forcing_start_date_array(), &

be conditional on use_restart_time (i.e. pass get_forcing_start_date_array if use_restart_time is true, get_cur_exp_date_array if false)?

Slack discussion is here: https://arccss.slack.com/archives/C6PP0GU9Y/p1576623326004300

aekiss added a commit to COSIMA/01deg_jra55_iaf that referenced this issue Dec 20, 2019
aekiss added a commit to COSIMA/01deg_jra55_ryf that referenced this issue Dec 20, 2019
aekiss added a commit to COSIMA/025deg_jra55_iaf that referenced this issue Dec 20, 2019
aekiss added a commit to COSIMA/025deg_jra55_ryf that referenced this issue Dec 20, 2019
aekiss added a commit to COSIMA/1deg_jra55_iaf that referenced this issue Dec 20, 2019
aekiss added a commit to COSIMA/1deg_jra55_ryf that referenced this issue Dec 20, 2019
aekiss added a commit to COSIMA/01deg_jra55_iaf that referenced this issue Dec 20, 2019
aekiss added a commit to COSIMA/01deg_jra55_ryf that referenced this issue Dec 20, 2019
aekiss added a commit to COSIMA/025deg_jra55_ryf that referenced this issue Dec 20, 2019
aekiss added a commit to COSIMA/025deg_jra55_iaf that referenced this issue Dec 20, 2019
aekiss added a commit to COSIMA/1deg_jra55_iaf that referenced this issue Dec 20, 2019
aekiss added a commit to COSIMA/1deg_jra55_ryf that referenced this issue Dec 20, 2019
@nichannah
Copy link
Contributor Author

I have fixed this. More explanation below.

In the case where use_restart_time = .false. the current experiment time will be set entirely by libaccessom2 (according to user input). When use_restart_time = .true. only the start year of the experiment (CICE calls this year_init) is determined by libaccessom2 and the current experiment year is calculated using this and the time-keeping values within the restart file. To handle this libaccessom2 needs to provide something different depending on use_restart_time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants