<div dir="ltr">John:<div><br></div><div>Please do not merge this yet.  As the main author of the ytep on the VR refactor, I have been meaning to look at it the last couple of weeks, but I had an unforeseen accident and surgery last weekend that prevented me from doing it.  I will look at it in the next day or two.  My main concern is that the functionality fits into the larger goals of the VR refactor in working with TimeSeries datasets instead of just working with single outputs, and I want to do a full review of the code to make sure everything works as advertised.  I've started this, but it's a big merge, and it is taking some time.  Given that this is a large undertaking and the key piece of they yt 3.2 release, I want to make sure it doesn't have any problems.</div><div><br></div><div>Cameron</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 22, 2015 at 3:59 PM, Michael Zingale <span dir="ltr"><<a href="mailto:michael.zingale@stonybrook.edu" target="_blank">michael.zingale@stonybrook.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I've been meaning to play with this, but I've been traveling.  I'll take a look later this week, but if others have already approved, then it may as well go in.</div><div class="gmail_extra"><div><div class="h5"><br><div class="gmail_quote">On Mon, Jun 22, 2015 at 5:11 PM, John Zuhone <span dir="ltr"><<a href="mailto:jzuhone@gmail.com" target="_blank">jzuhone@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Matt,<br>
<br>
VR refactor’s tests all pass and we have three approvals already. Are we raising the bar higher for including this one?<br>
<br>
If we are, we should probably specify how many approvals we need. If not, then we should just merge it.<br>
<br>
Best,<br>
<br>
John<br>
<div><div><br>
> On Jun 22, 2015, at 1:18 PM, Matthew Turk <<a href="mailto:matthewturk@gmail.com" target="_blank">matthewturk@gmail.com</a>> wrote:<br>
><br>
> Hi all,<br>
><br>
> There are a few big PRs still open.  I wanted to update you on them:<br>
><br>
> * VR refactor:<br>
> <a href="https://bitbucket.org/yt_analysis/yt/pull-request/1256/wip-volume-rendering-refactor" rel="noreferrer" target="_blank">https://bitbucket.org/yt_analysis/yt/pull-request/1256/wip-volume-rendering-refactor</a><br>
> .  This is ready for review.<br>
><br>
> * Particle octree refactor:<br>
> <a href="https://bitbucket.org/yt_analysis/yt/pull-request/1206/wip-forest-of-octrees" rel="noreferrer" target="_blank">https://bitbucket.org/yt_analysis/yt/pull-request/1206/wip-forest-of-octrees</a><br>
> .  This will simply not work in its current form.  I have redesigned<br>
> it around nested bitmap indices but not yet pushed to my fork; I'm<br>
> hoping to get to it sometime in the next few weeks.<br>
><br>
> * Hybrid Rockstar:<br>
> <a href="https://bitbucket.org/yt_analysis/yt/pull-request/1618/wip-hybrid-rockstar" rel="noreferrer" target="_blank">https://bitbucket.org/yt_analysis/yt/pull-request/1618/wip-hybrid-rockstar</a><br>
> .  Hilary has done some awesome work on this, and I think that it will<br>
> work nicely, but parallelism might need some help (according to her<br>
> bullet points)<br>
><br>
> * Grid visitors: The preliminary PR was accepted, and I am working on<br>
> a new set of PRs that will enable this broadly.  This should reduce<br>
> memory overhead.<br>
><br>
> * OpenGL volume rendering:<br>
> <a href="https://bitbucket.org/yt_analysis/yt/pull-request/1598/opengl-volume-rendering" rel="noreferrer" target="_blank">https://bitbucket.org/yt_analysis/yt/pull-request/1598/opengl-volume-rendering</a><br>
> .  This is blocked on the VR refactor, and once that is in I'll<br>
> continue to update this to utilize the new scene objects.<br>
><br>
> -Matt<br>
> _______________________________________________<br>
> yt-dev mailing list<br>
> <a href="mailto:yt-dev@lists.spacepope.org" target="_blank">yt-dev@lists.spacepope.org</a><br>
> <a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" rel="noreferrer" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
<br>
_______________________________________________<br>
yt-dev mailing list<br>
<a href="mailto:yt-dev@lists.spacepope.org" target="_blank">yt-dev@lists.spacepope.org</a><br>
<a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" rel="noreferrer" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br><div><div>Michael Zingale</div><div>Associate Professor</div><div><br></div><div>Dept. of Physics & Astronomy • Stony Brook University • Stony Brook, NY 11794-3800</div><div><i>phone</i>:  <a href="tel:631-632-8225" value="+16316328225" target="_blank">631-632-8225</a></div><div><i>e-mail</i>: <a href="mailto:Michael.Zingale@stonybrook.edu" target="_blank">Michael.Zingale@stonybrook.edu</a></div><div><i>web</i>: <a href="http://www.astro.sunysb.edu/mzingale" target="_blank">http://www.astro.sunysb.edu/mzingale</a></div></div>
</font></span></div>
<br>_______________________________________________<br>
yt-dev mailing list<br>
<a href="mailto:yt-dev@lists.spacepope.org">yt-dev@lists.spacepope.org</a><br>
<a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" rel="noreferrer" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Cameron Hummels<div>Postdoctoral Researcher</div><div>Steward Observatory</div><div>University of Arizona</div><div><a href="http://chummels.org" target="_blank">http://chummels.org</a></div></div>
</div>