[Insight-developers] Problems with SymmetricSecondRankTensor and DiffusionTensors3D

[Casey Goodlett]

Luke,

I put up a blank page on the ITK wiki
http://www.itk.org/Wiki/ITK_Working_Group_DTI.

I had a few other changes I'd like to see in the DTI infrastructure as
well.  So it would be helpful to have a central location for
discussion.  Can you add your proposed changes to this pages?

Thanks

Casey Goodlett



On Fri, Apr 24, 2009 at 11:27 AM, Luke Bloy <luke.bloy at gmail.com> wrote:
> 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
>>
>