python / python/cpython

Clarify what to override when subclassing random.Random

Đang mở
#131,244 5 bình luận 0 reaction 1 người được giao Xem trên GitHub

@tim-one đang làm issue này rồi.

Từ ngày 18/3/2025.

stdlib type-bug
Ngôn ngữ chính
Python
Star
77.2k
Fork
36k
Chỉ số merge pull request
Chỉ số pull request đang chờ

Mô tả

Documentation

In current documentation about the random.Random class, it said that overriding getrandbits() is optional, so it may make people think just overriding random() is sufficient to control the behavior of the class. But in practice methods like randbytes use getrandbits, which is implemented in C to get randomness from MT19937. So it has the following behavior:

import random


class CustomRand(random.Random):
    def random(self):
        print("CustomRand.random()")  # does not get called
        return 0.5


myrand = CustomRand()
myrand.seed(123)  # just for demonstrating that randbytes still use MT19937
r1 = myrand.randbytes(4)

random.seed(123)
r2 = random.randbytes(4)

print(r1 == r2)  # True

Which shows that overriding random() only is not sufficient. From the issue #84466, this seems to be the expected behavior (or not?). So I think this should be documented that overriding getrandbits() is not optional.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.