Opened 8 years ago
Closed 8 years ago
#743 closed bug (Fixed)
Incomplete pre-processer conditions in def_diags.F — at Version 1
Reported by: | fgrosse | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | Release ROMS/TOMS 3.7 |
Component: | Nonlinear | Version: | 3.7 |
Keywords: | Cc: |
Description (last modified by )
I think, in lines 626 and 701 of def_diags.F the pre-processor conditions should read as:
#if defined DIAGNOSTICS_BIO && defined BIO_FENNEL
Currently, they only read as:
#ifdef BIO_FENNEL
This can cause "undefined variable" errors for iDbio2 and iDbio3 during compilation when other diagnostics but not DIAGNOSTICS_BIO are enabled in combination with BIO_FENNEL.
Yes, thank you for bringing this problem to my attention. I cleaned the logic in def_diags.F, set_diags.F, and wrt_diags.F. I also added the 2D biological diagnotics for HYPOXIA_SRM. It was not included in the above files.
Also removed apostrophes in fennel.h documentation to suppress the warnings from ifort during compilation. I also cleaned a DIAGNOSTIC_BIO bug in hypoxia_srm.h. We don't need the 3D diagnostics arrays in this simple model.
Fixed OMP directives in initial.F, ad_initial.F, rp_initial.F, and tl_initial.F when the 4D-Var algorithms are activated. This bug is of no consequence since we cannot run any of the adjoint-based algorithms in shared-memory (OpenMP).
Change History (1)
comment:1 by , 8 years ago
Description: | modified (diff) |
---|---|
Resolution: | → Fixed |
Status: | new → closed |