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.
#2636 (Wrong definition of TOP arrays in AGRIF) – NEMO

#2636 closed Bug (fixed)

Wrong definition of TOP arrays in AGRIF

Reported by: jchanut Owned by: jchanut
Priority: low Milestone:
Component: AGRIF Version: v4.0.*
Severity: minor Keywords:
Cc:

Description

Context

AGRIF and TOP

Analysis

in agrif_user.F90, definition of top variables does not take into account the number of ghostcells correctly.

Fix

Replace lines 658-662 in agrif_declare_var_top routine:

# else
   CALL agrif_declare_variable((/2,2,0,0/),(/3,3,0,0/),(/'x','y','N','N'/),(/1,1,1,1/),(/nlci,nlcj,jpk,jptra/),trn_id)
   CALL agrif_declare_variable((/2,2,0,0/),(/3,3,0,0/),(/'x','y','N','N'/),(/1,1,1,1/),(/nlci,nlcj,jpk,jptra/),trn_sponge_id)
# endif

by

# else
   CALL agrif_declare_variable((/2,2,0,0/),(/ind3,ind3,0,0/),(/'x','y','N','N'/),(/1,1,1,1/),(/nlci,nlcj,jpk,jptra/),trn_id)
   CALL agrif_declare_variable((/2,2,0,0/),(/ind3,ind3,0,0/),(/'x','y','N','N'/),(/1,1,1,1/),(/nlci,nlcj,jpk,jptra/),trn_sponge_id)
# endif

where

ind3 = 2 + nbghostcells

...

Commit History (1)

ChangesetAuthorTimeChangeLog
14605jchanut2021-03-09T17:34:00+01:00

#2636, correct definition of TOP variables in AGRIF

Change History (2)

comment:1 Changed 17 months ago by jchanut

In 14605:

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

comment:2 Changed 17 months ago by jchanut

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.