Opened 14 years ago

Closed 14 years ago

#467 closed bug (Fixed)

IMPORTANT, corrected a problem in mod_netcdf.F

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

Description

Corrected a problem in mod_netcdf.F when determining the size (Asize) of the array to be read. The value Asize is used to land/sea masking values (Aspval) or adding offset (Aoffset) when appropriate. The assignment of Asize depends if the optional arguments start and total are present in the routine family netcdf_get_fvar_xx.

This may also explain complains about why the _FillValue (>=1E+35) values were not removed completed from the boundary conditions during reading. We were not scanning the full compact 1D array do to the incorrect value of Asize.

I finally have time to look at this problem in the debugger and test with different compilers.

Many thanks to Pierre St-Laurent for reporting this problem and for his persistence to get to the bottom of this issue.

I also fixed a bug in interp_float.F. Around line 198, we need to have instead:

j2=MIN(MAX(Jr+1,1),Mm(ng)+1)

I also corrected few typos in the documentation. I have been carefully revising ROMS code for the last month or so.

Change History (1)

comment:1 by arango, 14 years ago

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