<div dir="ltr">Oh I'm not suggesting that the code isn't solid, just that one of the main points in doing the refactor was to make the scene stuff more general and able to work with TimeSeries data instead of individual frames one-at-a-time.  I'd just as soon get that into the codebase before it gets released so any api changes are done up front before the api becomes more permanent.<div><br></div><div>What other work is blocked on this PR?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jun 2, 2015 at 12:05 PM, Matthew Turk <span dir="ltr"><<a href="mailto:matthewturk@gmail.com" target="_blank">matthewturk@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 Cameron,<br>
<span class=""><br>
On Tue, Jun 2, 2015 at 1:54 PM, Cameron Hummels <<a href="mailto:chummels@gmail.com">chummels@gmail.com</a>> wrote:<br>
><br>
>><br>
>> Yes -- I posted about it in the slack channel.  Sorry I didn't send it<br>
>> to the list.<br>
>><br>
> Yeah, I've been slacking on checking on slack, as I'm on an review panel<br>
> later this week that's eating up all my time.<br>
<br>
</span>OK -- well, the hangout was very short anyway, and we concluded that<br>
documentation was the big thing left.<br>
<span class=""><br>
><br>
>><br>
>> Do you want to help with documentation, or with porting the remaining<br>
>> cameras?<br>
>><br>
> I'm up for helping with stuff, but my contributions will have to wait until<br>
> the weekend after this panel.  If you want to jump into docs, I can review<br>
> and extend based on your first pass?<br>
<br>
</span>Sure.  I was planning on doing them all by myself anyway, so any help<br>
you can provide would be great.  Because the backwards compatible<br>
layer is already in, and a good amount of other work is now blocked on<br>
this PR, I'm hoping to finish it off right away.<br>
<br>
As a sidenote, in July a planetarium show is coming out that used the<br>
scene refactor for some of its rendering.  So I think the code might<br>
be solid at this point.<br>
<br>
-Matt<br>
<span class="im HOEnZb"><br>
><br>
>><br>
>> > Also, the VR refactor was originally meant to address TimeSeries so<br>
>> > movies<br>
>> > were more easily made (movie paths, etc.).  As the VR code gets merged<br>
>> > into<br>
>> > the mainline, I'd love to work with people on making the Scene class<br>
>> > more<br>
>> > inclusive of TimeSeries data before the 3.2 release goes out.<br>
>><br>
>> Please go ahead and do that!<br>
>><br>
> OK, awesome.  I'll check in with people on the mailing list and slack after<br>
> I've made some progress early next week.<br>
><br>
> Cameron<br>
>><br>
>><br>
> --<br>
> Cameron Hummels<br>
> Postdoctoral Researcher<br>
> Steward Observatory<br>
> University of Arizona<br>
> <a href="http://chummels.org" target="_blank">http://chummels.org</a><br>
><br>
</span><div class="HOEnZb"><div class="h5">> _______________________________________________<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" 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">yt-dev@lists.spacepope.org</a><br>
<a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" 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>-- <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>