[ITK Community] [Insight-developers] Is dashboard hackathon still needed?
Matt McCormick
matt.mccormick at kitware.com
Tue Dec 17 16:09:35 EST 2013
I have started an Experimental that just finished building. I will put
a link on the Gerrit change.
The emphasis on fixing warnings is excellent as is the emphasis that
this a community development process.
If a patch set can be found that fixes the Windows build errors, it
may be good to merge it, let it hit the dashboard, then let Sean get
credit for fixing the other warnings by submitting another patch set.
A feedback cycle with multiple iterations on the dashboard will be
required to fix all warnings and tests.
Thanks,
Matt
On Tue, Dec 17, 2013 at 3:59 PM, Johnson, Hans J <hans-johnson at uiowa.edu> 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.
> ________________________________
> _______________________________________________
> 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
_______________________________________________
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
More information about the Community
mailing list