New URL for NEMO forge!

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.
#1872 (NESTING TOOLS AGRIF read updated bathymetry for creating agrif restarts) – NEMO

Opened 7 years ago

Closed 5 years ago

#1872 closed Enhancement (invalid)

NESTING TOOLS AGRIF read updated bathymetry for creating agrif restarts

Reported by: jharlass Owned by: jchanut
Priority: low Milestone:
Component: AGRIF Version: v3.6
Severity: minor Keywords: AGRIF bathymetry nesting
Cc: sciliberti



Use bathy_updated from base model to correctly compute masks (partial steps) for nest restart creation.



  • (a) Revision vs. (b) Working

    a b  
    113113  !        
    114114  CALL set_child_name(parent_coordinate_file,Childcoordinates)    
    115115  CALL set_child_name(parent_meshmask_file,Childbathy)  
    116116  CALL set_child_name(parent_bathy_meter,Childbathymeter)    
     117  ! use bathy_updated for base model ONLY, if bathy_update=true in namelist.input 
     118  IF (bathy_update) THEN 
     119    parent_bathy_meter=updated_parent_file 
     120  END IF 
    117122  ! create this file 
    118123  ! 
    164170     ! 
    165171  ENDIF 
    166172  ! 
     173  WRITE(*,*) 'Reading Bathymetry: ',TRIM(parent_bathy_meter) 
     174  CALL Read_Ncdf_var('Bathymetry',parent_bathy_meter,G0%bathy_meter) 
     176  WRITE(*,*) 'Reading mbathy    : ',TRIM(parent_meshmask_file) 
    167177  CALL Init_mask(parent_meshmask_file,G0,x,y) 
     178  WRITE(*,*) 
     179  WRITE(*,*) 'Reading Bathymetry: ',TRIM(Childbathymeter) 
     180  CALL Read_Ncdf_var('Bathymetry',Childbathymeter,G1%bathy_meter) 
     181  WRITE(*,*) 'Reading mbathy    : ',TRIM(childbathy) 
    168182  CALL Init_mask(childbathy,G1,1,1) 
    170184  G0%tmask = 1.    

Commit History (0)

(No commits)

Change History (7)

comment:1 Changed 7 years ago by nemo

  • Keywords nesting added; NESTING removed

comment:2 Changed 7 years ago by jchanut

  • Component changed from OPA to AGRIF

comment:3 Changed 6 years ago by clevy

  • Owner changed from nemo to jchanut

comment:4 Changed 6 years ago by clevy

  • Cc sciliberti added
  • Owner changed from jchanut to sciliberti
  • Status changed from new to assigned

comment:5 Changed 5 years ago by jchanut

  • Owner changed from sciliberti to jchanut
  • Severity set to minor


Could you please explain why you need the updated parent bathymetry ? Input restart matches the original (i.e. not updated domain) so that I don't understand why you advise this. Maybe I missed something ?
Thanks for your help,

comment:6 Changed 5 years ago by jharlass

You are right and I can not entirely recall the reason. From my current perspective it is indeed better to use to treat the input fields equally. We came across this issue when adding the partial step support in the creation process of all masks (Init_mask). When a nested configuration starts and the first update takes place immediately, base values will be overwritten by nest values in any way. So either either bathy_updated would not fit not fit to the base restart or the base restart will not fit to 1_bathy_meter.
Ticket can be closed and proposed change dismissed.

comment:7 Changed 5 years ago by jchanut

  • Resolution set to invalid
  • Status changed from assigned to closed

Thanks Jan. I close the ticket.

Note: See TracTickets for help on using tickets.