Opened 2 years ago

Closed 23 months ago

#2284 closed Bug (fixed)

Bug in NST/agrif_top_update.F90

Reported by: acc Owned by: jchanut
Priority: low Milestone: Unscheduled
Component: AGRIF Version: release-4.0
Severity: minor Keywords:
Cc: Branch review:
MP ready?: Task progress:



Whilst doing some branch work where, amongst other changes, we are changing the trb, trn and tra arrays, I noticed this at line 140 in NST/agrif_top_update.F90:

                          trb(ji,jj,jk,jn) = tsb(ji,jj,jk,jn) &
                                 & + atfp * ( tabres_child(ji,jj,jk,jn) &
                                 &          - trn(ji,jj,jk,jn) ) * tmask(ji,jj,jk)

Presumeably that tsb on the RHS should be a trb?


The code doesn't seem to be active in any SETTE tests but does exist in the trunk and release-4.0.


Change tsb to trb if confirmed by an AGRIF expert.

Commit History (2)





Change History (5)

comment:1 Changed 2 years ago by jchanut

I confirm that's an ugly bug.

comment:2 Changed 2 years ago by clevy

  • Owner changed from systeam to jchanut
  • Status changed from new to assigned

comment:3 Changed 2 years ago by jchanut

In 11078:

comment:4 Changed 2 years ago by jchanut

In 11079:

comment:5 Changed 23 months ago by jchanut

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