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.
#2710 (Sign error in lid melting code in topographic melt pond scheme) – NEMO

Opened 3 years ago

Closed 3 years ago

#2710 closed Bug (fixed)

Sign error in lid melting code in topographic melt pond scheme

Reported by: dlivings Owned by: systeam
Priority: low Milestone:
Component: SI3 Version:
Severity: major Keywords: trunk SI3 melt pond
Cc:

Description

There is a sign error in the lid melting code in the topographic melt pond scheme in SI3. In the current version of NEMO/trunk/src/ICE/icethd_pnd.F90 (last changed at [14997]) the error is in line 664. This currently reads

zdvice = MIN( dh_i_sum_2d(ji,jj,jl)*a_ip(ji,jj,jl), v_il(ji,jj,jl) )

but should be

zdvice = MIN( -dh_i_sum_2d(ji,jj,jl)*a_ip(ji,jj,jl), v_il(ji,jj,jl) )

(compare line 597). In the code as currently written, zdvice is always negative or zero, with the result that the test at line 666 always fails and the code that transfers volume from the lid to the pond is never executed. I've confirmed this by adding a statement that writes a log message the first time this piece of code is executed.

Commit History (1)

ChangesetAuthorTimeChangeLog
15243clem2021-09-10T11:52:16+02:00

trunk: solve ticket #2710

Change History (2)

comment:1 Changed 3 years ago by clem

In 15243:

Error: Failed to load processor CommitTicketReference
No macro or processor named 'CommitTicketReference' found

comment:2 Changed 3 years ago by clem

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.