Skip to content

Conversation

@jorenham
Copy link
Member

@jorenham jorenham commented Nov 6, 2025

This fixes inspect.signature for

  • np.random.BitGenerator
  • np.random.Generator
  • np.random.MT19937
  • np.random.PCG64
  • np.random.PCG64DXSM
  • np.random.Philox
  • np.random.RandomState
  • np.random.SFC64
  • np.random.SeedSequence
  • np.random.bit_generator.SeedlessSeedSequence

This also fixes a typo in bit_generator.pxd that accidentally defined an empty unused class np.random.bit_generator.SeedlessSequence.

Related to #30104, #30114, #30121, #30124, #30126, #30137, #30138, #30140, #30143, #30146, #30147, and #30155

cdef get_assembled_entropy(self)

cdef class SeedlessSequence():
cdef class SeedlessSeedSequence:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, probably correct as SeedlessSequence doesn't appear anywhere else. Let's ping @rkern just to be sure.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suspect we can remove this alltogether, or maybe fix up some files that import bit_generator.pxd

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change is correct. SeedlessSeedSequence is the correct spelling. SeedlessSequence never existed, but SeedlessSeedSequence certainly does.

@charris charris merged commit 2b216b1 into numpy:main Nov 6, 2025
78 checks passed
@charris
Copy link
Member

charris commented Nov 6, 2025

Thanks Joren.

@jorenham jorenham deleted the fix-random-class-signatures branch November 6, 2025 19:51
cakedev0 pushed a commit to cakedev0/numpy that referenced this pull request Dec 5, 2025
This fixes `inspect.signature` for

- `np.random.BitGenerator`
- `np.random.Generator`
- `np.random.MT19937`
- `np.random.PCG64`
- `np.random.PCG64DXSM`
- `np.random.Philox`
- `np.random.RandomState`
- `np.random.SFC64`
- `np.random.SeedSequence`
- `np.random.bit_generator.SeedlessSeedSequence`

This also fixes a typo in `bit_generator.pxd` that accidentally defined an empty unused class `np.random.bit_generator.SeedlessSequence`.

Related to numpy#30104, numpy#30114, numpy#30121, numpy#30124, numpy#30126, numpy#30137, numpy#30138, numpy#30140, numpy#30143, numpy#30146, numpy#30147, and numpy#30155
IndifferentArea pushed a commit to IndifferentArea/numpy that referenced this pull request Dec 7, 2025
This fixes `inspect.signature` for

- `np.random.BitGenerator`
- `np.random.Generator`
- `np.random.MT19937`
- `np.random.PCG64`
- `np.random.PCG64DXSM`
- `np.random.Philox`
- `np.random.RandomState`
- `np.random.SFC64`
- `np.random.SeedSequence`
- `np.random.bit_generator.SeedlessSeedSequence`

This also fixes a typo in `bit_generator.pxd` that accidentally defined an empty unused class `np.random.bit_generator.SeedlessSequence`.

Related to numpy#30104, numpy#30114, numpy#30121, numpy#30124, numpy#30126, numpy#30137, numpy#30138, numpy#30140, numpy#30143, numpy#30146, numpy#30147, and numpy#30155
jorenham added a commit to jorenham/numpy that referenced this pull request Dec 22, 2025
seberg pushed a commit that referenced this pull request Dec 22, 2025
charris pushed a commit to charris/numpy that referenced this pull request Dec 22, 2025
charris added a commit that referenced this pull request Dec 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants