<p dir="ltr">I agree. And this is the current process. Some modules that are more special purpose are suitable as remote modules. This is also current practice. I don't see the need for another category.<br>
</p>
<div class="gmail_quote">On Jan 16, 2014 9:57 AM, "Brad King" <<a href="mailto:brad.king@kitware.com">brad.king@kitware.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 01/15/2014 05:40 PM, Matt McCormick wrote:<br>
> An Contrib Group is a neat idea.  What is the difference between this<br>
> and Nonunit/Review though?<br>
<br>
FWIW, one of the goals of modularization, conversion to Git, and<br>
introduction of code review tools was to get rid of the "Review"<br>
kit (which was just another name for "Contrib").  In practice code<br>
rarely graduates from contrib directories.  Code should either be<br>
supported or not included in the first place.<br>
<br>
A proposed module should simply be locally added to the correct<br>
group and located in the source tree where it would belong as a<br>
first-class module.  Then it should go through the Insight Journal<br>
or just Gerrit review.  If accepted then it should be merged<br>
immediately as a first-class supported module.  If not then it<br>
should be revised prior to inclusion in upstream.<br>
<br>
What is the purpose of IJ + Gerrit if not to review code to ensure<br>
sufficient quality before acceptance?  The bar should not be made<br>
lower just by labeling something "Contrib".  Such a lower bar is<br>
one reason code rarely graduates out of that label.<br>
<br>
-Brad K<br>
<br>
_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at<br>
<a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Kitware offers ITK Training Courses, for more information visit:<br>
<a href="http://kitware.com/products/protraining.php" target="_blank">http://kitware.com/products/protraining.php</a><br>
<br>
Please keep messages on-topic and check the ITK FAQ at:<br>
<a href="http://www.itk.org/Wiki/ITK_FAQ" target="_blank">http://www.itk.org/Wiki/ITK_FAQ</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://www.itk.org/mailman/listinfo/insight-developers" target="_blank">http://www.itk.org/mailman/listinfo/insight-developers</a><br>
</blockquote></div>