<div dir="ltr">Hi John,<div><br></div><div>I am trying to remember the history of why that code is there, and the reason escapes me right now. </div><div><br></div><div>Pat Marion: Do you remember this?</div><div><br></div><div>I propose that we remove it from the plugin until a time that we can sync with your latest work.</div><div><br></div><div>Does that sound reasonable?</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 14, 2015 at 5:03 AM, Biddiscombe, John A. <span dir="ltr"><<a href="mailto:biddisco@cscs.ch" target="_blank">biddisco@cscs.ch</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div lang="EN-GB" link="blue" vlink="purple">
<div>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Casey<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">I had a look at the PCL plugin and couldn’t help notice that you’ve copied some of my particle partition filtering code into the plugin, though it does not
 look functional as it’s not enabled in the cmake and does not include the relevant Zoltan sources or the vtkZoltanV1PartitionFilter base class.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">The partitioning (both particle and mesh) have been refactored (from the pv-meshles plugin) into their own self contained plugin for some time now and it would
 be cleaner in my view to have an option in the PCL plugin to enable parallel partitioning and use the pv-zoltan plugin as a submodule where necessary<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><a href="https://github.com/biddisco/pv-zoltan" target="_blank">https://github.com/biddisco/pv-zoltan</a><u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">(I tend to use GitExternal and SubProject cmake modules rather than a git submodule, I can send links if you’re interested).<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">The pv-zoltan code is currently under development (and is working with pv-4.4/master) to improve it as part of a GSOC project and should soon be compatible
 with zoltan2 rather than Zoltan. I can make sure a pv-4.4+zoltan1 compatible branch remains functional should we make major changes.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">I might even volunteer to do the clean integration of pv-zoltan with the pcl plugin should you be in need of volunteers.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">JB<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><b><span lang="EN-US" style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span lang="EN-US" style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> ParaView [mailto:<a href="mailto:paraview-bounces@paraview.org" target="_blank">paraview-bounces@paraview.org</a>]
<b>On Behalf Of </b>Casey Goodlett<br>
<b>Sent:</b> 13 July 2015 21:16<br>
<b>To:</b> Eric Younkin - NOAA Federal<br>
<b>Cc:</b> paraview<br>
<b>Subject:</b> Re: [Paraview] PCL Plugin<u></u><u></u></span></p><div><div class="h5">
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal">On Mon, Jul 13, 2015 at 1:32 PM, Eric Younkin - NOAA Federal <<a href="mailto:eric.g.younkin@noaa.gov" target="_blank">eric.g.younkin@noaa.gov</a>> wrote:<u></u><u></u></p>
<div>
<p class="MsoNormal">Casey,<u></u><u></u></p>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">I don't mean to take up so much of your time, but I've been reviewing the cmake configuration and I found that there were a number of HDF5 tests that failed (see below).  Is that significant?<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">I do not think this is a problem.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"> <u></u><u></u></p>
</div>
<blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0cm 0cm 0cm 6.0pt;margin-left:4.8pt;margin-right:0cm">
<div>
<div>
<p style="margin:0cm;margin-bottom:.0001pt">Also, one of the errors I saw upon rebuilding:<u></u><u></u></p>
<p style="margin:0cm;margin-bottom:.0001pt"><u></u> <u></u></p>
<p style="margin:0cm;margin-bottom:.0001pt"><b>error C1041: cannot open program database 'c:\users\eric.g.younkin\documents\pcl_paraview\paraview-v4.3.1-source\build\plugins\sciberquesttoolkit\paraviewplugin\sciberquesttoolkit.dir\debug\vc120.pdb'; if multiple
 CL.EXE write to the same .PDB file, please use /FS C:\Users\eric.g.younkin\Documents\PCL_ParaView\ParaView-v4.3.1-source\build\Plugins\SciberQuestToolKit\ParaViewPlugin\moc_pqSQFieldTracerImplementation.cpp)</b><u></u><u></u></p>
<p style="margin:0cm;margin-bottom:.0001pt"><u></u> <u></u></p>
<p style="margin:0cm;margin-bottom:.0001pt">Led me to the moc_pqSQFieldTracerImplementation.cpp which had a comment at the top stating<u></u><u></u></p>
<p style="margin:0cm;margin-bottom:.0001pt"><u></u> <u></u></p>
<p style="margin:0cm;margin-bottom:.0001pt"><b>"This file was generated using the moc from 5.5.0. It cannot be used with the include files from this version of Qt. (The moc has changed too much.)"</b><u></u><u></u></p>
<p style="margin:0cm;margin-bottom:.0001pt"><u></u> <u></u></p>
<p style="margin:0cm;margin-bottom:.0001pt">So I will try with Qt5.4.1 instead.<u></u><u></u></p>
<p style="margin:0cm;margin-bottom:.0001pt"><u></u> <u></u></p>
</div>
</div>
</blockquote>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">I am not familiar with this problem.  Maybe the filename is getting too long for windows?  Maybe another developer can help with the Qt issues, I am not an expert in this area.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Regards,<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"> <u></u><u></u></p>
</div>
</div>
<p class="MsoNormal">-- <u></u><u></u></p>
<div>
<div>
<div>
<div>
<p class="MsoNormal">Casey B Goodlett, PhD<br>
Technical Leader<br>
Kitware, Inc. - North Carolina Office<br>
<a href="http://www.kitware.com" target="_blank">http://www.kitware.com</a><br>
<a href="tel:%28919%29%20969-6990%20x310" target="_blank">(919) 969-6990 x310</a><u></u><u></u></p>
</div>
</div>
</div>
</div>
</div>
</div>
</div></div></div>
</div>

</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr">Casey B Goodlett, PhD<br>Technical Leader<br>Kitware, Inc. - North Carolina Office<br><a href="http://www.kitware.com" target="_blank">http://www.kitware.com</a><br>(919) 969-6990 x310</div></div></div></div>
</div>