[Insight-developers] cm != mm typo?

[Luis Ibanez]

Kevin,

Yes, it is clearly a bug:

The full section should be:

    if ( metaDataStr == "um. " )
      {
      m_MetaImage.DistanceUnits(MET_DISTANCE_UNITS_UM);
      }
    else if ( metaDataStr == "mm. " )
      {
      m_MetaImage.DistanceUnits(MET_DISTANCE_UNITS_MM);
      }
    else if ( metaDataStr == "cm. " )
      {
      m_MetaImage.DistanceUnits(MET_DISTANCE_UNITS_CM);
      }
    else


and to think that the Mars Climate Orbiter
was lost due to a bug like this one...             :-/

http://en.wikipedia.org/wiki/Mars_Climate_Orbiter#Communications_loss



Fixed now in Gerrit patch:
http://review.source.kitware.com/#change,1360


I tried to assign you as reviewer, but couldn't find you.

Have you created your Gerrit account  ?


     Thanks


           Luis


--------------------------------------------
On Wed, Apr 6, 2011 at 12:16 PM, Kevin H. Hobbs <hobbsk at ohio.edu> wrote:
> Is this a typo?
>
> ./Modules/IO/Meta/src/itkMetaImageIO.cxx line 974
>
> else if ( metaDataStr == "cm. " )
>  {
>  m_MetaImage.DistanceUnits(MET_DISTANCE_UNITS_MM);
>  }
>
>
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
>
> Kitware offers ITK Training Courses, for more information visit:
> http://kitware.com/products/protraining.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
>
>