-
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
time counter in BLAZE #577
Comments
How does this affect coupled CABLE? |
One option to address this would be to go to number of days (not number of seconds) as the time counter - that would require alterations to the coupling (factors of 86400.0 in various places). At the moment I'm not proposing to do anything to the code - just noting that this is issue. |
Even better :) . I can imagine that offline keeps track of this in the drivers but I cant picture it in coupled. The only time(s) I can think of are dels(=1800 or 1200 in AM3 [I think]) and ktau(_gl), timestep_number which it gets from the UM. |
@har917 Can you specify what time counter you are talking about so there is no ambiguity? |
@ccarouge Time in the model is extremely confusing - what I picked up is that the time in the output from the BLAZE_9184 runs instead of continuing onwards past ~2020 it backtracked to 1860(ish) (at least when picked up by xarray)
Even more weirdly the spin up run from 1700-1950 gives this In the BLAZE_9184 branch time in the output is the combination of the calendar attribute (from I've checked the time data in the files (via xarray) - for the 1700-1950 run monthly output
This is all a bit odd - I'm thinking this is something to do with However I'm aware that the output has been extensively rewritten in MAIN so this may not be a problem that needs following up. |
Perhaps lost in the above - I'm now suspecting that there's two things going on here:
|
A Y2020 bug - so its only offline/BLAZE then? |
@ccarouge @JhanSrbinovsky A follow up - this isn't a CABLE/CASA issue but a BLAZE issue.
Net result - I think we need to convert BLAZE%time from an INTEGER into a double precision Real and tweak the BLAZE_write_output routine accordingly. Updating the title accordingly and linking to #529 |
Fixed via b9d0735 - closing issue as solved. Code will be merged into other branches at later date alongside other developments. |
The time counter in CABLE (or at least parts of CABLE, but not CASA) is carried as an INTEGER number of seconds since 1/1/1700.
Now that we are looking to run past ~2020 we are hitting limits on byte size. This is effectively resetting the clock part way through a run - at least in the output - with unknown and potentially unwanted consequences.
Solutions could involve a combination of:
Any changes to core model will require care with interaction between CABLE and UM @ccarouge @JhanSrbinovsky
The text was updated successfully, but these errors were encountered: