Nathan,<div><br></div><div>Sorry, I may have been unclear. I definitely wanted the user to get a PlotWindow (or PW-like) object back. Your approach is great.</div><div><br></div><div>j<br><br>On Saturday, June 23, 2012, Nathan Goldbaum  wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Jeff,<br>
<br>
Yeah, I wanted to expose some of the convenience functions in the PlotWindow class (e.g. zoom() and pan()) and also keep the save() method.  If the plots return matplotlib axes objects then the user would have to manually generate the filename string to pass to savefig().  I really like how yt appends the field name to the filename of images it produces.<br>

<br>
I'm open to returning an MPL axes object instead, but I'd like to find a way to keep the nice filename conventions we have in yt.<br>
<br>
-Nathan<br>
<br>
On Jun 23, 2012, at 4:51 PM, j s oishi wrote:<br>
<br>
> Hi guys,<br>
><br>
> Things are looking great. I'm intrigued by making the plot commands functions returning PWViewerMPL objects. I'm guessing these are essentially end-user convenience functions, which is cool. How do you envision dealing with future backend swaps? Just rewriting the convenience function?<br>

><br>
> Regardless, I think this is shaping up very well.<br>
><br>
> j<br>
> On Saturday, June 23, 2012, Matthew Turk wrote:<br>
> Awesome work, Nathan!  I've looked these over and seen the example<br>
> images and they look good, but maybe Jeff could take a quick look at<br>
> how the PlotWindow is shaping up.<br>
><br>
><br>
> ---------- Forwarded message ----------<br>
> From: Nathan Goldbaum <<a href="javascript:;" onclick="_e(event, 'cvml', 'pullrequests-noreply@bitbucket.org')">pullrequests-noreply@bitbucket.org</a>><br>
> Date: Sat, Jun 23, 2012 at 4:39 PM<br>
> Subject: [yt_analysis/yt] Adding OffAxisSlice (pull request #178)<br>
> To: <a href="javascript:;" onclick="_e(event, 'cvml', 'matthewturk@gmail.com')">matthewturk@gmail.com</a><br>
><br>
><br>
> A new pull request has been opened by Nathan Goldbaum.<br>
><br>
> ngoldbaum/yt-ngoldbaum has changes to be pulled into yt_analysis/yt.<br>
><br>
> <a href="https://bitbucket.org/yt_analysis/yt/pull-request/178/adding-offaxisslice" target="_blank">https://bitbucket.org/yt_analysis/yt/pull-request/178/adding-offaxisslice</a><br>
><br>
> Title: Adding OffAxisSlice<br>
><br>
> This finishes up the new plotting interface by adding OffAxisSlice.<br>
> I've also taken the opportunity to add a docstring for<br>
> get_rotation_matrix() in utilities/math_utils.py and to correct a bug<br>
> in ObliqueFixedResolutionBuffer that caused the buffer to be inverted.<br>
><br>
> Changes to be pulled:<br>
><br>
><br>
><br>
> --<br>
> This is an issue notification from <a href="http://bitbucket.org" target="_blank">bitbucket.org</a>.<br>
> You are receiving this either because you are the participating<br>
> in a pull request, or you are following it.<br>
> _______________________________________________<br>
> yt-dev mailing list<br>
> <a href="javascript:;" onclick="_e(event, 'cvml', '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>
> !DSPAM:10175,4fe62c6215777636615206! _______________________________________________<br>
> yt-dev mailing list<br>
> <a href="javascript:;" onclick="_e(event, 'cvml', '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>
> !DSPAM:10175,4fe62c6215777636615206!<br>
<br>
_______________________________________________<br>
yt-dev mailing list<br>
<a href="javascript:;" onclick="_e(event, 'cvml', '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>