<br>Hi, <div> The current implementation of BoundaryFacesCalculator is based on the following assumption.</div><div><br></div><div>There is an overlap between requestToProcessRegion and nonBounaryRegion of BufferedRegion.</div>
<div><br></div><div>Hence the first region in the output list is the nonBoundaryRegion in requestToProcessRegion.</div><div><br></div><div>In your test, there is no overlap between requestToProcessRegion and bufferedRegion since requstToProcessRegion index[Dimension-1] = 1, which bufferedRegion index[Dimension - 1] = 0, size[Dimension - 1] = 1, . The ideal results ought to output 0 regions. </div>
<div><br></div><div>Should we change the implementationso the calculator could deal with any cases of requestToProcessRegion and BufferedRegion? I am willing to put a patch.</div><div><br></div><div><br></div><div>thanks</div>
<div><br></div><div><br></div><div>wanlin</div><div><br></div><div><br><div class="gmail_quote">On Tue, Apr 17, 2012 at 9:21 AM, Kris Zygmunt <span dir="ltr"><<a href="mailto:krismz@sci.utah.edu">krismz@sci.utah.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
While trying to compare results between itkScalarAnisotropicDiffusionF<u></u>unction and itkGPUScalarAnisotropicDiffusi<u></u>onFunction, I found a bug in the NeighborhoodAlgorithm::<u></u>BoundaryFacesCalculator when one of the image sizes is 1 ie a 3D image with size [5,5,1] (I cloned itk 4 today and the bug is still in master). In this case, the calculator repeats the final region twice. It seems like this might have been addressed once before in ITK 3.20 just before 4.0, but that may have been another bug or that fix might have introduced this bug.<br>
<br>
Anyway, I have been able to modify the test to demonstrate this bug and have put that patch at <a href="http://review.source.kitware.com/#/c/5213/" target="_blank">http://review.source.kitware.<u></u>com/#/c/5213/</a> . This may change the answer from some tests if they are unlucky enough to be using images that fall into this image size condition.<br>
<br>
-Kris<br>
______________________________<u></u>_________________<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/<u></u>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/<u></u>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_<u></u>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/<u></u>listinfo/insight-developers</a><br>
</blockquote></div><br></div>