<br><div class="gmail_quote">On Mon, Oct 12, 2009 at 1:22 PM, Luis Ibanez <span dir="ltr"><<a href="mailto:luis.ibanez@kitware.com">luis.ibanez@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Hi David,<br>
<br>
<br>
The class<br>
<br>
<br>
vtkModifiedBSPTree.h<br>
<br>
<br>
was added to VTK on August 5th 2009:<br>
<br>
<a href="http://public.kitware.com/cgi-bin/viewcvs.cgi/Filtering/vtkModifiedBSPTree.h?view=log" target="_blank">http://public.kitware.com/cgi-bin/viewcvs.cgi/Filtering/vtkModifiedBSPTree.h?view=log</a><br>
<br>
Revision 1.1<br>
date: 2009-08-05 10:30:33 -0400; author: biddisco; state: Exp;<br>
commitid: Evwm0eBFvHniZvYt;<br>
ENH: Add new Cell Locator class to Filtering. This is a BSP tree which<br>
store cells using axis aligned lists, it was originally optimized for<br>
RayCasting, but still works well as a general purpose cell locator.<br>
=============================================================================<br>
<br>
<br>
<br>
Therefore this class is not included in the VTK<br>
releases 5.4 nor 5.4.2.<br>
<br>
<br>
The Insight Journal build environment only supports<br>
recent official releases of the toolkits, not their CVS<br>
versions.<br>
<br>
<br>
Any chance that you could base your insight Journal<br>
submission on VTK 5.4 ?<br>
<br>
<br>
Please let us know,<br>
<br>
<br>
Thanks<br>
<br>
<br>
Luis<br></blockquote><div><br>Luis,<br>
<br>When you CC people (Julien, Zack) I assume it's for a good reason - should I reply-all to them or just reply to the list?<br>
<br>I found that the modified BSP tree produced a MASSIVE speed up (50x +) so it would seem silly to go back to using one of the other data structures. I guess there's nothing else we can do then? I suppose having the IJ build environment use CVS versions is out of the question? I guess it depends on the mission statement of the IJ build platform - if it is mainly for ITK then I guess having only the VTK release version makes sense. If it is trying to be a catch all for Kitware product code it doesn't seem too unreasonable to support the latest cvs versions of all the toolkits.<br>
<br>Let me know what you all think.<br><br clear="all">Thanks,<br><br>David <br></div></div>