<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Matt,<div><br></div><div>You have a wide variety of patches on the topic "PrimaryName". I am having difficulty reviewing this topic with as many commits as it is, and many of them don't appear to be depended on each other.</div><div><br></div><div>If it's a new A2D2, it would be best to get it in while there is still time to work on refining it. And you have an outstanding record of rapidly addressing issue that come up on the dashboard.</div><div><br></div><div>As for changing the ProcessObject, to support a custom PrimaryName, I am not enthused by this change. It seems like it's adding more complication to the base class without a huge benefit. With the interwoven commits on this topic it's hard to figure out where it's needed. It appears that this was the approached used to address some bugs in the BlockMatchingImageFilter. However, I don't see why the standard numbered inputs were not used. It appears that all 3 inputs are required, which matches the conventional usage of numbered inputs.</div><div><br></div><div>Sorry for the delayed response,</div><div>Brad</div><div><br></div><div><br><div apple-content-edited="true">
<span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; "><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><p style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><font face="Helvetica" size="3" style="font: normal normal normal 12px/normal Helvetica; ">========================================================</font></p><p style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><font face="Helvetica" size="3" style="font: normal normal normal 12px/normal Helvetica; ">Bradley Lowekamp<span class="Apple-converted-space"> </span><span class="Apple-converted-space"> </span></font></p><p style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><font face="Helvetica" size="3" style="font: normal normal normal 12px/normal Helvetica; ">Medical Science and Computing for</font></p><p style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><font face="Helvetica" size="3" style="font: normal normal normal 12px/normal Helvetica; ">Office of High Performance Computing and Communications</font></p><p style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><font face="Helvetica" size="3" style="font: normal normal normal 12px/normal Helvetica; ">National Library of Medicine<span class="Apple-converted-space"> </span></font></p><p style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><font face="Helvetica" size="3" style="font: normal normal normal 12px/normal Helvetica; "><a href="mailto:blowekamp@mail.nih.gov">blowekamp@mail.nih.gov</a></font></p></div></div></span></span>
</div>
<br><div><div>On Jun 18, 2012, at 7:16 PM, Matt McCormick wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div>Hi,<br><br>Please consider the following A2D2 patches for consideration in this<br>release cycle. While it is late in the release process, if they are<br>merged within the next few days we will still have funding to address<br>any issues that come up on the dashboard during the remainder of the<br>week.<br><br>This patch:<br><br> <a href="http://review.source.kitware.com/#/c/6155/">http://review.source.kitware.com/#/c/6155/</a><br><br>Is a class to perform the BlockMatching/FEM registration based on the<br>already merged feature selection class.<br><br>This patch:<br><br> <a href="http://review.source.kitware.com/#/c/6234/">http://review.source.kitware.com/#/c/6234/</a><br><br>Is needed to have the inputs named in the former filter as desired and<br>let the pipeline update properly.<br><br>There are also related bug fixes that were discovered in the process<br>of preparing the patch:<br><br> <a href="http://review.source.kitware.com/#/c/6238/">http://review.source.kitware.com/#/c/6238/</a><br> <a href="http://review.source.kitware.com/#/c/6239/">http://review.source.kitware.com/#/c/6239/</a><br> <a href="http://review.source.kitware.com/#/c/6245/">http://review.source.kitware.com/#/c/6245/</a><br><br>Thanks,<br>Matt<br></div></blockquote></div><br></div></body></html>