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

[Cameron Hummels]

Oh, I almost forgot to show examples:

current behavior of a non-weighted density projection:
http://i.imgur.com/vBSRRLq.png

proposed behavior of a non-weighted density projection:
http://i.imgur.com/UP6f5Nh.png

although i like the idea that Nathan has about having "column density" for
projected density plots.


On Mon, Jul 14, 2014 at 9:41 PM, Nathan Goldbaum <nathan12343 at gmail.com>
wrote:

>
>
>
> On Mon, Jul 14, 2014 at 9:35 PM, Cameron Hummels <chummels at gmail.com>
> wrote:
>
>> 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)".
>>
>
> +1. I also sort of like the idea of special-casing  - in particular for
> density, which I think should show up as "Column Density".
>
>
>> It will do this in the case of non-weighted projections.
>>
>
> Also only when proj_stype = "integrate".
>
>
>> 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)" ?
>>
>>
>
> I'm not sure about this.  Whatever we decide on, it should hopefully be
> compact.
>
>
>>
>> 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
>>
>> _______________________________________________
>> yt-dev mailing list
>> yt-dev at lists.spacepope.org
>> http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
>>
>>
>
> _______________________________________________
> yt-dev mailing list
> yt-dev at lists.spacepope.org
> http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
>
>


-- 
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/1dc4dfee/attachment.htm>