random.seeding is broken if urandom is absent #332

Merged
merged 1 commit into from Apr 26, 2012

Conversation

Projects
None yet
3 participants
@vibrant
Contributor

vibrant commented Apr 12, 2012

calling random.seed(random.random()) in every child process will leave the random number generator at identical state at each process (because random.random() will be the same), so I changed it to using the current time and process ID as seed which should fix things up.

@davisp

This comment has been minimized.

Show comment Hide comment
@davisp

davisp Apr 12, 2012

LGTM

davisp commented on 62ee085 Apr 12, 2012

LGTM

benoitc added a commit that referenced this pull request Apr 26, 2012

Merge pull request #332 from vibrant/master
random.seeding is broken if urandom is absent

@benoitc benoitc merged commit 8c716dd into benoitc:master Apr 26, 2012

@benoitc

This comment has been minimized.

Show comment Hide comment
@benoitc

benoitc Apr 26, 2012

Owner

@vibrant can you give me your informations to add you to the THANKS file?

Owner

benoitc commented Apr 26, 2012

@vibrant can you give me your informations to add you to the THANKS file?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment