Can you leave a comment to that effect on the PR so I don't forget?<span></span><br><br>On Thursday, June 19, 2014, Matthew Turk <<a href="mailto:matthewturk@gmail.com">matthewturk@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Nathan,<br>
<br>
I'm ready to hit the button any time, although I wonder if we ought to<br>
have a compat layer for a little while -- a property for .pf for<br>
instance.<br>
<br>
-Matt<br>
<br>
On Thu, Jun 19, 2014 at 1:12 AM, Nathan Goldbaum <<a href="javascript:;" onclick="_e(event, 'cvml', 'nathan12343@gmail.com')">nathan12343@gmail.com</a>> wrote:<br>
> This is ready now and passes all of the unit and answer tests:<br>
><br>
> <a href="https://bitbucket.org/yt_analysis/yt/pull-request/954/replacing-pf-with-ds/diff#chg-doc/docstring_idioms.txt" target="_blank">https://bitbucket.org/yt_analysis/yt/pull-request/954/replacing-pf-with-ds/diff#chg-doc/docstring_idioms.txt</a><br>

><br>
> Should we wait to merge this in until we can cut down on the PR backlog?<br>
><br>
><br>
> On Thu, Jun 12, 2014 at 9:29 AM, Matthew Turk <<a href="javascript:;" onclick="_e(event, 'cvml', 'matthewturk@gmail.com')">matthewturk@gmail.com</a>> wrote:<br>
>><br>
>> We usually use "convert-repo" for this purpose.<br>
>><br>
>> On Thu, Jun 12, 2014 at 10:27 AM, Sam Skillman <<a href="javascript:;" onclick="_e(event, 'cvml', 'samskillman@gmail.com')">samskillman@gmail.com</a>><br>
>> wrote:<br>
>> > +1, as it is in line with<br>
>> > <a href="http://ytep.readthedocs.org/en/latest/YTEPs/YTEP-0017.html" target="_blank">http://ytep.readthedocs.org/en/latest/YTEPs/YTEP-0017.html</a>. Perhaps use<br>
>> > a<br>
>> > different username like 'rename-pf' or something so that hg blame is<br>
>> > useful<br>
>> > for figuring out what was the cause? Also this should probably be done<br>
>> > at a<br>
>> > point when the number of open PRs is low.<br>
>> ><br>
>> ><br>
>> > On Wed, Jun 11, 2014 at 3:40 PM, Cameron Hummels <<a href="javascript:;" onclick="_e(event, 'cvml', 'chummels@gmail.com')">chummels@gmail.com</a>><br>
>> > wrote:<br>
>> >><br>
>> >> +1 since this will provide consistency with docs and make more sense<br>
>> >> for<br>
>> >> new users.  I support doing this asap so that any bugs can be dealt<br>
>> >> with<br>
>> >> prior to release.<br>
>> >><br>
>> >> Cameron<br>
>> >><br>
>> >><br>
>> >> On Wed, Jun 11, 2014 at 3:36 PM, Stuart Mumford <<a href="javascript:;" onclick="_e(event, 'cvml', 'stuart@mumford.me.uk')">stuart@mumford.me.uk</a>><br>
>> >> wrote:<br>
>> >>><br>
>> >>> +1 fwiw<br>
>> >>><br>
>> >>> Stuart<br>
>> >>><br>
>> >>> On 11 Jun 2014 22:55, "John ZuHone" <<a href="javascript:;" onclick="_e(event, 'cvml', 'jzuhone@gmail.com')">jzuhone@gmail.com</a>> wrote:<br>
>> >>>><br>
>> >>>> +1 for me.<br>
>> >>>><br>
>> >>>> On Jun 11, 2014, at 5:53 PM, Nathan Goldbaum <<a href="javascript:;" onclick="_e(event, 'cvml', 'nathan12343@gmail.com')">nathan12343@gmail.com</a>><br>
>> >>>> wrote:<br>
>> >>>><br>
>> >>>> > Hi all,<br>
>> >>>> ><br>
>> >>>> > Before 3.0 goes out, how do you all feel about a PR that replaces<br>
>> >>>> > "pf"<br>
>> >>>> > with "ds" across the whole codebase?<br>
>> >>>> ><br>
>> >>>> > I think this is worth doing, since it reduces the cognitive load of<br>
>> >>>> > dealing with yt internals.  One downside is that it will break user<br>
>> >>>> > scripts<br>
>> >>>> > if they're using internal APIs.  It will also be a big,<br>
>> >>>> > difficult-to-review<br>
>> >>>> > PR.<br>
>> >>>> ><br>
>> >>>> > Now is the time to do this if it ever gets done since there are a<br>
>> >>>> > large number of other breaking changes in 3.0.<br>
>> >>>> ><br>
>> >>>> > I'd be happy to do this if there's a consensus that it's worth<br>
>> >>>> > doing.<br>
>> >>>> ><br>
>> >>>> > Nathan<br>
>> >>>> > _______________________________________________<br>
>> >>>> > yt-dev mailing list<br>
>> >>>> > <a href="javascript:;" onclick="_e(event, 'cvml', '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>
>> >>>> _______________________________________________<br>
>> >>>> yt-dev mailing list<br>
>> >>>> <a href="javascript:;" onclick="_e(event, 'cvml', '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>
>> >>><br>
>> >>> _______________________________________________<br>
>> >>> yt-dev mailing list<br>
>> >>> <a href="javascript:;" onclick="_e(event, 'cvml', '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>
>> >><br>
>> >><br>
>> >><br>
>> >> --<br>
>> >> Cameron Hummels<br>
>> >> Postdoctoral Researcher<br>
>> >> Steward Observatory<br>
>> >> University of Arizona<br>
>> >> <a href="http://chummels.org" target="_blank">http://chummels.org</a><br>
>> >><br>
>> >> _______________________________________________<br>
>> >> yt-dev mailing list<br>
>> >> <a href="javascript:;" onclick="_e(event, 'cvml', '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>
>> ><br>
>> ><br>
>> > _______________________________________________<br>
>> > yt-dev mailing list<br>
>> > <a href="javascript:;" onclick="_e(event, 'cvml', '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>
>> _______________________________________________<br>
>> yt-dev mailing list<br>
>> <a href="javascript:;" onclick="_e(event, 'cvml', '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>
><br>
><br>
> _______________________________________________<br>
> yt-dev mailing list<br>
> <a href="javascript:;" onclick="_e(event, 'cvml', '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>
_______________________________________________<br>
yt-dev mailing list<br>
<a href="javascript:;" onclick="_e(event, 'cvml', '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>
</blockquote>