<div>David,</div><div><br></div>I was looking at your changes to the kit dependencies, and didn't understand one of them:<div><br></div><div>VTK/Infovis/CMakeLists.txt:</div><div><br></div><div><div>-SET(KIT_TCL_LIBS vtkWidgetsTCL)</div>
<div>-SET(KIT_PYTHON_LIBS vtkWidgetsPythonD)</div><div>-SET(KIT_JAVA_LIBS vtkWidgetsJava)</div><div>-SET(KIT_LIBS vtkWidgets ${VTK_LIBXML2_LIBRARIES} vtkalglib)</div><div>+IF(VTK_USE_RENDERING)</div><div>+  SET(KIT_TCL_LIBS vtkWidgetsTCL)</div>
<div>+  SET(KIT_PYTHON_LIBS vtkWidgetsPythonD)</div><div>+  SET(KIT_JAVA_LIBS vtkWidgetsJava)</div><div>+  SET(KIT_LIBS vtkWidgets ${VTK_LIBXML2_LIBRARIES} vtkalglib)</div><div>+ELSE(VTK_USE_RENDERING)</div><div>+  SET(KIT_TCL_LIBS vtkGraphicsTCL)</div>
<div>+  SET(KIT_PYTHON_LIBS vtkGraphicsPythonD)</div><div>+  SET(KIT_JAVA_LIBS vtkGraphicsJava)</div><div>+  SET(KIT_LIBS vtkGraphics  ${VTK_LIBXML2_LIBRARIES} vtkalglib)</div><div>+ENDIF(VTK_USE_RENDERING)</div><div><br>
</div><div>Nothing in the kit seems to be conditionally compiled based on rendering/widgets, so why is this conditional there? Views currently only depends on infovis, but should it really depend on widgets also? Then it seems we could get rid of this conditional and have infovis only depend on graphics.</div>
<div><br></div><div>Jeff</div><div><br></div><div class="gmail_quote">On Wed, Mar 18, 2009 at 12:55 PM, Thompson, David C <span dir="ltr"><<a href="mailto:dcthomp@sandia.gov">dcthomp@sandia.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Jeff,<br>
<br>
Yes, I will start moving these classes around as the wiki page outlines.<br>
<br>
   Thanks,<br>
<div class="im">   David<br>
________________________________________<br>
From: Jeff Baumes [<a href="mailto:jeff.baumes@kitware.com">jeff.baumes@kitware.com</a>]<br>
</div>Sent: Wednesday, March 18, 2009 08:07<br>
To: Shepherd, Jason F<br>
Cc: Thompson, David C; Pebay, Philippe P; <a href="mailto:vtk-developers@vtk.org">vtk-developers@vtk.org</a>; <a href="mailto:infovis@public.kitware.com">infovis@public.kitware.com</a><br>
<div class="im">Subject: Re: [InfoVis] Infovis and rendering kits<br>
<br>
</div><div><div></div><div class="h5">> We should consider removing Infovis/vtkTreeMapViewer.  It is basically duplicating functionality found in Views/vtkTreeMapView...<br>
I agree. The right thing to do is probably to mark all its methods as<br>
deprecated until the next release, pointing the user to vtkTreeMapView<br>
in the class comments, then we can remove the class.<br>
<br>
>> So... is Kitware OK with me moving these 5 classes from<br>
>> Infovis to Rendering:<br>
>>     vtkGraphMapper.cxx<br>
Depends on filters in Infovis.<br>
<br>
>>     vtkInteractorStyleAreaSelectHover.cxx<br>
Depends on vtkBalloonRepresentation (in Widgets).<br>
<br>
>>     vtkInteractorStyleTreeMapHover.cxx<br>
Depends on vtkBalloonRepresentation (in Widgets) and vtkTreeMapLayout<br>
(in Infovis).<br>
<br>
>>     vtkLabeledTreeMapDataMapper.cxx<br>
It looks like this could move.<br>
<br>
>>     vtkTreeMapViewer.cxx<br>
Depends on vtkTreeMapLayout (in Infovis).<br>
<br>
>> and this class from Infovis to Views:<br>
>>     vtkViewUpdater.cxx<br>
Yes, this should definitely happen.<br>
<br>
I made a wiki page describing how we could move things in order to<br>
change dependencies:<br>
<br>
<a href="http://www.kitware.com/InfovisWiki/index.php/VTK_Kit_Dependencies" target="_blank">http://www.kitware.com/InfovisWiki/index.php/VTK_Kit_Dependencies</a><br>
<br>
Dave, if you want to try to get this working, please feel free.<br>
<br>
Jeff<br>
<br>
<br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Jeff Baumes, Ph.D.<br>R&D Engineer, Kitware Inc.<br>(518) 881-4932<br><a href="mailto:jeff.baumes@kitware.com">jeff.baumes@kitware.com</a><br>
</div>