Repository navigation
math.factorial doc should mention integer return type #69921
Description
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
assignee = None
closed_at = <Date 2019-05-31.16:46:33.420>
created_at = <Date 2015-11-25.19:44:31.461>
labels = ['3.7', '3.8', 'docs']
title = 'math.factorial doc should mention integer return type'
updated_at = <Date 2019-05-31.16:58:30.519>
user = 'https://bugs.python.org/JohnYeung'bugs.python.org fields:
activity = <Date 2019-05-31.16:58:30.519>
actor = 'miss-islington'
assignee = 'docs@python'
closed = True
closed_date = <Date 2019-05-31.16:46:33.420>
closer = 'cheryl.sabella'
components = ['Documentation']
creation = <Date 2015-11-25.19:44:31.461>
creator = 'John.Yeung'
dependencies = []
files = ['41206']
hgrepos = []
issue_num = 25735
keywords = ['patch']
message_count = 11.0
messages = ['255382', '255479', '255542', '255693', '313909', '313910', '313911', '313913', '344088', '344092', '344093']
nosy_count = 10.0
nosy_names = ['rhettinger', 'terry.reedy', 'facundobatista', 'mark.dickinson', 'skrah', 'docs@python', 'John.Yeung', 'mine0901', 'cheryl.sabella', 'miss-islington']
pr_nums = ['6420', '13702']
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue25735'
versions = ['Python 3.7', 'Python 3.8']
Activity
JohnYeung commentedon Nov 25, 2015
The math module docs state
Except when explicitly noted otherwise, all return values are floats.
But math.factorial isn't what I would call explicit about returning int:
At minimum, shouldn't the first sentence be "Return x factorial as an int."? I haven't tested on all Python versions, but math.factorial on 2.7 and 3.2 definitely return int (or long in Python 2 when necessary).
terryjreedy commentedon Nov 27, 2015
mdickinson commentedon Nov 28, 2015
[Terry]
>>> factorial(decimal.Decimal(5.2)) 120Yep, that's definitely wrong. If we want to behave the same way as for float, we should accept only integral Decimal values. (Though I'm not much of a fan of the float behaviour: I would have preferred math.factorial not to accept floats at all.)
mine0901 commentedon Dec 2, 2015
States that math.factorial(x) returns x factorial as an integer.
csabella commentedon Mar 15, 2018
Should the documentation patch for this be converted to a PR or does this need to change to a bug issue to address Mark's concerns?
Thanks!
mdickinson commentedon Mar 15, 2018
I think so, yes. How about we open a new issue for the factorial(Decimal(...)) behaviour, and keep this one focused on the original reported issue?
csabella commentedon Mar 15, 2018
Sounds good. Thanks, Mark.
@mine0901, would you like to open a Github pull request for your patch? Thanks!
mdickinson commentedon Mar 15, 2018
I opened bpo-33083, and copied the nosy list from this issue across.
csabella commentedon May 31, 2019
New changeset 4612671 by Cheryl Sabella (Akshay Sharma) in branch 'master':
bpo-25735: math.factorial doc should mention integer return type (GH-6420)
4612671
csabella commentedon May 31, 2019
@John.Yeung, thank you for the report. @mine0901, thanks for the initial patch and @akshaysharma, thank you for the PR.
miss-islington commentedon May 31, 2019
New changeset fc3b843 by Miss Islington (bot) in branch '3.7':
bpo-25735: math.factorial doc should mention integer return type (GH-6420)
fc3b843