<div class="gmail_extra">Hi, Ken,<br>   Thanks for the suggestion. <br><br>   The order you suggested is exactly what I am following to make the changes,<br><br>   I would also love to hear any interesting work flow /usage scenarios related to temporal data.  <br>
<br>Leo<br><br><div class="gmail_quote">On Fri, Apr 27, 2012 at 2:40 PM, Moreland, Kenneth <span dir="ltr"><<a href="mailto:kmorel@sandia.gov" target="_blank">kmorel@sandia.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




<div style="font-size:14px;font-family:Calibri,sans-serif;word-wrap:break-word">
<div>
<div>
<div>The change sounds OK to me.  Although this class seemed like a natural thing to do when implementing temporal support, it has caused problems in unexpected ways.</div>
<div><br>
</div>
<div>I recommend flipping the order in which you implement this change.  First change all filters to not use vtkTemporalDataSet and replace that with CONTINUE_EXECUTION.  Make sure that the implementation is plausible, straightforward enough for users to implement,
 solve the problems we see with the current implementation, and don't introduce new weird problems.  After that remove vtkTemporalDataSet and the auto pipeline iteration, which at this point should be trivial.</div>
<div><br>
</div>
<div>-Ken</div>
<div>
<div><br>
</div>
</div>
</div>
</div>
<span>
<div style="border-right:medium none;padding-right:0in;padding-left:0in;padding-top:3pt;text-align:left;font-size:11pt;border-bottom:medium none;font-family:Calibri;border-top:#b5c4df 1pt solid;padding-bottom:0in;border-left:medium none">

<span style="font-weight:bold">From: </span>Yuanxin Liu <<a href="mailto:leo.liu@kitware.com" target="_blank">leo.liu@kitware.com</a>><br>
<span style="font-weight:bold">Date: </span>Friday, April 27, 2012 11:40 AM<br>
<span style="font-weight:bold">To: </span>VTK Developers <<a href="mailto:vtk-developers@vtk.org" target="_blank">vtk-developers@vtk.org</a>><br>
<span style="font-weight:bold">Cc: </span>Kitware-developers <<a href="mailto:kitware-developers@kitware.com" target="_blank">kitware-developers@kitware.com</a>><br>
<span style="font-weight:bold">Subject: </span>[EXTERNAL] [vtk-developers] changes in temporal support in VTK<br>
</div><div><div class="h5">
<div><br>
</div>
<blockquote style="BORDER-LEFT:#b5c4df 5 solid;PADDING:0 0 0 5;MARGIN:0 0 0 5">
<div>
<div>Hi, <br>
  We are currently considering a few backward-incompatible changes to the temporal support in the VTK pipeline and would like to hear comments.  We are also interested in hearing other suggestions to improve handling of time-dependent data.<br>

<br>
  The main change is to remove the use of vtkTemporalDataSet and to limit the time request to be a single time step, i.e. 
<ul>
<li>Currently, a filter can request objects from multiple time steps by setting the key UPDATE_TIME_STEPS to a vector of doubles; the resulting objects get wrapped by the pipeline into a single vtkTemporalDataSet object.<br>

</li><li>After the change, a filter can only request a single time step by setting the key UPDATE_TIME_STEP to a single double.  The "wrapping" no longer happens.<br>
</li></ul>
  For existing filters that depend on multiple time steps,  we will modify them to use the CONTINUE_EXECUTION key to iteratively execute the pipeline and store the objects from the iterations.   There will be super classes/utility classes to make this easy.<br>

<br>
  The main motivation is that, by pushing the temporal functionality from the core pipeline to the filters, the execution pipeline becomes simpler and easier to maintain. Some bugs, e.g.( ParaView bug
<a href="http://www.paraview.org/Bug/view.php?id=6662" target="_blank">Id=6662</a>) will go away as a result.<br>
<br>
Leo </div>
</div>
</blockquote>
</div></div></span>
</div>

</blockquote></div><br></div>