/core

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added a split for seeds and added it to the exposed API. #15

Closed
wants to merge 1 commit into
base: master
from

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@jcollard @evancz
@jcollard
Contributor

jcollard commented Nov 24, 2014

Exposed split for Seed to allow creating multiple generators. This is useful for randomness on tree / graph structures.
@evancz

This comment has been minimized.

Show comment
Hide comment
@evancz

evancz Nov 24, 2014

Member

I thought the recommendation was to not have this right now?
Member

evancz commented Nov 24, 2014

I thought the recommendation was to not have this right now?
@jcollard

This comment has been minimized.

Show comment
Hide comment
@jcollard

jcollard Nov 24, 2014

Contributor

Yes, I think you're right. I glanced over at the current System.Random and it looks like there have been quite a few internal changes.

At some point, someone will probably want to translate the new library to Elm.
Contributor

jcollard commented Nov 24, 2014

Yes, I think you're right. I glanced over at the current System.Random and it looks like there have been quite a few internal changes.

At some point, someone will probably want to translate the new library to Elm.

@jcollard jcollard closed this Nov 24, 2014

@jcollard jcollard deleted the jcollard:split-random branch Nov 24, 2014

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