syntax error in ge_colorbar: initializes cMap to a string instead of an array

GoogleCodeExporter commented 9 years ago

In my version of  "ge_colorbar", I changed the following line of code: 
OLD:                cMap = 'jet';
NEW:                cMap = jet;

My version had an error trying to read the string "jet" instead of the 
1x64 array of color values.

What steps will reproduce the problem?
1. maybe running the ge_colorbar demo
2.
3.

What is the expected output? What do you see instead?
I got an error message: 
??? Error using ==> interp1 at 185
There should be at least two data points.

Error in ==> ge_colorbar at 94
    YIRed = interp1(X,YRed,fM);

What version of the product are you using? On what operating system?
Windows XP

Please provide any additional information below.

Original issue reported on code.google.com by selt...@gmail.com on 6 May 2008 at 1:07

GoogleCodeExporter commented 9 years ago

I had the same error, but solved the problem otherwise. The problem was imho 
that in 
Line 43 and 44 of ge_colorbar

    coords(:,1) = x(:);
    coords(:,2) = y(:);

the variable winter from demo_ge_colorbar was a string. I added

cMap = colormap(cMap);

and now it works.

Full code:
function [output] = ge_colorbar(x,y,data,varargin)
% Reference page in help browser: 
% 
% <a href="matlab:web(fullfile(ge_root,'html','ge_colorbar.html'),'-
helpbrowser')">link</a> to html documentation
% <a 
href="matlab:web(fullfile(ge_root,'html','license.html'),'-helpbrowser')">show 
license statement</a> 
%

AuthorizedOptions = authoptions( mfilename );

if isempty(data)
    error(['Empty data array passed to function ', mfilename])
end
%% Assign default parameter values:
           altitude = 1.0;
       altitudeMode = 'clampToGround';
    cBarBorderWidth = 1;
      cBarFormatStr = '%g';
%            cLimHigh = max(data(:));
%             cLimLow = min(data(:));
               cMap = colormap('jet');
            extrude = 1;      
                 id = 'colorbar';
              idTag = 'id';
        msgToScreen = false;
               name = 'ge_colorbar';
           numUnits = 15;
      timeSpanStart = ' ';
       timeSpanStop = ' ';
          timeStamp = ' ';
          labels = {};
         visibility = 1;

parsepairs %script that parses Parameter/value pairs.

if msgToScreen
   disp(['Running ' mfilename '...']) 
end

if( isempty( x ) || isempty( y ))
    error('empty coordinates passed to ge_colorbar(...).');
else
    coords(:,1) = x(:);
    coords(:,2) = y(:);
    cMap = colormap(cMap);
end

