<div class="gmail_quote">On Fri, Feb 5, 2010 at 9:34 PM, Berk Geveci <span dir="ltr"><<a href="mailto:berk.geveci@kitware.com">berk.geveci@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
I wouldn't have the flag but if I was forced to have it, I'd have it as the last argument with a default value set to false (I think) so that one can still do<br><br>GetPoint(i, j, k, pt);<br><br>I think that it is better to force people to think in terms of extents because that is the way they need to implement their algorithms if they want them to work when streaming. So, if they are going to loop over all points, I'd rather they did:<br>

<br>for (i=extent[0]; i<extent[1]; i++)<br>  for (j=extent[2]; j<extent[2]; k++)<br>   
for (k=extent[3]; k<extent[4]; k++)<br><br>than<br><br>for (i=0; i<dims[1]; i++)<br>
  for (j=0; j<dims[2]; k++)<br>
   
for (k=0; k<dims[4]; k++)<br>
<br>Also, I wouldn't use x, y and z for variable names as many people associate those names with position rather than index (nitpicking here).<br><br>-berk<br><br></blockquote><div><br></div><div>All -</div><div><br>
</div><div>I committed the function. Please take a look and make sure we're all on the same page.</div><div><br></div><div>Berk - it's not nitpicking, it's clarity. I changed it. However, I set the flag to 'true' as David G has beaten into me that extents are your friend :)</div>
<div><br></div><div>David G. - please read the documentation I added with the function and make sure I got my extent interpretation correct.</div><br clear="all">Thanks for the discussion everyone - I hope this function will remove some of the confusion of the StructuredGrid.<br>
<br><div>David</div></div>