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

[Matthew Turk]

Hi Cameron,

I'm -0 on this, but mainly because I don't really like changing it to
have that information as part of the colorbar, rather than the title
for instance.

-Matt

On Tue, Jul 15, 2014 at 12:06 AM, Cameron Hummels <chummels at gmail.com> wrote:
> 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
>
> _______________________________________________
> yt-dev mailing list
> yt-dev at lists.spacepope.org
> http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
>