Changes between Initial Version and Version 1 of Ticket #202


Ignore:
Timestamp:
08/21/08 02:27:51 (17 years ago)
Author:
arango
Comment:

Yes, thank you. The index for erosion flux was renamed from idflux to idefx. I forgot to check other routines for consistency. NST is not a vector. This is a typo.

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #202

    • Property Resolution → Fixed
    • Property Status new → closed

  • Ticket #202 – Description

    initial v1  
    1 A couple of issues revealed when building SHOREFACE with latest revision (225):
     1A couple of issues revealed when building '''SHOREFACE''' with latest revision (225):
    22
    3 In ROMS/Modules/mod_ncparam.F, variable idflux is used but is not declared anywhere. (Should this be declared in mod_sediment.F?)
     3In '''ROMS/Modules/mod_ncparam.F''', variable '''idflux''' is used but is not declared anywhere. (Should this be declared in '''mod_sediment.F'''?)
    44
    5 With ASSUMED_SHAPE turned off, the declaration in ROMS/Nonlinear/sed_settling.F line 108...
     5With '''ASSUMED_SHAPE''' turned off, the declaration in '''ROMS/Nonlinear/sed_settling.F''' line 108...
    66{{{
    77      real(r8), intent(inout) :: settling_flux(LBi:UBi,LBj:UBj,NST(ng))
    88}}}
    9 ...triggers a compiler error, something about "Bad array specification for an explicitly shaped array". The form of this looks OK, so presumably the problem is that NST is not a properly declared integer array.
     9...triggers a compiler error, something about "Bad array specification for an explicitly shaped array". The form of this looks OK, so presumably the problem is that '''NST''' is not a properly declared integer array.