[yt-dev] Change to defaults of ProjectionPlot and OffAxisProjectionPlot

[Cameron Hummels]

Hey everyone,

I've created a pull request which changes the defaults of the
ProjectionPlot and OffAxisProjectionPlot, although I'm looking for feedback
from the community.

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)".  It will
do this in the case of non-weighted projections.  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.

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)" ?


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.

Anyway, what do people think about these potential changes?

Cameron


-- 
Cameron Hummels
Postdoctoral Researcher
Steward Observatory
University of Arizona
http://chummels.org
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.spacepope.org/pipermail/yt-dev-spacepope.org/attachments/20140714/5fcfcb1e/attachment.htm>