Opened 17 years ago
Closed 17 years ago
#213 closed bug (Fixed)
Typo in formula for SST nudging/OI — at Version 1
Reported by: | mathieu | Owned by: | arango |
---|---|---|---|
Priority: | major | Milestone: | Release ROMS/TOMS 3.2 |
Component: | Nonlinear | Version: | 3.2 |
Keywords: | Cc: |
Description (last modified by )
In ROMS/Nonlinear/set_data.F line 1022 please replace
cff1=Zr*cff+pcoef_SST(order,ng)
by
cff1=Zr*cff1+pcoef_SST(order,ng)
The error results in incorrect extrapolation of sea surface temperature.
Also, in ROMS/Modules/mod_scalar.F the variables assi, pert, Prms and Prms are defined but not used anywhere.
This cause a problem because their allocation is not done correctly in line 1635-1637. After the loop on ng in line 1551-1584, the variable ng takes the value 2 (using ifort 10.1). The result is that the allocation is not correct and is detected with "-check bounds".
When running with no "-check bounds" with either ASSIMILATION_SST or NUDGING_SST and the error polynomial being 1 so that the temperature is not modified, the result is a blowup of the model caused by this memory error.
Change History (1)
comment:1 by , 17 years ago
Description: | modified (diff) |
---|---|
Resolution: | → Fixed |
Status: | new → closed |
Summary: | Incorrect formulas → Typo in formula for SST nudging/OI |
Done. Thank you. I collapsed your two tickets into one. There is not need to submit multiple tickets. We appreciate a single ticket if the issues are related.