Changes between Initial Version and Version 2 of Ticket #645


Ignore:
Timestamp:
10/15/14 16:11:23 (10 years ago)
Author:
arango
Comment:

Yes, thank for reporting these problems. Now, we have:

    case {Vname.hraw}
      if ~any(V.Variables(n).Size == 0),
        got.hraw=true;
      end

and

if (got.spher),
  spherical=nc_read(ncfile,Vname.spher);
  if (ischar(spherical)),
    if (spherical == 'T' || spherical == 't')
      spherical = true;
    else
      spherical = false;
    end
  end
end

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #645

    • Property ResolutionFixed
    • Property Status newclosed
  • Ticket #645 – Description

    initial v2  
     1'''editmask.m''' throws some errors due to conflicts with new or incomplete grid files when variable '''hraw''' has not been loaded.
    12
    2 editmask.m throws some errors due to conflicts with new or incomplete grid files.
     3These can be solved by making a few changes to '''read_mask.m'''
    34
    4 These can be solved by making a few changes to read_mask.m
     5A grid file may have a variable '''hraw''' that has not been filled with data. This will cause an error in '''editmask'''. I suggest test for the case hraw has an empty dimension:
    56
    6 A grid file may have a variable hraw that has not been filled with data. This will cause an error in editmask. I suggest test for the case hraw has an empty dimension:
    7 
     7{{{
    88case {Vname.hraw}
    99  got.hraw=true;
     
    1111     got.hraw=false;
    1212  end
     13}}}
    1314
    14 
    15 read_mask presently only checks for "spherical" in the form of a character, and always returns false even when spherical=1 (numeric).
     15'''read_mask''' presently only checks for '''spherical''' in the form of a character, and always returns false even when spherical=1 (numeric).
    1616Make this robust and compatible with both conventions with:
    1717
     18{{{
    1819 if (got.spher)
    1920    spher=nc_read(ncfile,Vname.spher);
     
    2627    end
    2728  end
     29}}}