Opened 2 years ago

Closed 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
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 2 years ago.
bdydta.F90 fix for dev_merge_2017
bdydta_3.6_STABLE.F90 (40.7 KB) - added by molines 2 years ago.
Fix in bdydta for 3.6_STABLE (@rev 9373)

Download all attachments as: .zip

Change History (6)

Changed 2 years ago by molines

bdydta.F90 fix for dev_merge_2017

Changed 2 years ago by molines

Fix in bdydta for 3.6_STABLE (@rev 9373)

comment:1 Changed 2 years ago by clevy

  • Owner set to clem
  • Status changed from new to assigned
  • Version changed from release-3.6 to trunk

comment:2 Changed 2 years ago by clem

In 9810:

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

comment:3 Changed 2 years ago by clem

In 9811:

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

comment:4 Changed 2 years ago by clem

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