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

[Luis Ibanez]

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
>
>