[yt-dev] Reason, PlotCollection and yt-3.0

[Sam Skillman]

I'm +1 for deprecation warning in 2.x of PlotCollection, and removal for
3.0.

The one piece that seems to be missing to me is a super-container like what
PlotCollection was. I have a piece of code that does this the way I like
it, but I'm not sure how generally applicable it is.
http://paste.yt-project.org/show/4297/

Allows you to do:
pm = PlotManager()
for ax in 'xyz':
    pm['%s-slice' % ax] = SlicePlot(pf, ax, field_list)

pm.apply_method(".set_cmap('all', 'RdBu_r')")
pm.apply_method(".save()")

I'm happy to PR this if there is interest.


On Mon, Feb 10, 2014 at 7:43 AM, Britton Smith <brittonsmith at gmail.com>wrote:

> I think we have now covered all the functionality of PlotCollection in
> Slice/Projection/Profile/PhasePlot.  I would be +1 for marking it as
> officially deprecated in the next (and final?) 2.x release and getting rid
> of it in 3.0.  I'm worried that the ground has moved enough underneath
> PlotCollection that supporting in any fashion in 3.0 will require a nonzero
> amount of work.
>
>
> On Sun, Feb 9, 2014 at 10:47 PM, Cameron Hummels <chummels at gmail.com>wrote:
>
>> I'm +0 on both of these, but if we are to remove either, as John points
>> out, now seems like a good time before yt goes gold with 3.0.
>>
>> It seems like some people continue to use PlotCollection for things in
>> 2.x.  Are there any use cases that are left uncovered by replacements?
>>
>> I figured reason was going to be replaced by something else wholesale
>> with ipython, but I'm open to either option really.
>>
>> Cameron
>>
>>
>> On Sun, Feb 9, 2014 at 8:42 PM, Nathan Goldbaum <nathan12343 at gmail.com>wrote:
>>
>>> On Sun, Feb 9, 2014 at 7:33 PM, Matthew Turk <matthewturk at gmail.com>
>>> wrote:
>>> >
>>> > On Feb 9, 2014 10:24 PM, "Nathan Goldbaum" <nathan12343 at gmail.com>
>>> wrote:
>>> >>
>>> >> Hi all,
>>> >>
>>> >> I want to raise the possibility of removing the PlotCollection and
>>> >> Reason code from yt before the 3.0 release.
>>> >>
>>> >> I haven't checked recently but given the changes in yt since they were
>>> >> last updated, both features likely do not work correctly right now.
>>> >> Given the move to notebooks* and PlotWindow-style plotting going
>>> >> forward, I think now is the time to rip off the bandage and remove the
>>> >> code.
>>> >>
>>> >> It's likely some of the gui code can be repurposed for a widget, but
>>> >> that doesn't mean reason as it is right now needs to be bundled with
>>> >> the release or exist in the dev repo.
>>> >>
>>> >
>>> > I'm -1 on removing reason wholesale, but -0.5 on removing the command
>>> and
>>> > some mentions of it. Having recently spent some time trying to work
>>> with
>>> > IPython internals, I think there is some utility in an extremely simple
>>> > system (simpler than reason is) that is application focused rather than
>>> > notebook widget focused, even if not globally applicable. Plus the
>>> Reason
>>> > system is pretty convenient for those situations where fork() is not
>>> > implemented.
>>>
>>> I guess that means you'll be using the reason code and are planning on
>>> updating it eventually.  In that case I think it makes sense to keep
>>> it in the repo.  Suggestion withdrawn :)
>>>
>>> >
>>> > I am -0 on removing plot collection, but could be convinced if enough
>>> people
>>> > were in favor.
>>> >
>>> >> -Nathan
>>> >>
>>> >> *As a side note, you should all check out the IPython javascript demos
>>> >> in the release candidate for IPython 2.0.  Since the widgets
>>> >> communicate with a live kernel, you unfortunately can't demo it with a
>>> >> static nbviewer page. Look in the examples folder in the root of the
>>> >> IPython repository.
>>> >> _______________________________________________
>>> >> 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
>>> >
>>> _______________________________________________
>>> 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
>>
>>
>
> _______________________________________________
> yt-dev mailing list
> yt-dev at lists.spacepope.org
> http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.spacepope.org/pipermail/yt-dev-spacepope.org/attachments/20140210/7b4352f2/attachment.htm>