#2310 closed Bug (fixed)
bug in timestepping in OFF?
Reported by: | davestorkey | Owned by: | davestorkey |
---|---|---|---|
Priority: | normal | Milestone: | Unscheduled |
Component: | OFF | Version: | v4.0 |
Severity: | minor | Keywords: | v4.0 |
Cc: | acc |
Description
BE CAREFUL !!! Due to dynamic behaviour of this ticket creation page, it is highly recommend to set first all other fields before writing the ticket description below.
If you have lost your draft after an unwanted reload, you can click on the link 'Restore Form' in the contextual menu upper right to recover it.
Remove these lines after reading.
Context
The OFF module allows you to evolve passive tracers using previously-calculated ocean dynamics fields read in from a file. At each timestep the "now" fields are read in from the file (and possibly time-interpolated) using fld_read. But with the nonlinear free surface, the SSH and cell thicknesses (e3*) are timestepped forward using the "now" horizontal divergence field read in from the file.
The order in which things are done in nemo_gcm (OFF version) is:
- Call dta_dyn to read in "now" fields.
- Call dta_dyn_ssh to calculate the "after" SSH and e3* fields based on the "now" divergence field.
- Call dta_dyn_swp to time-filter and time-swap the SSH field.
- Call trc_stp to advance the passive tracers by one timestep.
But I think the order of 3 and 4 is wrong. It means that when trc_stp is called then the "now" SSH field is actually one timestep ahead of the "now" velocity fields. Dta_dyn_swp should be called after trc_stp. This would then be consistent with the order that things are done in stp where the time filtering and time swapping is done *after* the call to trc_stp.
Analysis
...
Fix
...
Commit History (2)
Changeset | Author | Time | ChangeLog |
---|---|---|---|
12026 | davestorkey | 2019-12-02T15:54:57+01:00 | Small bug fixes to to initial Euler timestep and OFF timestepping to ensure bit comparison the IMMERSE_first_steps branch with the trunk. Tickets #2310, #2311, #2343 apply. |
11480 | davestorkey | 2019-08-29T11:23:25+02:00 | 2019/dev_r10721_KERNEL-02_Storkey_Coward_IMMERSE_first_steps : Merge in changes from branch of branch. Main changes:
Notes:
|
Change History (4)
comment:1 Changed 5 years ago by davestorkey
comment:2 Changed 5 years ago by davestorkey
In 12026:
comment:3 Changed 5 years ago by davestorkey
- Resolution set to fixed
- Status changed from new to closed
Fixed in trunk @ 12026
comment:4 Changed 2 years ago by nemo
- Keywords v4.0 added
In 11480: