A good path for improvement would (in my humble opinion), - Fix build errors on Windows - Fix warnings on all platforms (hopefully this will also help with tests) - Fix tests on all platforms - Enable the modules on the CDash@Home Gerrit builds - Happily continue with collaborative development :-)
There is no specific urgency for the timeline. There are many things that should be improved in ITK :-). Thanks, Matt On Tue, Dec 17, 2013 at 4:42 PM, Vladimir S. FONOV <[email protected]> wrote: > > 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
