<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 14, 2014 at 9:35 PM, Cameron Hummels <span dir="ltr"><<a href="mailto:chummels@gmail.com" target="_blank">chummels@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 dir="ltr">Hey everyone,<div><br></div><div><div>I've created a pull request which changes the defaults of the ProjectionPlot and OffAxisProjectionPlot, although I'm looking for feedback from the community.  </div>



<div><br></div><div>Right now, when you create a projection, say for "Density", it labels the colorbar with "Density" and then gives its projected units (instead of g/cm^3, it gives g/cm^2).  My PR is simply to change the default label to be "Projected <field>" in this case "Projected Density (g/cm^2)".  </div>

</div></div></blockquote><div><br></div><div>+1. I also sort of like the idea of special-casing  - in particular for density, which I think should show up as "Column Density".</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div dir="ltr"><div><div>It will do this in the case of non-weighted projections.  </div></div></div></blockquote><div><br></div><div>Also only when proj_stype = "integrate".</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div dir="ltr"><div><div>I think this is the expected behavior and more accurate than the former behavior, but I'm open to discussion from the rest of the dev community.</div>

<div><br></div><div>In addition, it might be worthwhile to change the defaults on weighted-projections (e.g. density-weighted temperature projection), to give it and appropriate label as well, but I'm less convinced of this change.  Perhaps something like "<weight_field>-Weighted <field> (units)" ?  </div>

</div></div></blockquote><div><br></div><div>I'm not sure about this.  Whatever we decide on, it should hopefully be compact.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div dir="ltr"><div>

<div><br></div><div>There is also a PR awaiting approval by John Regan that deals with this behavior which will allow users to easily specify whatever label they want for the colorbar, but I thought having a sensible default was appropriate as well.</div>



<div><br></div><div>Anyway, what do people think about these potential changes?</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Cameron</div><div><br></div><div><br></div>-- <br>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>
</font></span></div></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" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
<br></blockquote></div><br></div></div>