Skip to content

Conversation

dg-pb
Copy link
Contributor

@dg-pb dg-pb commented Apr 9, 2025

PYEXE1='main/python.exe'
PYEXE2='new/python.exe'

$PYEXE1 -m timeit -s 'from random import randint' 'randint(0, 2 - 1)'    # 298 nsec
$PYEXE2 -m timeit -s 'from random import randint' 'randint(0, 2 - 1)'    # 218 nsec

$PYEXE1 -m timeit -s 'from random import randint' 'randint(0, 2**32 - 1)'    # 519 nsec
$PYEXE2 -m timeit -s 'from random import randint' 'randint(0, 2**32 - 1)'    # 357 nsec

$PYEXE1 -m timeit -s 'from random import randint' 'randint(0, 2**128 - 1)'    # 819 nsec
$PYEXE2 -m timeit -s 'from random import randint' 'randint(0, 2**128 - 1)'    # 670 nsec

@eendebakpt
Copy link
Contributor

A similar change was rejected before: see #117388.

@dg-pb
Copy link
Contributor Author

dg-pb commented Apr 9, 2025

A similar change was rejected before: see #117388.

Thanks, my bad.

@dg-pb dg-pb closed this Apr 9, 2025
@dg-pb dg-pb deleted the randbelow_fix branch April 9, 2025 15:13
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.

2 participants