[Community] [Insight-developers] FastChamferDistanceImageFilterTest failure with Intel ICPC compiler?

[Williams, Norman K]

Completely unrelated.  I tried the old code and got the same results,
which, if you look at the patch, makes sense.  The code is functionally
identical -- the change made was to suppress a warning.


--
Kent Williams norman-k-williams at uiowa.edu






On 10/23/13 10:57 AM, "Matt McCormick" <matt.mccormick at kitware.com> wrote:

>Hi Kent, Brian,
>
>Is this recent? Related to this patch merged yesterday?
>
>  http://review.source.kitware.com/#/c/13098/1
>
>Thanks,
>Matt
>
>On Wed, Oct 23, 2013 at 11:42 AM, Williams, Norman K
><norman-k-williams at uiowa.edu> wrote:
>> I notice that there's on Intel ICPC compiler on the nightly dashboards
>> under exotics that doesn't show up this error, but I don't know if we're
>> using a more recent version or different enough flags to matterŠ I'm
>>using
>> the RelWithDebInfo build type.
>>
>> I already fixed one test that was failing because it expected (and did
>>not
>> get) exact equality between floating point values
>> (JointDomainImageToListSampleAdaptorTest). I'll put a patch up for that.
>>
>> But FastChamferDistanceImageFilterTest requires some judgement on the
>>part
>> of people who know what it's actually doing.  The results are different:
>>
>> 500: Band initial size: 0
>> 500: Band size: 171536
>> 500: Inner positive points: 49496 != 49472
>> 500: Inner negative points: 28952 != 28928
>> 500: Rest of points: 93088 != 93136
>> 500: outweights = 0x2fcbf4c
>> 500: Test passed
>>
>> To me, that looks as though the difference is small enough that both the
>> expected answer and what ICPC generates could be reasonable.  But I'm
>>not
>> sure how to verify this, nor how exactly to modify the test such that
>>ICPC
>> will pass.
>>
>> Suggestions?
>>
>> --
>> Kent Williams norman-k-williams at uiowa.edu
>>
>>
>>
>>
>>
>>
>> ________________________________
>> Notice: This UI Health Care e-mail (including attachments) is covered
>>by the Electronic Communications Privacy Act, 18 U.S.C. 2510-2521, is
>>confidential and may be legally privileged.  If you are not the intended
>>recipient, you are hereby notified that any retention, dissemination,
>>distribution, or copying of this communication is strictly prohibited.
>>Please reply to the sender that you have received the message in error,
>>then delete it.  Thank you.
>> ________________________________
>> _______________________________________________
>> 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.php
>>
>> 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
>> _______________________________________________
>> Community mailing list
>> Community at itk.org
>> http://public.kitware.com/cgi-bin/mailman/listinfo/community



________________________________
Notice: This UI Health Care e-mail (including attachments) is covered by the Electronic Communications Privacy Act, 18 U.S.C. 2510-2521, is confidential and may be legally privileged.  If you are not the intended recipient, you are hereby notified that any retention, dissemination, distribution, or copying of this communication is strictly prohibited.  Please reply to the sender that you have received the message in error, then delete it.  Thank you.
________________________________
_______________________________________________
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.php

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