Thread Subject:

Bug in mat2cell?

Hello

X=rand(2,2,2);
C=mat2cell(X,2,[1 1],[1 1]);

The last instruction produces the following error message,
which I think is a bug.

??? Index exceeds matrix dimensions.

Error in ==> mat2cell at 227
        c{cind} = x(tref{:});

My MATLAB version is 7.3.0.267 (R2006b).

Comments?

Bruno

See above

I look at the Mathworks's mat2cell code, and at the line
#198, the array ref is updated under certain condition:

...
if (inc~=1) && ~(cellsize(1)==1)
    [ref{1:inc-1}] = deal(refstart{1:inc-1});
    refstatic(1:inc-1) = 0;
end
...

The condition ~(cellsize(1)==1) seems to cause the Bug. If I
remove it from the test, as following:

...
if (inc~=1)
    [ref{1:inc-1}] = deal(refstart{1:inc-1});
    refstatic(1:inc-1) = 0;
end
...

it works. I'm not sure what is the test on cellsize(1) is
for, and I'm not sure I would introduce a mess in mat2cell
function by removing it. Hope someone at mathworks could
answer the question.

Bruno

In article <fibjt3$64b$1@fred.mathworks.com>,
Bruno Luong <brunoluong@yahoo.com> wrote:

>X=rand(2,2,2);
>C=mat2cell(X,2,[1 1],[1 1]);

>The last instruction produces the following error message,
>which I think is a bug.

>??? Index exceeds matrix dimensions.

>My MATLAB version is 7.3.0.267 (R2006b).

No error is produced in R2007a.
--
   "Okay, buzzwords only. Two syllables, tops." -- Laurie Anderson

roberson@ibd.nrc-cnrc.gc.ca (Walter Roberson) wrote in message
>
> No error is produced in R2007a.

Thanks!

"Bruno Luong" <brunoluong@yahoo.com> wrote in message
<fibp2d$sff$1@fred.mathworks.com>...
> I look at the Mathworks's mat2cell code, and at the line
> #198, the array ref is updated under certain condition:
>
> ...
> if (inc~=1) && ~(cellsize(1)==1)
> [ref{1:inc-1}] = deal(refstart{1:inc-1});
> refstatic(1:inc-1) = 0;
> end
> ...
>
> The condition ~(cellsize(1)==1) seems to cause the Bug. If I
> remove it from the test, as following:
>
> ...
> if (inc~=1)
> [ref{1:inc-1}] = deal(refstart{1:inc-1});
> refstatic(1:inc-1) = 0;
> end
> ...
>

Ditto, the test ~(cellsize(1)==1) is removed from the source
code of mat2cell in R2007b.

It seems to be a right fix for customer who still own older
version of MATLAB. I would expect Mathwork would react to my
question, but they seem to ignore it.

Bruno