[Insight-developers] Global Code Reviews

[Bill Lorensen]

I did not mean "clearing" literally.

My concern is with the external project movement. I think we should review
the "Review" classes, move/repair the ones that satisfy "some criteria".
Once we have done this, then we can create an External Package.

There are so many changes to our software process, I'm concerned that adding
another step (working with an external process) will hurt our productivity.
I ajm also concerned that there are only a handfull of people that
understand the current, new process.

Bill

On Tue, Apr 26, 2011 at 1:40 PM, Bradley Lowekamp <blowekamp at mail.nih.gov>wrote:

> Hello Bill,
>
> I am not sure that "clearing" out review it the correct way to view thing.
> This phrase implies to me moving everything out of review and into ITK
> proper.
>
> There are many good filters that people have been using for quite a while.
> These certainly should be moved. While there are others that have not used
> and still need work. While others should replace filters in ITK or are
> otherwise redundant.
>
> Luis has certainly put together a good list of things to check, and it's a
> good list regarding the effort.
>
> Please only move filters that you can expect the community to be able to
> trust the results of. (itkFFTComplexToComplexImageFilter has no
> implementation, for non-FFTW and segfaults,
> http://www.itk.org/Bug/view.php?id=11934)
>
> Or perhaps we could start look into the filters where are currently in ITK
> and not up to the standards. (BinaryPrunningImageFilter,
> BinaryThinningImageFilter)
>
> Or perhaps we could start looking into the redundant filters in ITK.
> (ErodeObjectMorphologyImageFilter, and DilateObjectMorphologyImageFilter no
> one can tell me how this is different then setting for foreground and
> background values in the BinaryDilateImageFilter )
>
>
>
> *reads Luis's e-mail*
>
> On Apr 26, 2011, at 1:12 PM, Luis Ibanez wrote:
>
> As we discussed during the Boston meeting, a global
> code review effort will take place before the Beta release.
>
> The code assignments to contractors that Jim created,
> have been posted at:
>
> https://spreadsheets.google.com/a/kitware.com/spreadsheet/ccc?key=t8spmfVf1nHiKKkNnVunkUQ&authkey=CM-C2vcL#gid=0
>
>
> We are organizing the procedure for the code reviews in:
> http://www.itk.org/Wiki/ITK_Release_4/Global_Code_Review
>
>
> One of the first action items will be to move the Review
> directory out of the repository and make it available as
> an External module.
>
>
> Then we will identify the classes from Review that are
> commonly used (e.g. in applications like Slicer), and will
> put those classes in the top priorities of code review to
> move them into their final destination inside of the toolkit.
>
>
>   Please share your thoughts on this process,
>
>
> This process make sense to me.
>
>
>
>       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
>
>
>
> _______________________________________________
> 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
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20110426/904a1502/attachment.htm>