So,
what's the current status ? Is there are warnings on official ITK build
dashboard that have to be addressed right now?
(Apart from M_PI issue on Visual Studio) ?
On 13-12-17 04:36 PM, Matt McCormick wrote:
On Tue, Dec 17, 2013 at 4:15 PM, Vladimir S. FONOV
<[email protected]> 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@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" <[email protected]>
Date: Tuesday, December 17, 2013 2:55 PM
To: Bill Lorensen <[email protected]>, Hans Johnson
<[email protected]>
Cc: Sean McBride <[email protected]>, ITK
<[email protected]>
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
<[email protected]> 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" <[email protected]>
Date: Tuesday, December 17, 2013 2:28 PM
To: Sean McBride <[email protected]>
Cc: ITK <[email protected]>
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
[email protected]
http://public.kitware.com/cgi-bin/mailman/listinfo/community
--
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
