[Insight-developers] [ITK Community] Is dashboard hackathon still needed?

Matt McCormick matt.mccormick at kitware.com
Tue Dec 17 16:36:52 EST 2013


On Tue, Dec 17, 2013 at 4:15 PM, Vladimir S. FONOV
<vladimir.fonov at gmail.com> wrote:
> So, this approach actually have nothing to do with the question if feature
> is actually needed or not?
>

This feature definitely adds huge value both to ITK and the medical
imaging community in general.

> I suggest a new radical solution - remove all the features from ITK, then
> you will have a perfectly clean and neat dashboard and nothing to worry
> about

That would be very sad.

The objective is get the Nightly dashboard builds all green, then the
CDash at Home builds green with this feature enabled.  This way, if
something unrelated to MINCIO is added but breaks MINCIO, it is
noticed and fixed by the author.  Getting and keeping the dashboard
clean is way to prevent everyone from breaking each other's code so
that we all benefit from each others work.

Thank you for all the work that has been put into the ITK-MINC
integration.  It is highly valued.  With these improvements, your
maintenance burden will be reduced.

Thanks,
Matt


>
>
>
> On 13-12-17 03:59 PM, Johnson, Hans J wrote:
>>
>> If warnings persist, that feature in ITK will be placed into a deprecated
>> status, and will be prime material to remove in future releases.
>>
>> It does not necessarily need to be you, but in a community development
>> environment, if the there is not sufficient support for removing compiler
>> warnings that is a strong statement about the lack of importance for that
>> component of the toolkit in future versions.
>>
>> I understand the that there are never enough resources.  Please try to
>> recruit a fan-base of others who feel this is an important part of ITK and
>> that they should address the issues that you can not.
>>
>> Hans
>>
>> -----Original Message-----
>> From: "Vladimir S. FONOV" <vladimir.fonov at gmail.com>
>> Date: Tuesday, December 17, 2013 2:55 PM
>> To: Bill Lorensen <bill.lorensen at gmail.com>, Hans Johnson
>> <hans-johnson at uiowa.edu>
>> Cc: Sean McBride <sean at rogue-research.com>, ITK
>> <insight-developers at itk.org>
>> Subject: Re: [Insight-developers] [ITK Community] Is dashboard hackathon
>> still needed?
>>
>> OK,
>>
>> I personally have a situation when I have to allocate limited resources
>> (my time) between different projects, not all of them being software
>> development.
>>
>> So, convince me why I have to spend any more time on this particular
>> project then absolutely necessary.
>>
>> By the way, did anybody actually verify that my changes actually solve
>> original problem (i.e compilation for Windows) instead of
>> procrastinating and fixing compiler warnings ?
>>
>>
>> On 13-12-17 03:46 PM, Bill Lorensen wrote:
>>>
>>> A few years back at a NA_MIC project Week in SLC, I suggested that
>>> warnings are the graffiti of software quality. Here is a summary:
>>>
>>> The book, The Tipping Point, analyses trends and how they take hold.
>>> As (NAME YOUR SOFTWARE) grows and the number of contributors
>>> increases, there needs to be tighter controls on the quality of the
>>> system. The larger the code base, number of platforms and developers,
>>> the more need there is for attention to quality.
>>>
>>> In the Tipping Point, a chapter on increasing crime in New York City
>>> in the 1990's describes how the trend was reversed. The city
>>> concentrated areas like cleaning graffiti from subway cars. The author
>>> claims that this and other seemingly trivial goals may have led to a
>>> reversal of the crime trend.
>>>
>>> This project hypothesizes that compilation warnings are the graffiti
>>> of software quality. A lack of attention to this seemingly trivial
>>> goal shows a lack of attention to the overall quality of software.
>>> This project will reduce warnings to zero for a popular c++
>>> compiler,(NAME YOUR COMPILER).
>>>
>>>
>>> On Tue, Dec 17, 2013 at 3:36 PM, Johnson, Hans J
>>> <hans-johnson at uiowa.edu> wrote:
>>>>
>>>> ABSOLUTELY THEY MUST BE FIXED!
>>>>
>>>> A tremendous amount of developer energy used in maintaining code that
>>>> has
>>>> warnings.  The cost of software maintenance is substantially reduce when
>>>> warnings are removed.
>>>>
>>>> It is my experience that warnings VERY highly correlated with real bugs.
>>>>
>>>> Hans
>>>>
>>>>
>>>> -----Original Message-----
>>>> From: "Vladimir S. FONOV" <vladimir.fonov at gmail.com>
>>>> Date: Tuesday, December 17, 2013 2:28 PM
>>>> To: Sean McBride <sean at rogue-research.com>
>>>> Cc: ITK <insight-developers at itk.org>
>>>> Subject: Re: [Insight-developers] [ITK Community] Is dashboard hackathon
>>>> still needed?
>>>>
>>>> Hello,
>>>>
>>>>
>>>> do we really have to fix warnings at this stage? I.e everything else
>>>> actually compiles?
>>>>
>>>> On 13-12-17 03:05 PM, Sean McBride wrote:
>>>>>
>>>>> Hi all,
>>>>>
>>>>> As for the MINC compiler warnings... I've fixed a bunch in my github
>>>>> fork, Vlad can you review and merge?
>>>>>
>>>>> <https://github.com/seanm/libminc/tree/develop>
>>>>>
>>>>> I've also disabled a bunch of others, because they are a long term
>>>>> project...
>>>>>
>>>>> Cheers,
>>>>>
>>>>
>>>>
>>>> --
>>>> Best regards,
>>>>
>>>>     Vladimir S. FONOV ~ vladimir.fonov <at> gmail.com
>>>> _______________________________________________
>>>> 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
>>>>
>>>>
>>>>
>>>> ________________________________
>>>> Notice: This UI Health Care e-mail (including attachments) is covered
>>>> by the Electronic Communications Privacy Act, 18 U.S.C. 2510-2521, is
>>>> confidential and may be legally privileged.  If you are not the intended
>>>> recipient, you are hereby notified that any retention, dissemination,
>>>> distribution, or copying of this communication is strictly prohibited.
>>>> Please reply to the sender that you have received the message in error,
>>>> then delete it.  Thank you.
>>>> ________________________________
>>>> _______________________________________________
>>>> 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
>>>
>>>
>>>
>>>
>>
>>
>> --
>> Best regards,
>>
>>    Vladimir S. FONOV ~ vladimir.fonov <at> gmail.com
>>
>>
>>
>> ________________________________
>> Notice: This UI Health Care e-mail (including attachments) is covered by
>> the Electronic Communications Privacy Act, 18 U.S.C. 2510-2521, is
>> confidential and may be legally privileged.  If you are not the intended
>> recipient, you are hereby notified that any retention, dissemination,
>> distribution, or copying of this communication is strictly prohibited.
>> Please reply to the sender that you have received the message in error, then
>> delete it.  Thank you.
>> ________________________________
>>
>
>
> --
> Best regards,
>
>  Vladimir S. FONOV ~ vladimir.fonov <at> gmail.com
> _______________________________________________
> 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
> _______________________________________________
> Community mailing list
> Community at itk.org
> http://public.kitware.com/cgi-bin/mailman/listinfo/community


More information about the Insight-developers mailing list