<div dir="ltr">This is ready now and passes all of the unit and answer tests:<div><br></div><div><a href="https://bitbucket.org/yt_analysis/yt/pull-request/954/replacing-pf-with-ds/diff#chg-doc/docstring_idioms.txt">https://bitbucket.org/yt_analysis/yt/pull-request/954/replacing-pf-with-ds/diff#chg-doc/docstring_idioms.txt</a><br>

</div><div><br></div><div>Should we wait to merge this in until we can cut down on the PR backlog?</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jun 12, 2014 at 9:29 AM, Matthew Turk <span dir="ltr"><<a href="mailto:matthewturk@gmail.com" target="_blank">matthewturk@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">We usually use "convert-repo" for this purpose.<br>
<div class="HOEnZb"><div class="h5"><br>
On Thu, Jun 12, 2014 at 10:27 AM, Sam Skillman <<a href="mailto:samskillman@gmail.com">samskillman@gmail.com</a>> 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 a<br>
> different username like 'rename-pf' or something so that hg blame is useful<br>
> for figuring out what was the cause? Also this should probably be done 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="mailto:chummels@gmail.com">chummels@gmail.com</a>> wrote:<br>
>><br>
>> +1 since this will provide consistency with docs and make more sense for<br>
>> new users.  I support doing this asap so that any bugs can be dealt with<br>
>> prior to release.<br>
>><br>
>> Cameron<br>
>><br>
>><br>
>> On Wed, Jun 11, 2014 at 3:36 PM, Stuart Mumford <<a href="mailto: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="mailto: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="mailto: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 "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 scripts<br>
>>>> > if they're using internal APIs.  It will also be a big, 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 doing.<br>
>>>> ><br>
>>>> > Nathan<br>
>>>> > _______________________________________________<br>
>>>> > yt-dev mailing list<br>
>>>> > <a href="mailto: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="mailto: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="mailto: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="mailto: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="mailto: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="mailto: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>
</div></div></blockquote></div><br></div>