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.
#831 (ghost line not passed across processors in SUBROUTINE zgr_sco of domzgr.F90) – NEMO

Opened 13 years ago

Closed 11 years ago

Last modified 2 years ago

#831 closed Bug (fixed)

ghost line not passed across processors in SUBROUTINE zgr_sco of domzgr.F90

Reported by: agn Owned by: acc
Priority: normal Milestone:
Component: OCE Version: v3.3
Severity: Keywords: MPP OPA SCO v3.3
Cc:

Description

Hi,
With s-coordinates on multiple processors (4x4) ORCA2_LIM2 was failing with undefined e3

===>>> : E R R O R

===========

zgr_sco : e3w or e3t =< 0 at point (i,j,k)= 1 39

..etc

Gurvan suggested the following fix, which cured the problem. However might there be a better way of solving the problem involving adding a 4th argument to the preceding call to lbc_lnk
CALL lbc_lnk( zenv, 'T', 1._wp ) ?


$ svn diff NEMOGCM/NEMO/OPA_SRC/DOM/domzgr.F90

Index: NEMOGCM/NEMO/OPA_SRC/DOM/domzgr.F90
===================================================================
--- NEMOGCM/NEMO/OPA_SRC/DOM/domzgr.F90 (revision 2769)
+++ NEMOGCM/NEMO/OPA_SRC/DOM/domzgr.F90 (working copy)
@@ -1262,6 +1262,14 @@

! ! ================ !
!
! ! envelop bathymetry saved in hbatt

+ DO ji = nlci+1, jpi ! Fill ghost line
+ zenv(ji,1:nlcj) = zenv(nlci,1:nlcj)
+ END DO
+
+ DO jj = nlcj+1, jpj
+ zenv(:,jj) = zenv(:,nlcj)
+ END DO
+

hbatt(:,:) = zenv(:,:)
IF( MINVAL( gphit(:,:) ) * MAXVAL( gphit(:,:) ) <= 0._wp ) THEN

CALL ctl_warn( ' s-coordinates are tapered in vicinity of the Equator' )

Commit History (1)

ChangesetAuthorTimeChangeLog
3563acc2012-11-15T18:03:44+01:00

#831 and #962: quick fix to initialise ghost rows of the envelope bathymetry in zgr_sco (domzgr.F90). This avoids undefined e3 errors later on but does not solve the generic problem of unset ghost rows with some mpp decompositions.

Change History (8)

comment:1 Changed 11 years ago by acc

  • Owner changed from NEMO team to acc

comment:2 Changed 11 years ago by acc

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

Implemented the quick fix solution suggested here at revision 3563. Longer term, a more generic solution to the unset ghost rows may be preferable.

comment:3 Changed 8 years ago by nicolasmartin

  • Keywords s-coordinates added; s-coodinate removed

comment:4 Changed 8 years ago by nicolasmartin

  • Keywords nemo_v3_3* added

comment:5 Changed 8 years ago by nicolasmartin

  • Keywords SCO added; s-coordinates removed

comment:6 Changed 8 years ago by nicolasmartin

  • Keywords MPP added; mpp removed

comment:7 Changed 6 years ago by nemo

  • Keywords nemo_v3_3* removed

comment:8 Changed 2 years ago by nemo

  • Keywords OPA v3.3 added
Note: See TracTickets for help on using tickets.