if ~(isequal(altitudeMode,'clampToGround')||...
   isequal(altitudeMode,'relativeToGround')||...
   isequal(altitudeMode,'absolute'))

    error(['Variable ',39,'altitudeMode',39, ' should be one 
of ' ,39,'clampToGround',39,', ',10,39,'relativeToGround',39,', 
or ',39,'absolute',39,'.' ])

end 

if exist('nanValue','var')&&~isnan(nanValue)
    data(data==nanValue)=NaN;
end
if ~exist('cLimHigh','var')
    cLimHigh = max(data(:));
end
if ~exist('cLimLow','var')
    cLimLow = min(data(:));
end

html = ['<TABLE border=' num2str(cBarBorderWidth) ' bgcolor=#FFFFFF>',10];

X = linspace(0,1,size(cMap,1))';
YRed = cMap(:,1);
YGreen = cMap(:,2);
YBlue = cMap(:,3);

    if ~isempty(labels)

        if (length(labels) ~= 1) && (length(labels) ~= numUnits)
            error(['dataLabels array is of improper length.']);   
        end

        if length(labels) == 1
            for r=1:numUnits
                labels{r} = labels{1};
            end
        end  
    end

for k=numUnits:-1:1

    fL = (k-1)/numUnits;
    fM = (k-0.5)/numUnits;
    fH = k/numUnits;

    YIRed = interp1(X,YRed,fM);
    YIGreen = interp1(X,YGreen,fM);
    YIBlue = interp1(X,YBlue,fM);
    TDbgColor = conv2colorstr(YIRed,YIGreen,YIBlue);

    if ~isempty(labels)

        html = [html,'<TR><TD bgcolor=#',TDbgColor, '> </TD>',...
            '<TD bgcolor=#FFFFFF>',...
            num2str(cLimLow+fL*(cLimHigh-cLimLow),cBarFormatStr),...
            ' to ',...
            num2str(cLimLow+fH*(cLimHigh-cLimLow),cBarFormatStr),...
            '<TD>',labels{k},'</TD>','</TR>',10];

    else

        html = [html,'<TR><TD bgcolor=#',TDbgColor, '> </TD>',...
            '<TD bgcolor=#FFFFFF>',...
            num2str(cLimLow+fL*(cLimHigh-cLimLow),cBarFormatStr),...
            ' to ',...
            num2str(cLimLow+fH*(cLimHigh-cLimLow),cBarFormatStr),...
            '</TR>',10];
    end

end
html = [html,'</TABLE>'];

coords(:,3) = altitude;
id_chars = [ idTag '="' id '"' ];
poly_id_chars = [ idTag '="poly_' id '"' ];
name_chars = [ '<name>',10, name,10, '</name>',10 ];
description_chars = [ '<description>',10,'<![CDATA[' html ']]
>',10,'</description>',10 ];
visibility_chars = [ 
'<visibility>',10,int2str(visibility),10,'</visibility>',10 ];
% lineColor_chars = [ '<color>',10, LineColor,10, '</color>',10 ];
% polyColor_chars = [ '<color>',10, PolyColor ,10,'</color>',10 ];
% lineWidth_chars= [ '<width>',10, num2str(LineWidth, '%.2f') ,10,'</width>',10 
];
altitudeMode_chars = [ '<altitudeMode>',10, altitudeMode,10, 
'</altitudeMode>',10 ];
% snippet_chars = [ '<Snippet>',10, Snippet ,10,'</Snippet>' ];
extrude_chars = [ '<extrude>',10, int2str(extrude),10, '</extrude>' ];

if timeStamp == ' '
    timeStamp_chars = '';
else
    timeStamp_chars = [ '<TimeStamp><when>' timeStamp '</when></TimeStamp>',10 ];
end

if timeSpanStart == ' '
    timeSpan_chars = '';
else
    if timeSpanStop == ' ' 
        timeSpan_chars = [ '<TimeSpan><begin>' 
timeSpanStart '</begin></TimeSpan>',10 ];
    else
        timeSpan_chars = [ '<TimeSpan><begin>' timeSpanStart '</begin><end>' 
timeSpanStop '</end></TimeSpan>',10 ];    
    end

end

header=['<Placemark ',id_chars,'>',10,...
    name_chars,10,...
    timeStamp_chars,...
    timeSpan_chars,...
    visibility_chars,10,...
    description_chars,...
    '   <Style>',...
        '<IconStyle>',...
            '<scale>1</scale>',...
            '<Icon>',...

    '<href>http://maps.google.com/mapfiles/kml/shapes/donut.png</href>',...
            '</Icon>',...
        '</IconStyle>',...
        '<ListStyle>',...
        '</ListStyle>',...
    '</Style>',...
  '<Point ',poly_id_chars,'>',10,...
    altitudeMode_chars,...
    extrude_chars,...
    '<tessellate>',10,'1',10,'</tessellate>',10,...
    '<coordinates>',10];

footer = ['</coordinates>',10,...
    10,'</Point>',10,...
    10,'</Placemark>',10];  

output = '';

if ~isnan(coords)
    coordinates = conv_coord(coords);
    output = [ header, coordinates, footer ]; 
end

if msgToScreen
   disp(['Running ' mfilename '...Done']) 
end
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
%% LOCAL FUNCTIONS START HERE
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%

function s = conv_coord(M)
%% conv_coord(M)
% helper function to conver decimal degree coordinates into character array
s=[];

for r=1:size(M,1)
    for c=1:size(M,2)
        s = [s,sprintf('%.6f',M(r,c))];
        s = trim_trail_zero(s);
        if c==size(M,2)
            s=[s,10];
        else
            s=[s,','];          
        end
    end
end

%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%

function s_out = trim_trail_zero(s_in)
%   helper function meant to trim trailing character zeros from a character
%   array.

dig = 1;
L = length(s_in);
last_char = s_in(L);

cont = true;

while (strcmp(last_char,'0') || strcmp(last_char,'.')) && cont==1
    if strcmp((last_char),'.')
        cont = 0;
    end
    s_in = s_in(1:L-dig);
    last_char = s_in(length(s_in));
    dig = dig+1;
end

s_out = s_in;

%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%

function S = conv2colorstr(R,G,B)
% Please note that this conv2colorstr is different from that in
% ge_imagesc. This one writes HTML formatted hexadecimal 
% colorstrings, ge_imagesc() writes KML formatted colorstr.

S='000000';

hexR = dec2hex(round(R*255));
hexG = dec2hex(round(G*255));
hexB = dec2hex(round(B*255));

LR = length(hexR);
LG = length(hexG);
LB = length(hexB);

S(3-LR:2)=hexR;
S(5-LG:4)=hexG;
S(7-LB:6)=hexB;

Original comment by wegas...@gmail.com on 26 May 2008 at 9:48

GoogleCodeExporter commented 9 years ago

Original comment by jhspaaks@hotmail.com on 9 Jan 2009 at 7:09

Changed state: Fixed

mgdevereux / googleearthtoolbox

syntax error in ge_colorbar: initializes cMap to a string instead of an array #10