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.
#2172 (bug in case of direct sea ice assimilation) – NEMO

Opened 4 years ago

Closed 4 years ago

Last modified 8 months ago

#2172 closed Bug (fixed)

bug in case of direct sea ice assimilation

Reported by: mathiot Owned by: mathiot
Priority: low Milestone: 2018 release-4.0
Component: ASM Version: trunk
Severity: minor Keywords: DA ICE v4.0
Cc:

Description

Context

In case of direct assimilation of sea ice, zincwgt is used before it is initialised.

Analysis

After discussion with Dan Lea, the main issue is that in direct assimilation case, we should not use an IAU weight (zincwgt) at all.

Fix

Replace in asminc.F90 in the sea ice direct assimilation case:

            ! Nudge sea ice depth to bring it up to a required minimum depth
            WHERE( zseaicendg(:,:) > 0.0_wp .AND. hm_i(:,:) < zhicifmin )
               zhicifinc(:,:) = (zhicifmin - hm_i(:,:)) * zincwgt
            ELSEWHERE
               zhicifinc(:,:) = 0.0_wp
            END WHERE

by:

            ! Nudge sea ice depth to bring it up to a required minimum depth
            WHERE( zseaicendg(:,:) > 0.0_wp .AND. hm_i(:,:) < zhicifmin )
               zhicifinc(:,:) = (zhicifmin - hm_i(:,:))
            ELSEWHERE
               zhicifinc(:,:) = 0.0_wp
            END WHERE

Commit History (1)

ChangesetAuthorTimeChangeLog
10355mathiot2018-11-21T18:31:21+01:00

Fix #2172

Change History (2)

comment:1 Changed 4 years ago by mathiot

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

In 10355:

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

comment:2 Changed 8 months ago by nemo

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