Opened 23 months ago

Closed 23 months ago

Last modified 23 months ago

#2151 closed Defect (fixed)

Add neuler test before reading restart in dynspg_ts

Reported by: mathiot Owned by: mathiot
Priority: low Milestone: 2018 release-4.0
Component: DYN Version: trunk
Severity: minor Keywords: Euler
Cc:

Description (last modified by nicolasmartin)

Context

With ln_bt_fw, dynspg_ts needs ub2_b, vb2_b, un_bf, vn_bf, sshbb_e, ubb_e, vbb_e, sshb_e, ub_e, vb_e to start. If we start from an Euler time step (or a reduce restart), reading these variables is not needed (overwrite later on).
A test on neuler was present in 3.6_STABLE to avoid reading these variables when started on an euler time step. The test was removed between 3.6_STABLE and 4.0_beta .

Recommendation

Add a test on neuler in ts_rst to avoid reading variables mentioned earlier.

It means replace:

   SUBROUTINE ts_rst( kt, cdrw )
 ...
         IF( ln_rstart .AND. ln_bt_fw) THEN    !* Read the restart file
            CALL iom_get( numror, jpdom_autoglo, 'ub2_b'  , ub2_b  (:,:), ldxios = lrxios )
            ...


by

   SUBROUTINE ts_rst( kt, cdrw )
 ...
         IF( ln_rstart .AND. ln_bt_fw .AND. (neuler/=0) ) THEN    !* Read the restart file
            CALL iom_get( numror, jpdom_autoglo, 'ub2_b'  , ub2_b  (:,:), ldxios = lrxios )
            ...

Tests with AMM12 show no change in results.

Commit History (1)

ChangesetAuthorTimeChangeLog
10256mathiot2018-10-30T14:04:17+01:00

fix #2151

Change History (2)

comment:1 Changed 23 months ago by mathiot

  • Resolution set to fixed
  • Status changed from new to closed

In 10256:

fix #2151

comment:2 Changed 23 months ago by nicolasmartin

  • Description modified (diff)
Note: See TracTickets for help on using tickets.