[Insight-developers] vnl updates

[Luis Ibanez]

Hi Tom,

The Experimental build (From Ubuntu Linux with gcc 4.2):

http://www.cdash.org/CDash/viewSite.php?siteid=1086&project=2&currenttime=1230512400

came up with three failing tests.

http://www.cdash.org/CDash/viewTest.php?onlyfailed&buildid=239928

They are:

    760 - itkInterpolateTest (Failed)
    1310 - ResampleImageFilter3Test1 (Failed)
    1314 - ResampleImageFilter9Test (Failed)


We will have to address these failure before we
update the vnl_math.h file.


Would you have a chance to look at one of them ?

The first one and third one seem to be the ones providing a better hint
to the source of the problem.

In particular, the diff image at
http://www.cdash.org/CDash/testDetails.php?test=14663712&build=239928
may help.


   Please let us know,

----------------------------------------------------------------------------------------------
On Mon, Dec 29, 2008 at 12:27 PM, Luis Ibanez <luis.ibanez at kitware.com>wrote:

> Hi Tom,
>
> Thanks for looking at this.
>
> I'm running an experimental build with this change (new vnl_math.h).
>
> If it comes out green I'll commit the new vnl_math.h file,
> and we will see the effect on tomorrow's Nightly builds...
>
>
>       Luis
>
>
>
> -------------------------------------------------------------------------------------------------------
>
> On Mon, Dec 29, 2008 at 12:24 PM, Tom Vercauteren <tom.vercauteren at m4x.org
> > wrote:
>
>> Small mistake... I meant VNL_CONFIG_ENABLE_SSE2_ROUNDING will not be
>> defined by cmake.
>> Tom
>>
>> On Mon, Dec 29, 2008 at 6:04 PM, Tom Vercauteren
>> <tom.vercauteren at m4x.org> wrote:
>> >> Could you point me to the set of lines that we want to import
>> >> from the new version of vnl_math.h  ?
>> >
>> > The difference between ITK's vnl_math.h and vxl's vnl_math.h is
>> > exactly what is needed:
>> >
>> http://vxl.cvs.sourceforge.net/viewvc/vxl/vxl/core/vnl/vnl_math.h?r1=1.50&r2=1.42
>> >
>> > So the only thing that needs to be done is replace the current
>> > vnl_math.h file by the most up-to-date one.
>> >
>> > As I mentioned, the sse2 optimization will not be compiled since
>> > VNL_CHECK_FPU_ROUNDING_MODE will not be defined by cmake.
>> >
>> >
>> >> We could insert them today or tomorrow...
>> >
>> > Great!
>> >
>> > Thanks,
>> > Tom
>> >
>> > P.S.: As far as vnl_math.h is concerned, there should be no backward
>> > compatibility issue.
>> >
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20081229/0a45066c/attachment.htm>