Opened 15 years ago
Closed 15 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.