Crashes when updating the postion in the status bar

[praveenivp]

Here is the error I got:

Index in position 1 is invalid. Array indices must be positive integers or logical values.
Error in asCursorPosClass/setPosition (line 236)
obj.setPosText([posY, posX],img(posY,posX));
Error in arrShow/updFig (line 3607)
obj.cursor.setPosition(obj.cursor.getPosition(),true);
Error in arrShow/fpResize (line 3374)
obj.updFig;
Error in arrShow>@(src,evnt)obj.fpResize (line 533)
'ResizeFcn',@(src, evnt)obj.fpResize);
Error while evaluating Figure SizeChangedFcn.

The posY is zero when the error occurs. However, the source of the error is in arrShow.m(line 4066). rounding values<1 yields 0. Could you you ceil() instead of round()?

                   ' if arrShow.mouseInsideAxes(position, currAxes);'
                       ' x = round(position(1,1));'
                        'y = round(position(1,2));'
                        'obj.cursor.setPosition([y,x],false);'
                        'break;'
                   'end'

Thank you very much for this toolboox:)

[tsumpf]

Hi praveenivp,
thanks for sharing this issue.

Unfortunately, I haven't used Matlab since ages and do not even own a liscence anymore. So I cannot really test your proposal.
However, I'm sceptical that ceil is a good solution here. I believe the function was supposed to identify and mark the pixel below the cursor. This should give you a blue rectangle around the selected pixel (besides other callback actions). As far as I remember, this used to work quite well back in the days...
If we used ceil instead, I'm afraid that the blue rectangle might be in advance of the cursor when a mouse movement progresses over the center of the current pixel (?).
You could try to change the proposed lines and watch the effect on a low resolution image (say 8 x 8 pixel or so).

Have you already tried if ceil would really solve the issue and is the rectangle still marking the right pixel? I think, we might be better off by just clamping the rounded results to a range <= 1?

Oh, and another thing:
From your error stack it looks as if the problem occurs after resizing the window? Could it be, that you just shrinked the window down to a point where the size of the axes goes below 1?

Anyway. I'm afraid I currently cannot really solve arrayShow issues anymore. You're gonna have to try and fix it by yourself. If you do find a good solution though, I'm happy to look at your pull-requests :-)

Greetz from Luebeck,
Til

[praveenivp]

Hi Til,
This issue arise before resizing the window. I usually get this error when the very first image for some reason doesn't fill the window. when updating the selection counter/ even moving the window, it goes back to normal. It look like some resolution/scaling conflict. I'll pull request if I manage to find that.

You are right. ceil() is not a perfect fix. I just noticed for small images with round(), the cursor can be anywhere in the pixel to select that particular, now with ceil(), it should be in the first half. So, clipping is a better dirty fix.

Thank you once again for sharing your work. I used it extensively during my master thesis.

Best
Praveen