Remove limits on the builtin len() function where possible.
まだ誰も着手していません。
評価
調査の方向性
まず組み込みの len() 実装と PyObject_Size C API を調べます。この issue ではリポジトリ内のファイルやテストは指定されていません。説明されている range と random.py の呼び出し経路を比較し、その後、大きな range の値とユーザー定義の長さが報告された OverflowError なしで機能することを確認します。
索引モデルが issue の本文から書いたものです。
説明
Problem
Pure Python __len__ methods can return values larger than sys.maxsize:
from sys import maxsize
class A:
def __len__(self):
return maxsize * 2
>>> A().__len__()
18446744073709551614
However, the builtin len() function unnecessarily fails:
>>> len(A())
Traceback (most recent call last):
...
OverflowError: cannot fit 'int' into an index-sized integer
The builtin range() type added support for ranges larger than sys.maxsize. Larger indices work; negative indices work; forward iteration works; reverse iteration works; and access to the attributes work:
>>> s = range(maxsize*10)
>>> s[maxsize * 2]
18446744073709551614
>>> s[-1]
92233720368547758069
>>> next(iter(s))
0
>>> next(reversed(s))
92233720368547758069
>>> s.start, s.stop, s.step
(0, 92233720368547758070, 1)
However, len() unnecessarily fails:
len(s)
Traceback (most recent call last):
...
Error: Python int too large to convert to C ssize_t
The random.sample() and random.choice() functions both depend on the builtin len() function, so they unnecessarily fail when used with large range objects or with large user defined sequence objects. Users have reported this issue on multiple occasions. We closed those issues because there was no practical way to fix them short of repairing the builtin len() function:
>>> import random
>>> random.choice(range(maxsize * 5))
Traceback (most recent call last):
...
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/random.py", line 371, in choice
return seq[self._randbelow(len(seq))]
OverflowError: Python int too large to convert to C ssize_t
>>> random.sample(range(maxsize * 5), k=10)
Traceback (most recent call last):
...
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/random.py", line 438, in sample
n = len(population)
OverflowError: Python int too large to convert to C ssize_t
Proposal
Make the builtin len() function smarter. Let it continue to first try the C PyObject_Size() function which is restricted to Py_ssize_t. Now add two new fallbacks, one for range objects and the other for calling the __len__ method though the C API allowing arbitrary objects to be returned.
Rough sketch:
def builtin_len(obj):
try:
ssize_t_int = PyObject_Size(obj)
return PyLong_FromSsize_t(ssize_t_int)
except OverflowError:
pass
if isinstance(obj, type(range)):
start, stop, step = obj.start, obj.stop, obj.step
assert step != 0
if step > 0:
return (stop - start + step - 1) // step
return (start - stop - step - 1) // -step
return PyObject_CallMethod(obj, '__len__', NULL)
Bug or Feature Request
Traditionally, extending support for sizes beyond Py_ssize_t has been considered a new feature, range() and itertools.count() for example.
In this case though, arguably it is a bug because the range() support was only 90% complete, leaving off the ability to call len(). Also it could be considered a bug because users could always write a __len__ method returning values larger than Py_ssize_t and could access that value with obj.__len__ but the len() function inexplicably failed due to an unnecessary and implementation dependent range restriction.
Other other thought: maxsize varies across builds, so it is easily possible to get code tested and working on one Python and have it fail on another. All 32-bits builds are affected and all Windows builds.
It would be easy for us to remove the artificial limitation for range objects and for objects that define __len__ directly rather than through sq_length or mp_length. That includes all pure Python classes and any C classes that want to support large lengths.
Linked PRs
- gh-150581
- 主要言語
- Python
- スター
- 77.2k
- フォーク
- 36k
- 平均マージ
- 1日 9時間
- マージ済み PR(30日)
- 558
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
python/cpython のほかの issue
-
docs pending
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
stdlib type-feature
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
stdlib type-feature
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
-
build type-bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
-
stdlib topic-email type-feature
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
似ている issue
-
🐛 Bug 🔔 Pending processing
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
jumpserver/jumpserver#17584 ·
-
link-check link-check:sphinx-theme
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 90/100
modelscope/DiffSynth-Studio#1702 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
qgis/QGIS-Documentation#11275 ·
-
bug priority:normal ready-for-dev
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
OpenHands/extensions#626 · コメント 1 件 ·