[Numpy-discussion] Pull Request Review: R-like sample function

Christopher Jordan-Squire cjordan1 at uw.edu
Fri Sep 2 13:14:00 EDT 2011

I made the changes discussed here and pushed them to pull request.


I changed the new function's name from sample to choice and added the
size=1 default as Robert suggested. I also reverted all the changes
for sample, random, and ranf and added them to the reference docs.

-Chris JS

On Thu, Sep 1, 2011 at 10:55 PM, Nathaniel Smith <njs at pobox.com> wrote:
> On Thu, Sep 1, 2011 at 8:31 PM, Christopher Jordan-Squire
> <cjordan1 at uw.edu> wrote:
>> On Thu, Sep 1, 2011 at 11:14 PM, Robert Kern <robert.kern at gmail.com> wrote:
>>> On Thu, Sep 1, 2011 at 22:07, Christopher Jordan-Squire <cjordan1 at uw.edu> wrote:
>>>> So in the mean time, are there any suggestions for what this R sample
>>>> function should be called, since random.sample is apparently taken?
>>> If you default to size=1 (which you probably should anyways), then
>>> np.random.choice() makes sense, in analogy to random.choice() from the
>>> standard library.
>> Alright. I can make that change tomorrow. I'd prefer np.sample in the
>> long-run, for compatibility with R. (False friends are loathsome
>> things.)  How does one petition to get function names deprecated?
> I was about to argue that "random.choice" was a better name anyway,
> but then I remembered that the standard library "random.sample" exists
> and does something similar. So instead I'd like to argue that making
> this compatible with Python is more important than making it
> compatible with R :-).
> Specifically, 'np.random.sample(array_like, k)', with no further
> arguments, should perform random sampling *without* replacement.
> Ideally it should also efficiently handle the case where array_like is
> an xrange object. Docs are here:
> http://docs.python.org/library/random.html
> -- Nathaniel
> _______________________________________________
> NumPy-Discussion mailing list
> NumPy-Discussion at scipy.org
> http://mail.scipy.org/mailman/listinfo/numpy-discussion

More information about the NumPy-Discussion mailing list