[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Pkg-octave-devel] Bug#394982: octave2.9: Crashes on "x(:,

From: Kim Hansen
Subject: Re: [Pkg-octave-devel] Bug#394982: octave2.9: Crashes on "x(:,
Date: Thu, 26 Oct 2006 12:15:36 +0200

On 10/26/06, David Bateman <address@hidden> wrote:

Its really really difficult to change. It was a design choice I made
early in the Sparse<T>::assign and Sparse<T>::assign1 functions.
Basically in most cases when you'll add data to this manner to a sparse
matrix it will need to be enlarged. Even if it doesn't need to be
enlarged, there is a good chance that you might add the data in the
middle of the existing elements, so much movement of data would be
needed. So the assign functions just create a new sparse matrix, fill it
from the old and new matrices and be done with it.

Changing would require a complete rewrite of the assign function, and
also I feel won't make much sense unless you also relaxed the
requirement in a sparse column that the elements are sorted, which I
also made to simplify a lot of code.. I personally don't see a reason to
go to this effort when the workaround is fairly simple...

My idea was just to add support for extending the sparse matrix at the
end without breaking the sorting of the elements, I am not even sure
that the compressed format works with unsorted data.

Kim Hansen
Vadgårdsvej 3, 2.tv
2860 Søborg
Fastnet: 3956 2437  --  Mobil: 3091 2437

reply via email to

[Prev in Thread] Current Thread [Next in Thread]