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.
#1461 (incompatibility of key_by and key_lim2) – NEMO

Opened 9 years ago

Closed 9 years ago

Last modified 2 years ago

#1461 closed Bug (fixed)

incompatibility of key_by and key_lim2

Reported by: njourdain Owned by: vancop
Priority: low Milestone: 2015 release-3.6
Component: OCE Version: v3.6
Severity: Keywords: BDY LIM* OPA v3.6
Cc:

Description

Hi,

When I compile NEMO with the following CPP keys, I have no compilation issue :

bld::tool::fppkeys key_bdy key_trabbl key_traldf_c2d key_dynspg_flt key_dynldf_c2d key_ldfslp key_zdftke key_iomput key_mpp_mpi key_lim3

But if I use key_lim2 instead of key_lim3, there is some issue related to the use of both key_bdy and key_lim2 :

/short/e14/ncj561/now/models/NEMO3.6/NEMOGCM/CONFIG/AMU12/BLD/ppsrc/nemo/bdyice_lim.f90(185): error #6404: This name does not have a type, and must have an explicit type. [RSWITCH]

rswitch = 1.0 - MAX( 0.0_wp , SIGN ( 1.0_wp , - at_i(ji,jj) + 0.01 ) ) ! 0 if no ice


compilation aborted for /short/e14/ncj561/now/models/NEMO3.6/NEMOGCM/CONFIG/AMU12/BLD/ppsrc/nemo/bdyice_lim.f90 (code 1)
fcm_internal compile failed (256)
gmake: * [bdyice_lim.o] Error 1

I don't know what is the purpose of this line, but it should probably be specific to key_lim3 (or rswitch should be declared in LIM_SRC_2/ice_2.F90).

cheers
nico

Commit History (0)

(No commits)

Change History (7)

comment:1 Changed 9 years ago by clem

Yes, rswitch was missing in ice_2 to work with BDY. I corrected it.
But be aware that ice boundary conditions are well set up for LIM3 but I am not very sure if it works ok for LIM2

comment:2 Changed 9 years ago by clevy

  • Owner changed from NEMO team to vancop

comment:3 Changed 9 years ago by clem

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

comment:4 Changed 6 years ago by nemo

  • Keywords LIM* added; LIM2 removed

comment:5 Changed 6 years ago by nemo

  • Keywords release-3.6* added

comment:6 Changed 6 years ago by nemo

  • Keywords release-3.6* removed

comment:7 Changed 2 years ago by nemo

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