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.
#1865 (compiling trunk with key_agrif , now key_bdy has been removed) – NEMO

Opened 7 years ago

Closed 6 years ago

Last modified 6 years ago

#1865 closed Bug (duplicate)

compiling trunk with key_agrif , now key_bdy has been removed

Reported by: cbricaud Owned by: jchanut
Priority: normal Milestone:
Component: AGRIF Version: trunk
Severity: minor Keywords:
Cc:

Description (last modified by nemo)

Since the key_bdy has been removed in the trunk, NEMO is not compiling with key_agrif

With bdy, variables are declared with target attribute:
exp: trc.F90:

TYPE(OBC_DATA),    PUBLIC, ALLOCATABLE, DIMENSION(:,:), TARGET ::  trcdta_bdy

When te code is pre-compiled with Agrif, those variables get the attribut pointer:
exp: trc.f90

type(OBC_DATA), target, pointer, dimension(:,:) :: trcdta_bdy

So the variable has target and pointer attribut.

Commit History (0)

(No commits)

Change History (5)

comment:1 Changed 7 years ago by clevy

  • Owner changed from nemo to jchanut

comment:2 Changed 7 years ago by mdunphy

Would it work to simply to wrap that line with

!$AGRIF_DO_NOT_TREAT
...
!$AGRIF_END_DO_NOT_TREAT

as is done in OPA_SRC/BDY/bdydta.F90 and OPA_SRC/BDY/bdy_oce.F90 ?

Last edited 6 years ago by nemo (previous) (diff)

comment:3 Changed 7 years ago by jchanut

  • Component changed from OPA to AGRIF

comment:4 Changed 6 years ago by jchanut

  • Resolution set to duplicate
  • Severity set to minor
  • Status changed from new to closed

This problem is more general and linked to the use of key_top see, #1972.

Just a warning concerning the use of AGRIF compiler directives. Each time one uses them, it means that the very same variable will be shared among grids. In the bdy case mentioned above, it means that one can not have open boundaries in child grids (but indeed, the target attribute is the problem here). Such a restriction should be removed one day so that I encourage people not to do that unless there is really no workaround.

comment:5 Changed 6 years ago by nemo

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