[Insight-developers] Problems with SymmetricSecondRankTensor and DiffusionTensors3D

[Luke Bloy]

Hi Casey,

Done. My patch doesn't address the pre/Post multiply issues, because as 
you pointed out the methods are just wrong. I also can't see a reason 
(other then to perform rotation etc) that I would want to do these 
multiplications.

Also in this case I'm not sure that keeping backward compatibility is 
the correct thing to do. I know that if it was my research, I'd rather 
have to fix my code, then to continue to get incorrect results without  
knowing it.

-Luke

Casey Goodlett wrote:
> Hi Luke,
>
> With regards to item 3, could you break out that part and attach it to
> the bug report?  We still have to deal with the issue of backwards
> compatibility.  Hopefully we can do something to change the current
> method, because it cannot be implemented correctly with the current
> return type.
>
> I'd volunteer to help review other parts of the patch as I'm familiar
> with the topic.
>
> Thanks
>
> Casey Goodlett
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html
>
> Please keep messages on-topic and check the ITK FAQ at: http://www.itk.org/Wiki/ITK_FAQ
>
> Follow this link to subscribe/unsubscribe:
> http://www.itk.org/mailman/listinfo/insight-developers
>