Changes between Initial Version and Version 1 of Ticket #178


Ignore:
Timestamp:
08/05/08 18:30:30 (16 years ago)
Author:
arango
Comment:

Oh yes, good catch!!! This is a shared-memory bug and a serial with partition bug. The distributed-memory algorithm is correct. I missed this one because we usually initialize the model with zero free-surface. The problem arises when the free-surface is not zero like in restarts or forecasting.

The solution was to split ini_fields into two routines (ini_zeta and ini_fields) so we can have the parallel synchronization. The TLM, RPM, and ADM were also corrected for consistency. Recall that we can only run the ADM in distributed memory.

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #178

    • Property Resolution → Fixed
    • Property Status new → closed

  • Ticket #178 – Description

    initial v1  
    1 When running with multiple tiles in serial mode, Zt_avg1 gets set on one tile, set_depth_tile gets called and Hz is recomputed internal to that tile. Then, Hz is used (including one ghost cell) to compute the vertical sums of u and v for resetting ubar, vbar. The ghost cells aren't up to date and  that one edge of ubar, vbar is not quite the same as it should be.
     1When running with multiple tiles in serial mode, '''Zt_avg1''' gets set on one tile, '''set_depth_tile''' gets called and '''Hz''' is recomputed internal to that tile. Then, '''Hz''' is used (including one ghost cell) to compute the vertical sums of '''u''' and '''v''' for resetting '''ubar''', '''vbar'''. The ghost cells aren't up to date and  that one edge of '''ubar''', '''vbar''' is not quite the same as it should be.