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

[Bradley Lowekamp]

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


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