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.
#2114 (Redundant logical keys in lateral diffusion for passive tracers) – NEMO

Opened 4 years ago

Closed 4 years ago

Last modified 8 months ago

#2114 closed Enhancement (fixed)

Redundant logical keys in lateral diffusion for passive tracers

Reported by: lovato Owned by: cetlod
Priority: normal Milestone: 2018 release-4.0
Component: TOP Version: trunk
Severity: minor Keywords: LDF TOP v4.0
Cc:

Description

Context

In NEMO 4.0, the revised lateral diffusion for passive tracers (namtrc_ldf) can be turned off and on, with the latter case using the same parameterizations of the active tracers (namtra_ldf)
It seems very redundant to have in namtrc_ldf two options, one to control on/off ln_trcldf_OFF and one to force use of active tracers setting ln_trcldf_tra

Proposal

I suggest to simply have a single logical to control the lateral diffusion for passive tracers ln_trcldf that turns off and on (this using ln_trcldf_tra settings) and
possibly output a log warning to check ldf parameters in namtra_ldf only when using offline transport.

Commit History (0)

(No commits)

Change History (4)

comment:1 Changed 4 years ago by gm

Hi Tomas,

Thanks for you suggestion.
I agree that ln_trcldf_OFF and ln_trcldf_tra seem to be a duplication, and ln_trcldf_OFF is sufficient to make the choice.
Nevertheless, keeping the two is better.
Indeed, having two logicals, both set to .false. by default (namlist_top_ref), forces a user to make a choice (If he does nothing, the model will stop).
He therefore have to think about it and make a voluntary choice.

This change in the philosophy of mamlist_ref/_cfg is for me an improvement in the v4.0.
A short description is the revised philosophy is given here: New namelist framework

If you agree, please close this ticket.

Gurvan

comment:2 Changed 4 years ago by clevy

  • Owner set to cetlod
  • Status changed from new to assigned

comment:3 Changed 4 years ago by cetlod

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

I agree Gurvan

comment:4 Changed 8 months ago by nemo

  • Keywords v4.0 added
Note: See TracTickets for help on using tickets.