Re: [eigen] Bug in traspose

[Benoît Jacob <jacob@xxxxxxxxxxxxxxx>]

Thanks a lot Ramon! Indeed you're right.

There is no bugzilla, you did the right thing by posting this bug here. 
Tuxfamily does not offer a bugzilla service, and KDE's one has a terrible 
bug-reporting interface IMO so I'm still not sure what to do. For now the 
number of bug reports does not justify opening a bugzilla anyway.

The fact that my unit-tests did not catch this bug is interesting (I can see 
the reason). I will keep that in mind for the next round of unit-tests 
improvements (alpha4?). I am also thinking that QTestLib is not very good for 
us, actually I am already bypassing large parts of it in the main() function, 
so we might get rid of it.

Cheers,

Benoit


On Tuesday 15 January 2008 00:27:09 ollupaC De La Pradera wrote:
> Hello,
>
> I think there's a bug in Transpose.h, at least in current svn revision
> (761491):
>     enum {
>       RowsAtCompileTime = MatrixType::Traits::ColsAtCompileTime,
>       ColsAtCompileTime = MatrixType::Traits::RowsAtCompileTime,
>       MaxRowsAtCompileTime = MatrixType::Traits::MaxRowsAtCompileTime,
>       MaxColsAtCompileTime = MatrixType::Traits::MaxColsAtCompileTime
>     };
> should be:
>     enum {
>       RowsAtCompileTime = MatrixType::Traits::ColsAtCompileTime,
>       ColsAtCompileTime = MatrixType::Traits::RowsAtCompileTime,
>       MaxRowsAtCompileTime = MatrixType::Traits::MaxColsAtCompileTime,
>       MaxColsAtCompileTime = MatrixType::Traits::MaxRowsAtCompileTime
>     };
>
> It's being more difficult that what I initially thought to refresh my
> algebra and numerical methods while trying to understand all that complex
> templates stuff, but I'm still here working even if I don't show up a lot.
>
> By the way, I'm not sure if I have to post it here, or to some bugzilla, or
> just send it to Jacob?
>
> Cheers
>
> Ramon

Attachment: signature.asc
Description: This is a digitally signed message part.