Opened 6 years ago

Closed 5 years ago

#568 closed task (fixed)

svn merge of dmi_trunk_8.0 [1 h]

Reported by: Ian Culverwell Owned by: Ian Culverwell
Priority: normal Milestone: ROPP9.1 carry over
Component: ropp_io Version: 9.0
Keywords: Cc:

Description

https://trac.romsaf.org/ropp/changeset/5427/ropp_src/branches/dev/Share/dmi_trunk_9.0/ropp_io/tools/ucar2ropp.f90

Original changesets:

svn merge of dmi_trunk_8.0: https://trac.romsaf.org/ropp/changeset/4735/ropp_src/branches/dev/Share/dmi_trunk_8.1/ropp_io/tools/ucar2ropp.f90

Attachments (3)

ucar_test_new.nc​ (502.7 KB ) - added by Ian Culverwell 5 years ago.
ucar_test_new.nc
ucar_testr_old.txt​ (2.7 KB ) - added by Ian Culverwell 5 years ago.
ucar_testr_old.txt
ucar_testr_new.txt​ (2.8 KB ) - added by Ian Culverwell 5 years ago.
ucar_testr_new.txt

Download all attachments as: .zip

Change history (4)

comment:1 by Ian Culverwell, 5 years ago

Resolution: → fixed
Status: new → closed

This change just adds an optional ranchk=ranchk clause to the call to ropp_io_read in ucar2ropp. But testing on out-of-range data (see attached ucar_test_new.nc, which has N -> -N) shows that this will have no effect unless it is accompanied by a corresponding clause in the subsequent call to ropp_io_thin, which range-checks by default. (Unless one is running with the -u option, which bypasses the thinning.)

Include both changes at r6243. As usual, we choose not to document --no-ranchk options.

Closing ticket.

by Ian Culverwell, 5 years ago

Attachment: ucar_test_new.nc​ added

ucar_test_new.nc

by Ian Culverwell, 5 years ago

Attachment: ucar_testr_old.txt​ added

ucar_testr_old.txt

by Ian Culverwell, 5 years ago

Attachment: ucar_testr_new.txt​ added

ucar_testr_new.txt

Note: See TracTickets for help on using tickets.