New URL for NEMO forge!   http://forge.nemo-ocean.eu

Since March 2022 along with NEMO 4.2 release, the code development moved to a self-hosted GitLab.
This present forge is now archived and remained online for history.
#2553 (restartability broken when LEV melt pond activated (ln_pnd_LEV)) – NEMO

Opened 3 years ago

Closed 3 years ago

#2553 closed Bug (fixed)

restartability broken when LEV melt pond activated (ln_pnd_LEV)

Reported by: mathiot Owned by: systeam
Priority: low Milestone:
Component: SI3 Version: trunk
Severity: minor Keywords: restartability,
Cc:

Description

Context

Despite restartability test OK with ORCA2_ICE_PISCES in SETTE, it appears ln_pnd_LEV break restartability because the melt pond variables are not read from the ice restart in icedyn_adv_pra.F90.

Analysis

Ckeck of melt pond variable presence in icedyn_adv_pra is using the wrong file id, so SI3 assume the restart does not contains the melt pond variables.

It affects NEMO4 and trunk.

Fix

Replace numror by numrir when checking presence of sxap and sxvl.

  • icedyn_adv_pra.F90

     
    10051005            END DO 
    10061006            ! 
    10071007            IF( ln_pnd_LEV ) THEN                                    ! melt pond fraction 
    1008                IF( iom_varid( numror, 'sxap', ldstop = .FALSE. ) > 0 ) THEN 
     1008               IF( iom_varid( numrir, 'sxap', ldstop = .FALSE. ) > 0 ) THEN 
    10091009                  CALL iom_get( numrir, jpdom_auto, 'sxap' , sxap , psgn = -1._wp ) 
    10101010                  CALL iom_get( numrir, jpdom_auto, 'syap' , syap , psgn = -1._wp ) 
    10111011                  CALL iom_get( numrir, jpdom_auto, 'sxxap', sxxap ) 
     
    10231023               ENDIF 
    10241024                  ! 
    10251025               IF ( ln_pnd_lids ) THEN                               ! melt pond lid volume 
    1026                   IF( iom_varid( numror, 'sxvl', ldstop = .FALSE. ) > 0 ) THEN 
     1026                  IF( iom_varid( numrir, 'sxvl', ldstop = .FALSE. ) > 0 ) THEN 
    10271027                     CALL iom_get( numrir, jpdom_auto, 'sxvl' , sxvl , psgn = -1._wp ) 
    10281028                     CALL iom_get( numrir, jpdom_auto, 'syvl' , syvl , psgn = -1._wp ) 
    10291029                     CALL iom_get( numrir, jpdom_auto, 'sxxvl', sxxvl ) 

Commit History (2)

ChangesetAuthorTimeChangeLog
13637mathiot2020-10-19T16:16:19+02:00

fix ticket #2553 in trunk

13634mathiot2020-10-19T16:12:54+02:00

fix ticket #2553 in NEMO4

Change History (3)

comment:1 Changed 3 years ago by mathiot

In 13634:

Error: Failed to load processor CommitTicketReference
No macro or processor named 'CommitTicketReference' found

comment:2 Changed 3 years ago by mathiot

In 13637:

Error: Failed to load processor CommitTicketReference
No macro or processor named 'CommitTicketReference' found

comment:3 Changed 3 years ago by mathiot

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

Clem agreed it is a bug but we really did not understand why SETTE didn't catch this bug. I didn't go further in trying to understand why sette missed this bug (no melt pond at time of restart ? something else ?)

Note: See TracTickets for help on using tickets.