<div dir="ltr">Thanks Niels for the feedback. When we tag the RC1, please let us know if we missed any of these patch.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 4, 2017 at 10:20 AM, Dženan Zukić <span dir="ltr"><<a href="mailto:dzenanz@gmail.com" target="_blank">dzenanz@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_default" style="font-family:verdana,sans-serif;font-size:small">Hi Niels,</div><div class="gmail_default" style="font-family:verdana,sans-serif;font-size:small"><br></div><div class="gmail_default" style="font-family:verdana,sans-serif;font-size:small">the RC1 tag will be added to the current master, which means all the patches which have been merged to master will be included.</div><div class="gmail_default" style="font-family:verdana,sans-serif;font-size:small"><br></div><div class="gmail_default" style="font-family:verdana,sans-serif;font-size:small">Regards,</div><div class="gmail_default" style="font-family:verdana,sans-serif;font-size:small">Dženan</div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 4, 2017 at 5:08 AM, Niels Dekker <span dir="ltr"><<a href="mailto:niels-xtk@xs4all.nl" target="_blank">niels-xtk@xs4all.nl</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>Francois Budin wrote on December 1st 2017:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
We are planning to tag ITK 4.13 Release Candidate 1 on Monday,<br>
December 4th. If there is any enhancement that you would like to see<br>
included in ITK 4.13.0, please submit your patch as soon as possible<br>
as they need to be reviewed and merged prior. If you have patches that<br>
have been in the review process for a while and not yet accepted,<br>
mention that you would like these to be integrated in the upcoming<br>
release. As a reminder, this means feature freeze for ‘master’<br>
and, therefore, enhancements should not be merged until after the<br>
release candidate period. The final release is planned for December<br>
18th.<br>
</blockquote>
<br></span>
Thank you for the information! Can you please make sure all the Hough Transform patches that I submitted in the last two months will be included with the release?<br>
<br>
 * ENH: Allow different types accumulator and radius image Hough Circles filter<br>
 * DOC: Mentioned that HoughTransform2DCirclesImageFi<wbr>lter also finds discs<br>
 * STYLE: HoughTransform calling convenience overload of SetVariance<br>
 * PERF: Improved speed of HoughTransform2DCirclesImageFi<wbr>lter::GenerateData()<br>
 * BUG: HoughTransform GetLines() returned too many lines<br>
 * ENH: Improved HoughTransform2DCirclesImageFi<wbr>lter accuracy, ushttps://<a href="http://webmail.xs4all.nl/?_task=mail&_action=compose&_id=18151380785a251a18310b7#ing" rel="noreferrer" target="_blank">webmail.xs4all.nl/?_<wbr>task=mail&_action=compose&_id=<wbr>18151380785a251a18310b7#ing</a> Math::Round<br>
 * STYLE: Removed HoughTransform GetLines parameter (n), which appears obsolete<br>
 * BUG: HoughTransform GetCircles() should avoid circles with accumulator <= 0<br>
 * ENH: Tested HoughTransform CirclesList is empty when NumberOfCircles is zero<br>
 * BUG: HoughTransform CirclesList should be empty when NumberOfCircles is zero<br>
 * COMP: Replaced deprecated GetCircles(n) call in HoughTransform example<br>
 * ENH: Added deprecated GetCircles(unsigned int) overload to HoughTransform<br>
 * STYLE: Removed HoughTransform GetCircles parameter (n) which appears obsolete<br>
 * DOC: Added comment that the grafted output image should not be modified.<br>
 * PERF: GetCircles now calls Graft, instead of copying pixels from GetOutput(0)<br>
 * PERF: Avoid redundant search in GetCircles(n)<br>
 * STYLE: Removed useless (or obsolete) local bool, 'found'.<br>
<br>
I think there is still some more room for improvement, but I hope that getting these patches into the release will trigger more feedback from ITK users on how to further improve the ITK Hough Transform.<br>
<br>
<br>
Kind regards, Niels<br>
<br>
-- <br>
Niels Dekker<br>
Scientific programmer<br>
LKEB, Leiden University Medical Center, Netherlands<br>
The ITK community is transitioning from this mailing list to <a href="http://discourse.itk.org" rel="noreferrer" target="_blank">discourse.itk.org</a>. Please join us there!<br>
______________________________<wbr>________<br>
Powered by <a href="http://www.kitware.com" rel="noreferrer" 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" rel="noreferrer" target="_blank">http://www.kitware.com/opensou<wbr>rce/opensource.html</a><br>
<br>
Kitware offers ITK Training Courses, for more information visit:<br>
<a href="http://kitware.com/products/protraining.php" rel="noreferrer" target="_blank">http://kitware.com/products/pr<wbr>otraining.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" rel="noreferrer" target="_blank">http://www.itk.org/Wiki/ITK_FA<wbr>Q</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/insight-developers" rel="noreferrer" target="_blank">http://public.kitware.com/mail<wbr>man/listinfo/insight-developer<wbr>s</a><br>
</blockquote></div><br></div>
</div></div><br>The ITK community is transitioning from this mailing list to <a href="http://discourse.itk.org" rel="noreferrer" target="_blank">discourse.itk.org</a>. Please join us there!<br>
______________________________<wbr>________<br>
Powered by <a href="http://www.kitware.com" rel="noreferrer" 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" rel="noreferrer" target="_blank">http://www.kitware.com/<wbr>opensource/opensource.html</a><br>
<br>
Kitware offers ITK Training Courses, for more information visit:<br>
<a href="http://kitware.com/products/protraining.php" rel="noreferrer" target="_blank">http://kitware.com/products/<wbr>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" rel="noreferrer" target="_blank">http://www.itk.org/Wiki/ITK_<wbr>FAQ</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/insight-developers" rel="noreferrer" target="_blank">http://public.kitware.com/<wbr>mailman/listinfo/insight-<wbr>developers</a><br>
<br>The ITK community is transitioning from this mailing list to <a href="http://discourse.itk.org" rel="noreferrer" target="_blank">discourse.itk.org</a>. Please join us there!<br>
______________________________<wbr>__________<br>
Community mailing list<br>
<a href="mailto:Community@itk.org">Community@itk.org</a><br>
<a href="http://public.kitware.com/mailman/listinfo/community" rel="noreferrer" target="_blank">http://public.kitware.com/<wbr>mailman/listinfo/community</a><br>
<br></blockquote></div><br></div>