[Insight-developers] Dashboard: Persistant itkClampImageFilterTest

[Bradley Lowekamp]

Cory,

I didn't think the following statement was true, and least for all compiler we use to support:

On May 7, 2013, at 10:00 AM, Cory Quammen <cquammen at cs.unc.edu> wrote:
> 
> __int64 in Visual Studio appears to really be a "long long", and there
> are numeric traits for that type as well as "unsigned long long"

A quick check of the VS 2012 MSDN documentation says[1]: :The __int64 type has no ANSI equivalent."

http://msdn.microsoft.com/en-us/library/29dh1w7z.aspx

In thinking about my VS build vs the others I may be the only one without 64-bit IDs enabled... So the error may... be related to that.

> 
> Bill Lorensen and I banged out heads on a similar error in this filter
> that happened only on Windows for this type (I think it was this
> type). It turned out that turning off optimization for this particular
> test solved the problem on windows. This filter was recently moved to
> the IntensityFilter module, but the CMakeLists.txt file there looks
> like it is turning off optimization for this test, so I'm not sure
> what is causing these errors.

There is likely a real problem with the filter. And just disabling optimization likely hides it.

My quick look at it says there are likely some issues with 64-bit integers, being converted to doubles:

https://github.com/Kitware/ITK/blob/master/Modules/Filtering/ImageIntensity/include/itkClampImageFilter.h#L95

Brad

> 
> Cory
> 
> On Tue, May 7, 2013 at 9:40 AM, Bradley Lowekamp <blowekamp at mail.nih.gov> wrote:
>> Hello all,
>> 
>> I just wanted to point out that the persistent ClampImageFilterTest on the dashboard:
>> 
>> http://open.cdash.org/testSummary.php?project=2&name=itkClampImageFilterTest&date=2013-05-04
>> 
>> http://open.cdash.org/testDetails.php?test=187338359&build=2895241
>> 
>> 
>> Seems related to the unsigned __int64 type.
>> 
>> http://open.cdash.org/testDetails.php?test=187338359&build=2895241
>> 
>> 
>> It seems to be just failing on my windows machines. I have not followed the changes to this filter to know what might be the issue.
>> 
>> I thought we removed the __int64 pixel type from being supported in images? I don't think we have NumbericTraits for it any more.
>> 
>> Thanks,
>> Brad
>> _______________________________________________
>> 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
> 
> 
> 
> -- 
> Cory Quammen
> Research Associate
> Department of Computer Science
> The University of North Carolina at Chapel Hill

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/pipermail/insight-developers/attachments/20130507/6d7d268b/attachment.htm>