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.
#1618 (bug in code (incomplete code?) if ln_mask_file = .true. in bdyini.F90) – NEMO

Opened 8 years ago

Closed 7 years ago

#1618 closed Bug (fixed)

bug in code (incomplete code?) if ln_mask_file = .true. in bdyini.F90

Reported by: jamesharle Owned by: jamesharle
Priority: low Milestone:
Component: OCE Version: trunk
Severity: Keywords: BDY
Cc:

Description

I've been trying a few regional simulations using bdy_msk to define my region. After looking at the restart files it appears that even the domain outside the bdy_msk is 'active'. Looking at bdyini.F90 bdytmask appears to be overwritten (at ln 1161 with ssmask) after the bdy_msk is read in. There are also several new mask arrays defined in dommsk.F90 that are not updated here - maybe they don't need to be - but I couldn't correct the issues I had running with a bdy_msk. I thought the cleanest way (rather than updating a bunch of mask arrays after they'd been calculated - and also liable to change in future versions) was just to perform a single update to the tmask in dommsk using bdy_msk and remove all the mask update code in bdyini.F90 (which solved my problems).. Good, bad or ugly? Attached are two patches at revision 5840.

Commit History (1)

ChangesetAuthorTimeChangeLog
6946jamesharle2016-09-23T12:41:24+02:00

Address ticket #1618: bdy_msk not masking

Attachments (2)

dommsk.patch (4.1 KB) - added by jamesharle 8 years ago.
bdyini.patch (1.7 KB) - added by jamesharle 8 years ago.

Download all attachments as: .zip

Change History (5)

Changed 8 years ago by jamesharle

Changed 8 years ago by jamesharle

comment:1 Changed 8 years ago by jamesharle

  • Owner changed from NEMO team to jamesharle

comment:2 Changed 8 years ago by nicolasmartin

  • Keywords BDY added; bdy removed

comment:3 Changed 7 years ago by jamesharle

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

This code was added in the 2016 merge.

Note: See TracTickets for help on using tickets.