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.
#2063 (Bug in bdydta for ice (3.6_STABLE and dev_merge_2017)) – NEMO

Opened 6 years ago

Closed 6 years ago

Last modified 2 years ago

#2063 closed Defect (fixed)

Bug in bdydta for ice (3.6_STABLE and dev_merge_2017)

Reported by: molines Owned by: clem
Priority: low Milestone: 2018 release-4.0
Component: BDY Version: trunk
Severity: critical Keywords: BDY ICE v4.0
Cc:

Description

Context

When using mono category ice bdy data, and having more than one boundary set involved for ice, the model crash with segmentation fault in ice_var_itd (ex lim_var_itd) routine.

Analysis

The input arguments for ice_var_itd are the ice thickness, snow thickness and ice concentration from the monocategory boundary files. Field index for those 3 fields are respectively

jfld_hti, jfld_hts and jfld_ai, which are set into bdy_dta_init. Unfortunatly, when there are more than one boundary set, those index are overwrtitten, and when used in the call to ice_var_itd, cause the model to crash.

Recommendation

Use array of size jp_bdy to save these 3 index for each boundary set, fixes the bug.

Commit History (2)

ChangesetAuthorTimeChangeLog
9811clem2018-06-19T17:15:12+02:00

debug bdy with sea ice if more than 1 boundary is used. See ticket #2063

9810clem2018-06-19T17:09:31+02:00

debug bdy with sea ice if more than 1 boundary is used. See ticket #2063

Attachments (2)

bdydta.F90 (37.6 KB) - added by molines 6 years ago.
bdydta.F90 fix for dev_merge_2017
bdydta_3.6_STABLE.F90 (40.7 KB) - added by molines 6 years ago.
Fix in bdydta for 3.6_STABLE (@rev 9373)

Download all attachments as: .zip

Change History (7)

Changed 6 years ago by molines

bdydta.F90 fix for dev_merge_2017

Changed 6 years ago by molines

Fix in bdydta for 3.6_STABLE (@rev 9373)

comment:1 Changed 6 years ago by clevy

  • Owner set to clem
  • Status changed from new to assigned
  • Version changed from v3.6 to trunk

comment:2 Changed 6 years ago by clem

In 9810:

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

comment:3 Changed 6 years ago by clem

In 9811:

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

comment:4 Changed 6 years ago by clem

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

comment:5 Changed 2 years ago by nemo

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