<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Word 14 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:Tahoma;
        panose-1:2 11 6 4 3 5 4 4 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:12.0pt;
        font-family:"Times New Roman","serif";}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:purple;
        text-decoration:underline;}
p
        {mso-style-priority:99;
        mso-margin-top-alt:auto;
        margin-right:0in;
        mso-margin-bottom-alt:auto;
        margin-left:0in;
        font-size:12.0pt;
        font-family:"Times New Roman","serif";}
span.EmailStyle18
        {mso-style-type:personal-reply;
        font-family:"Calibri","sans-serif";
        color:#1F497D;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-size:10.0pt;
        font-family:"Calibri","sans-serif";}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
        {page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">Berk/all,<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">I have read this thread with interest, and have been impressed with how open and honest everyone has been.  We have a problem, we discuss the problem, we find
 fixes for the problem.  After discussing these issues with my local team, one comment came up over and over again.  I realize that the VTK workflow and this thread are different than ParaView, but since my experience is with ParaView, I will comment accordingly. 
 Further, I realize that the two projects have different needs, but I think the following comments still apply. 
<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">Although the current procedures do have issues slowing down community contributions, they also have strengths.  This is true for ParaView, and I assume for
 VTK.  I have been doing a lot of testing of ParaView for the upcoming 4.2 release, and ParaView is amazingly “clean” at this point.  GUI’s are clear, workflows are simple (or as simple as can be), lots of stuff gets tested in the dashboards, and everything
 just works (well, most things just work).  I am now somewhat surprised when I find bugs, as opposed to the past, where I was surprised when I didn’t.  Numerous years ago, before development was reviewed before moving it into head, it was always a crap shoot
 when updating.  Now, it is rare that Master isn’t in a state that is release candidate quality.  And I believe that Master should always be in a state that is release candidate quality.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">My executive summary is this: as we move forward, let’s make sure we don’t lose the processes and controls that have given us the quality products we currently
 have.  <o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">my $0.02.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">Alan<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal" style="margin-left:.5in"><b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> vtk-developers [mailto:vtk-developers-bounces@vtk.org]
<b>On Behalf Of </b>Berk Geveci<br>
<b>Sent:</b> Wednesday, August 27, 2014 7:11 AM<br>
<b>To:</b> Ronald Römer<br>
<b>Cc:</b> VTK Developers<br>
<b>Subject:</b> [EXTERNAL] Re: [vtk-developers] Driving away your existing developers<o:p></o:p></span></p>
<p class="MsoNormal" style="margin-left:.5in"><o:p> </o:p></p>
<div>
<p class="MsoNormal" style="margin-left:.5in">You are both absolutely right. The review process is broken when it comes to accepting patches from the community. It is broken in 3 ways:<o:p></o:p></p>
<div>
<p class="MsoNormal" style="margin-left:.5in"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:.5in">1. As Kitware, we haven't put the right attention to reviewing contributed code from top down (i.e. me and other managers using project resources and asking developers to review contributed work). Even when we have
 specific projects under which this can be done, we favor developing new code to reviewing contributions.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:.5in"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:.5in">2. As individual contributors to VTK and ParaView, many of the Kitware developers are focused on their primary goals. We haven't built the open source community spirit as well as we can. Part of this needs to be
 done by the larger community with encouragement by senior folks (not just ones at Kitware).<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:.5in"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:.5in">3. As others pointed out, there is not enough emphasis and encouragement in reviewing code by the larger community. My code would languish in Gerrit if I didn't have the ability to walk to someone next door and
 ask them to review it please. David Gobbi and others had suggestions on how to encourage the larger community to be more involved in reviewing.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:.5in"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:.5in">My opinion is that this all boils down to making people feel ownership for VTK rather than seeing it as a tool that needs to be patched as necessary for other work or some project goal (deliver x,y and z and you
 are done). This is not something I can easily force people to do nor is it a top down thing. I am doing my best by encouraging more community engagement, removing barriers from tools and setting an example by contributing in various ways. I'd like to hear
 suggestions on how to achieve this and what I can do towards this.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:.5in"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:.5in">One idea: more community hack-a-tons. I just sent out a suggestion for addressing issues in the bug tracker. Why not have another one for reviewing code?<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:.5in"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:.5in">One final note. Everyone please keep in mind that while there are quite a lot of contributors to VTK, certain parts of it are developed by only a few people. Core pipeline: I am it at this point. Rendering: by next
 year, it will be Ken and Marcus. Imaging: hardly anyone in the scientific vis team at Kitware does anything with imaging filters. We update them when we make changes to the pipeline or data model but that's it. So some of those are maintained and developed
 by others or they are some orphaned. This can only be addressed by either encouraging new (or existing) developers to take ownership or by removing those components all together.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:.5in"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:.5in">My 2 cents.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:.5in">-berk<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:.5in"><o:p> </o:p></p>
</div>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:0in;margin-right:0in;margin-bottom:12.0pt;margin-left:.5in">
<o:p> </o:p></p>
<div>
<p class="MsoNormal" style="margin-left:.5in">On Wed, Aug 27, 2014 at 8:37 AM, Ronald Römer <<a href="mailto:rroemer@gmail.com" target="_blank">rroemer@gmail.com</a>> wrote:<o:p></o:p></p>
<p style="margin-left:.5in">You are absolutely right, in most of your points. None if my commits were previewed by any native kitware developers. Its a pain to find the right previewers and after that none of them will do that job. So it is very hard for a
 new developer from outside the organization to get the right attention. This is frustrating...<o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:0in;margin-right:0in;margin-bottom:12.0pt;margin-left:.5in">
<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 <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">
http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/vtk-developers" target="_blank">http://public.kitware.com/mailman/listinfo/vtk-developers</a><br>
<br>
<o:p></o:p></p>
</div>
<p class="MsoNormal" style="margin-left:.5in"><o:p> </o:p></p>
</div>
</div>
</body>
</html>