Opened 14 years ago

Closed 14 years ago

#468 closed bug (Fixed)

IMPORTANT: Corrected parallel bug in uv3dmix4_geo.h and friends

Reported by: arango Owned by: arango
Priority: major Milestone: Release ROMS/TOMS 3.4
Component: Nonlinear Version: 3.4
Keywords: Cc:

Description

Revisited routine uv3dmix4_geo.h and its TLM, RPM, and ADM versions again to correct parallel bugs. The NLM version now works in shared-memory and serial with partitions. The biharmonic operator for the stress tensor along geopotentials is very ticky!!! The good news is that I don't have to revisit these routines again. I already worked all the logic for nesting.

In src:ticket:466 I fixed the problems in distributed-memory but I ignored shared-memory and serial with partitions.

Also corrected and adjoint bug in ad_uv3dmix4_geo.h. I also updated the TLM, RPM, and ADM versions to include code for options VISC_3DCOEF and UV_U3ADV_SPLIT.

The adjoint version of these routines now passes the operator symmetry sanity checks for WC13.

Change History (1)

comment:1 by arango, 14 years ago

Resolution: Fixed
Status: newclosed
Note: See TracTickets for help on using tickets.