[Insight-developers] Dashboard Cleanup : Timeout of Join Series in Dash11.kitware

[Bradley Lowekamp]

Luis,

Unfortunately, I have a prior meeting tomorrow afternoon, so I will not be able to attend the TCON. 

Brad


On Jan 21, 2010, at 1:35 PM, Luis Ibanez wrote:

> Hi Brad,
> 
> Thanks for taking care of that test.
> 
> ---
> 
> Yes, we should discuss what to include for ITK 3.18.
> 
> We could focus on that during tomorrow's tcon,
> if you are available.
> 
> 
> 
>      Luis
> 
> 
> 
> ----------------------------------------------
> On Thu, Jan 21, 2010 at 11:25 AM, Bradley Lowekamp
> <blowekamp at mail.nih.gov> wrote:
>> Hello Luis,
>> I failed to write the code I intended and the test failed again:
>> http://www.cdash.org/CDash/viewTest.php?onlyfailed&buildid=519781
>> Failure on my part to correctly apply: !(A || B) => !A && !B
>> ---
>> Regarding the 3.18 release
>> The items I would like to close before this release, are the current
>> AnaylzeIO bug I am working on, finalizing the interfaces for the new methods
>> in itkMath, and verifying that Doxygen generates good info for this
>> namespace as well. The Analyze bug should be committed shortly. As for the
>> math, I would like to restrict the types that rounding methods can be
>> instantiated with. I actually have the code written for this and it just
>> needs to be committed. I have hesitated due to the new constraint, which
>> might cause problems with some compilers:
>> namespace Concept
>> {
>> /** Concept requiring T to an integer type. */
>> template <typename T>
>> struct Integer
>> {
>>   typedef Integer Self;
>>   itkStaticConstMacro(IsInteger, bool, NumericTraits<T>::is_integer);
>>   struct Constraints
>>   {
>>     typedef itk::Concept::Detail::UniqueType_bool<true> TrueT;
>>     typedef
>> itk::Concept::Detail::UniqueType_bool<itkGetStaticConstMacro(IsInteger)>
>> IntegerT;
>>     void constraints()
>>       {
>>         IntegerT a = TrueT();
>>         itk::Concept::Detail::IgnoreUnusedVariable(a);
>>       }
>>     };
>> 
>>   itkConceptConstraintsMacro();
>> };
>> }
>> 
>> Due to a miss understanding I copied some numeric constants into the Math
>> namespace. These likely should just be removed.
>> The issues surround the definition and usage of CastWithRangeCheck still
>> exist. I am not sure what the best solution is at this point.
>> Brad
>> 
>> On Jan 20, 2010, at 3:30 PM, Luis Ibanez wrote:
>> 
>> Brad,
>> 
>> Excellent,
>> Thanks for fixing the test.
>> 
>> ---
>> 
>> About the release schedule, yes, we are working
>> towards a light release (ITK 3.18).
>> 
>> 
>> That's part of he motivation for cleaning up the
>> Dashboard.
>> 
>> 
>> As you pointed out, we should review the schedule
>> given that our previous target of mid-january have
>> already passed.
>> 
>> 
>> Any preferences ?  Suggestions ?
>> 
>> 
>>     Thanks
>> 
>> 
>>           Luis
>> 
>> 
>> =======================================================
>> On Wed, Jan 20, 2010 at 2:23 PM, Bradley Lowekamp
>> <blowekamp at mail.nih.gov> wrote:
>> 
>> I forgot abou the LEGACY_SILENT definition! That is exactly what this
>> 
>> warning message should have! I'll work on getting that committed.
>> 
>> BTW, did I miss an updated release date:
>> 
>> http://www.itk.org/Wiki/ITK_Release_Schedule
>> 
>> As I am wondering the date that this dash clean up is working towards?
>> 
>> Brad
>> 
>> 
>> On Jan 20, 2010, at 1:56 PM, Luis Ibanez wrote:
>> 
>> Brad,
>> 
>> Thanks for looking into this.
>> 
>> 
>> The configuration for Dash11 does not
>> 
>> have LEGACY_REMOVE   ON   either.
>> 
>> (it has LEGACY_SILENT set to ON though).
>> 
>> 
>> Do you mean that we should enable the flag
>> 
>> LEGACY_REMOVE   in dash11 in order to
>> 
>> prevent the large amount of messages from
>> 
>> this test  ?
>> 
>> 
>> If so I'll modify the ctest scripts,
>> 
>> 
>> Please let us know,
>> 
>> 
>>       Thanks
>> 
>> 
>>             Luis
>> 
>> 
>> ----------------------------------------------
>> 
>> On Wed, Jan 20, 2010 at 11:44 AM, Bradley Lowekamp
>> 
>> <blowekamp at mail.nih.gov> wrote:
>> 
>> Hello again Luis,
>> 
>> I wrote this test just a month ago for the following bug:
>> 
>> http://public.kitware.com/Bug/view.php?id=10008
>> 
>> I have been trying to look through the cdash history which has been slow.
>> 
>> This was the results of the first day of the new test:
>> 
>> http://www.cdash.org/CDash/testSummary.php?project=2&name=itkJoinSeriesImageFilterStreamingTest&date=2009-12-24
>> 
>> The cywin magically fixed itself, so I negated to look into the vs7 as well.
>> 
>> It does appear that the cywin did not have the LEGACY_REMOVE turned on, this
>> 
>> would be consistent with your finds that the message is printed too many
>> 
>> times as well.
>> 
>> Brad
>> 
>> 
>> 
>> 
>> On Jan 20, 2010, at 8:40 AM, Luis Ibanez wrote:
>> 
>> The test:
>> 
>> 
>>          itkJoinSeriesImageFilterStreamingTest
>> 
>> http://www.cdash.org/CDash/viewTest.php?onlyfailed&buildid=519006
>> 
>> 
>>            Is timing out in dash11.kitware.
>> 
>> The current timeout is set to 120 seconds.
>> 
>> Does anybody has an objection to raising the
>> 
>> TIMEOUT value of dash11  ?
>> 
>> 
>>   Thanks
>> 
>> 
>>        Luis
>> 
>> _______________________________________________
>> 
>> 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
>> 
>> 
>> 
>> ========================================================
>> 
>> Bradley Lowekamp
>> 
>> Lockheed Martin Contractor for
>> 
>> Office of High Performance Computing and Communications
>> 
>> National Library of Medicine
>> 
>> blowekamp at mail.nih.gov
>> 
>> 
>> 
>> ========================================================
>> 
>> Bradley Lowekamp
>> 
>> Lockheed Martin Contractor for
>> 
>> Office of High Performance Computing and Communications
>> 
>> National Library of Medicine
>> 
>> blowekamp at mail.nih.gov
>> 
>> 

========================================================
Bradley Lowekamp  
Lockheed Martin Contractor for
Office of High Performance Computing and Communications
National Library of Medicine 
blowekamp at mail.nih.gov


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20100121/ed96d0f3/attachment.htm>