Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

small deterministic example update #1240

Merged
merged 3 commits into from
Aug 4, 2022
Merged

small deterministic example update #1240

merged 3 commits into from
Aug 4, 2022

Conversation

TheIronBorn
Copy link
Contributor

No description provided.

@dhardy
Copy link
Member

dhardy commented Jul 8, 2022

.sum()? Nice.

Can you add an assert on the result while you're modifying this? It is deterministic.

@TheIronBorn
Copy link
Contributor Author

TheIronBorn commented Jul 8, 2022 •
edited
Loading

Added. Confirmed other seeds and other batch/batch-size combos do change it

@TheIronBorn TheIronBorn merged commit 1d55a97 into rust-random:master Aug 4, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@dhardy dhardy

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@TheIronBorn @dhardy