<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Mar 16, 2014 at 11:42 AM, John Wise <span dir="ltr"><<a href="mailto:jwise@physics.gatech.edu" target="_blank">jwise@physics.gatech.edu</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi Nathan,<br>
<br>
I think the default image size is fine at 1000x1000px, which fits on a 1920x1080 screen nicely.  Screen resolution is going to continue increasing, and I don't think it's a good idea to decrease the default.  If anything, it should increase in the future.  For data inspection, I'd rather scroll around large (e.g. 4000x4000) image than zooming/re-centering a 600x600 image numerous times.<br>


<br>
Also, I agree with Matt that we shouldn't be optimizing for notebook operation yet.  Although notebooks are great for interactivity, I find it painfully slow to edit and re-run scripts (even with keyboard shortcuts) when compared to an editor + image viewer.<br>


<br>
Is there a way to make the plot window aware that it's embedded in a notebook and adjust the default image size accordingly?  I think that would be a good plan, if it's technically possible.<br></blockquote><div>

<br></div><div>Unfortunately no.  See this e-mail thread: <a href="http://mail.scipy.org/pipermail/ipython-dev/2014-February/013234.html">http://mail.scipy.org/pipermail/ipython-dev/2014-February/013234.html</a></div><div>

<br></div><div>We could scale down the plot using css, but I don't think that's a good solution since it will make the axes text, tick marks, and colorbar label harder to read.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">


<br>
Cheers,<br>
John<div><div class="h5"><br>
<br>
On 03/16/2014 01:42 PM, Nathan Goldbaum wrote:<br>
</div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><div class="h5">
Hi all,<br>
<br>
I recently opened a PR that fixes a number of bugs in yt's plotting<br>
code.  Hopefully this will fix many common visual issues that show up in<br>
yt plots.<br>
<br>
As part of the PR I decided to change the default size of yt plots from<br>
8 inches to 6.  My justification is that the current default will create<br>
a plot that overflows an IPython notebook output cell, particularly on<br>
small displays.<br>
<br>
The PR is here:<br>
<a href="https://bitbucket.org/yt_analysis/yt/pull-request/723/fixing-some-longstanding-issues-with-plot/diff#comment-1433174" target="_blank">https://bitbucket.org/yt_<u></u>analysis/yt/pull-request/723/<u></u>fixing-some-longstanding-<u></u>issues-with-plot/diff#comment-<u></u>1433174</a><br>


<br>
The discussion about the figure size is here:<br>
<a href="https://bitbucket.org/yt_analysis/yt/pull-request/723/fixing-some-longstanding-issues-with-plot/diff#comment-1432943" target="_blank">https://bitbucket.org/yt_<u></u>analysis/yt/pull-request/723/<u></u>fixing-some-longstanding-<u></u>issues-with-plot/diff#comment-<u></u>1432943</a><br>


<br>
As I mention in the PR description I'm willing to back down on this<br>
change if the consensus is that this is a bad idea.<br>
<br>
Matt has already come out against the change, but before I revert it I<br>
wanted to bring it up here and get opinions from other people.<br>
<br>
Thanks for your help :)<br>
<br>
-Nathan<br>
<br>
<br></div></div><div class="">
______________________________<u></u>_________________<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" target="_blank">http://lists.spacepope.org/<u></u>listinfo.cgi/yt-dev-spacepope.<u></u>org</a><br>
<br>
</div></blockquote><span class=""><font color="#888888">
<br>
-- <br>
John Wise<br>
Assistant Professor of Physics<br>
Center for Relativistic Astrophysics, Georgia Tech<br>
<a href="http://cosmo.gatech.edu" target="_blank">http://cosmo.gatech.edu</a></font></span><div class=""><div class="h5"><br>
______________________________<u></u>_________________<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" target="_blank">http://lists.spacepope.org/<u></u>listinfo.cgi/yt-dev-spacepope.<u></u>org</a><br>
</div></div></blockquote></div><br></div></div>