[issue33292] Fix secrets.randbelow docstring

[Vex Woo]

Vex Woo  added the comment:

It looks simple and good.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33292] Fix secrets.randbelow docstring

[Mark Dickinson]

Mark Dickinson  added the comment:

"""Return a random integer x satisfying 0 <= x < n""" ?

--
nosy: +mark.dickinson

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33292] Fix secrets.randbelow docstring

[Steven D'Aprano]

Steven D'Aprano  added the comment:

Possibly you may not be familiar with interval notation?

https://www.mathsisfun.com/sets/intervals.html

Since this is Python and not maths, maybe we should rethink the wording.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33292] Fix secrets.randbelow docstring

[Serhiy Storchaka]

Serhiy Storchaka  added the comment:

I don't think that intervals [0, n-1] or (-1, n) would look better than [0, n).

--
resolution:  -> not a bug
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33292] Fix secrets.randbelow docstring

[Vex Woo]

Vex Woo  added the comment:

Please use () or []

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33292] Fix secrets.randbelow docstring

[Serhiy Storchaka]

Serhiy Storchaka  added the comment:

What is wrong with the current docstring?

--
nosy: +serhiy.storchaka, steven.daprano

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33292] Fix secrets.randbelow docstring

[Roundup Robot]

Change by Roundup Robot :


--
keywords: +patch
pull_requests: +6194
stage:  -> patch review

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33292] Fix secrets.randbelow docstring

[Vex Woo]

New submission from Vex Woo :

- https://github.com/python/cpython/blob/master/Lib/secrets.py#L28

Please fix

"""Return a random int in the range [0, n)."""

to 

"""Return a random int in the range(0, n)."""

--
components: Library (Lib)
messages: 315376
nosy: Nixawk
priority: normal
severity: normal
status: open
title: Fix secrets.randbelow docstring
type: resource usage
versions: Python 3.8

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com