[Insight-developers] ITK 3.10 : Turning ITK_IMAGE_BEHAVES_AS_ORIENTED_IMAGE = ON by default

[Luis Ibanez]

Hi Tom,


Excellent question.


                Fortunately the answer is:  Yes !


We have been testing the toolkit with this flag ON for several weeks
now in specific machines. (Redwall, Zion, Camelot), and they have been
producing green builds.


As further confirmation, we turned the flag ON, yesterday, so today's
Nightly builds are all (unless they explicitly turned the flag OFF)
reporting results by having the itk::Image behave just like the
itk::OrientedImage.


Unfortunately, I broke the build while fixing coding style violations
in Code/Common, so we will have to wait for tomorrow's Dashboard to
have a clear picture of the effect of turning the flag ON in all the
Nightly submissions.


Point taken about the Software Guide, and update of the book is overdue.


   Regards,


       Luis


----------------------
Tom Vercauteren wrote:
> Hi Luis,
> 
> It's great to see ITK becoming more consistent with respect to image
> orientation!
> 
> Out of curiosity, I am wondering whether the correct behavior of the
> registration (parametric or not) with oriented images is already being
> tested by the unit tests?
> 
> Also I think that once the orientation framework becomes stable, it
> would be worth updating the ITK user guide to explain how orientation
> information is used (or should be) in ITK.
> 
> Just my two cents.
> 
> Regards,
> Tom
> 
> On Thu, Oct 16, 2008 at 7:42 PM, Luis Ibanez <luis.ibanez at kitware.com> wrote:
> 
>>Hi Leila,
>>
>>Thanks for the feedback.
>>
>>BTW,
>>to make the fix complete we will also turn ON by default the
>>CMake variable
>>
>>           ITK_USE_ORIENTED_IMAGE_DIRECTION
>>
>>
>>This flag is used by gradient computation classes, to make the
>>gradient orientations consistent with the image orientation.
>>
>>This fixes an long standing bug in the registration framework.
>>
>>
>>Regards,
>>
>>
>>  Luis
>>
>>
>>------------------------
>>Leila Baghdadi wrote:
>>
>>>Hi Luis,
>>>
>>>It sounds like a great plan. If the direction cosines are taken into
>>>account the image is read ( this is the behavior of all the software
>>>that we have in our center), then there is nothing to worry about.
>>>
>>>I also like the idea that the cmake flag gets rid of alot of confusion
>>>for the developers.
>>>
>>>Leila
>>>
>>>On Thu, 2008-10-16 at 09:03 -0400, Luis Ibanez wrote:
>>>
>>>
>>>>In preparation for the release of ITK 3.10,
>>>>
>>>>
>>>>We are proposing to turn ON by default the CMake flag:
>>>>
>>>>
>>>>          ITK_IMAGE_BEHAVES_AS_ORIENTED_IMAGE
>>>>
>>>>
>>>>When this flag is ON, the itk::Image provides a behavior identical
>>>>to the itk::OrientedImage. Essentially taking the image direction
>>>>into account when converting Indices to Physical Points and back.
>>>>
>>>>
>>>>The itk::OrientedImage should be retained for backward compatibility,
>>>>despite of becoming redundant at this point.
>>>>
>>>>
>>>>The rational for turning this flag ON by default is that this is
>>>>the correct behavior to expect from a medical image, while ignoring
>>>>the direction when converting indices to points is actually a bug.
>>>>
>>>>
>>>>The buggy behavior will still be available as an option, to those
>>>>who require a backward compatible frame (e.g. applications that
>>>>have already worked around the previous bug). In those cases you
>>>>should simply turn the flag OFF, in order to get the old behavior.
>>>>
>>>>
>>>>Please let us know if you have any questions or concerns.
>>>>
>>>>
>>>>  Thanks
>>>>
>>>>
>>>>     Luis
>>>>
>>>>
>>>>
>>>>-------------------------
>>>>Karthik Krishnan wrote:
>>>>Perhaps that
>>>>
>>>>
>>>>>could be on the list of things ToDo whenever the next lengthy
>>>>>discussion on Orientation comes up in ITK. I think we have one
>>>>>every month. :)
>>>>>
>>>>>Thanks
>>>>>
>>>>
>>>>_______________________________________________
>>>>Insight-developers mailing list
>>>>Insight-developers at itk.org
>>>>http://www.itk.org/mailman/listinfo/insight-developers
>>>
>>>
>>>
>>_______________________________________________
>>Insight-developers mailing list
>>Insight-developers at itk.org
>>http://www.itk.org/mailman/listinfo/insight-developers
>>
> 
>