<div dir="ltr">I have used that max_level in the past, for both debugging and restricted analysis (and hope to again soon, for the same reasons).  But if the same functionality is available elsewhere, I don't have any stake in that particular keyword in that particular function.<br>

</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 9, 2013 at 3:07 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"><div class="im">On Thu, May 9, 2013 at 5:05 PM, Nathan Goldbaum <<a href="mailto:nathan12343@gmail.com">nathan12343@gmail.com</a>> wrote:<br>


> Hi all,<br>
><br>
> As part of the eternal journey that is making yt's plotting code as awesome<br>
> as possible, we've endeavored to make the plotting code as flexible as it<br>
> needs to be for quick use but simple enough that a use isn't overloaded by<br>
> unnecessary or barely-used keyword arguments.<br>
><br>
> Right now there is a pending PR to rework some of the plotting routines,<br>
> adding a couple of new features, and, per the subject of this e-mail,<br>
> removing the max_level keyword from ProjectionPlot.<br>
><br>
> I could see how having max_level might be useful if projections are very<br>
> slow for a user's dataset.  However, she should be able to get the exact<br>
> same result by explicitly constructing a projection data object and then<br>
> creating a plot for it using projection.to_pw().  I also think in practice<br>
> max_level isn't used very often since projections are quite fast, even on<br>
> large datasets.<br>
<br>
</div>I'm strongly in favor of keeping the ProjectionPlot, SlicePlot, etc,<br>
all focused on display-related items, rather than data-related items.<br>
I feel like they should be a good way to make sensible default objects<br>
and then detailed modifications of plots from them.  That's why we<br>
have .to_pw(), right?  :)  But I would like to hear if anyone<br>
disagrees and thinks we need to keep max_level.<br>
<div class="im"><br>
><br>
> Am I incorrect in that assessment?  Please let me know if you'd like to keep<br>
> the max_level keyword around and I'll happily revert that part of the pull<br>
> request.<br>
><br>
> Thanks for your help,<br>
><br>
> Nathan<br>
><br>
</div>> _______________________________________________<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>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Sent from my computer.
</div>