[Insight-users] Error in the file itkCovarianceSampleFilterTest.cc

Karthik Krishnan karthik.krishnan at kitware.com
Sun Dec 27 19:06:18 EST 2009


Nice catch. Committed. thanks

Insight/Testing/Code/Review/Statistics/itkCovarianceSampleFilterTest.cxx
new revision: 1.3



On Sun, Dec 27, 2009 at 6:41 PM, Ricardo Ferrari
<rjf.araraquara at gmail.com> wrote:
>  I would like to point out an error in the itkCovarianceSampleFilterTest.
>    cc located in the folder ITK/Insight/Testing/Code/
> Review/Statistics
>    Please notice that in the current cvs repository code we have
>      if ( ( vcl_fabs( mean[0] - mean[0]) > epsilon )  ||
>           ( vcl_fabs( mean[1] - mean[1]) > epsilon)  ||
>           ( vcl_fabs( mean[2] - mean[2]) > epsilon) )
>        {
>        std::cerr << "Mean parameter value retrieved using GetMean() and the
>    decorator\
>                      are not the same:: " <<  mean << "," << mean2 <<
>    std::endl;
>        return EXIT_FAILURE;
>        }
>    The correct should be
>      if ( ( vcl_fabs( mean[0] - mean2[0]) > epsilon )  ||
>           ( vcl_fabs( mean[1] - mean2[1]) > epsilon)  ||
>
> - Ignored:
>           ( vcl_fabs( mean[2] - mean2[2]) > epsilon) )
>        {
>        std::cerr << "Mean parameter value retrieved using GetMean() and the
>    decorator\
>                      are not the same:: " <<  mean << "," << mean2 <<
>    std::endl;
>        return EXIT_FAILURE;
>        }
>
> _____________________________________
> 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://www.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-users
>
>



-- 
_________________________________
Karthik Krishnan
R&D Engineer,
Kitware Inc.
Ph: +1 5188814919, +91 9538477060


More information about the Insight-users mailing list