[issue31082] reduce takes iterable, not just sequence

Change by Dong-hee Na : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ ___

[issue31082] reduce takes iterable, not just sequence

Dong-hee Na added the comment: New changeset cd3c2bdd5d53db7fe1d546543d3270916552 by Zackery Spytz in branch 'master': bpo-31082: Use "iterable" in the docstring for functools.reduce() (GH-20796) https://github.com/python/cpython/commit/cd3c2bdd5d53db7fe1d546543d3270916552

[issue31082] reduce takes iterable, not just sequence

Change by Zackery Spytz : -- keywords: +patch nosy: +ZackerySpytz nosy_count: 6.0 -> 7.0 pull_requests: +19992 stage: -> patch review pull_request: https://github.com/python/cpython/pull/20796 ___ Python tracker

[issue31082] reduce takes iterable, not just sequence

Serhiy Storchaka added the comment: The term "sequence" also is used in the docstring of itertools.starmap() (actually an iterable is accepted). -- ___ Python tracker

[issue31082] reduce takes iterable, not just sequence

Nick Coghlan added the comment: Checking older versions of the docs, it looks like we simply haven't reworded those docs since Python 2.0 (when the "list" of the 1.5 docs became "sequence" for 2.0). So I think the most appropriate resolution here would be to replace the current use of

[issue31082] reduce takes iterable, not just sequence

Serhiy Storchaka added the comment: Yes, I think it can. But be aware that "sequence" is used not only in the signature, but in following description. The term "sequence" in the documentation and the docstring looks to me like a synonym of "iterable", not a reference to

[issue31082] reduce takes iterable, not just sequence

Terry J. Reedy added the comment: Serhiy, given that the parameters of reduce are currently positional only (by experiment, it has not gotten the C.A. treatment yet), can 'sequence' in the signature in the docstring be changes to 'iterable'? -- nosy: +serhiy.storchaka, terry.reedy

[issue31082] reduce takes iterable, not just sequence

New submission from Stefan Pochmann: functools.reduce has a parameter called "iterable" and it only needs to be an iterable, not a sequence. The paragraph documenting it says "sequence" instead of "iterable" six times: https://docs.python.org/3/library/functools.html#functools.reduce The help