Skip to content
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

Update System.Random documentation regarding seeds #3074

Closed
mlodato517 opened this issue Aug 24, 2019 · 1 comment · Fixed by #3084
Closed

Update System.Random documentation regarding seeds #3074

mlodato517 opened this issue Aug 24, 2019 · 1 comment · Fixed by #3084
Assignees
Labels
cross-platform Indicates issues or PRs that relate to content applicable to both Framework and Core.
Milestone

Comments

@mlodato517
Copy link

The current documentation for System.Random says:

Initializing two random number generators in a tight loop or in rapid succession creates two random number generators that can produce identical sequences of random numbers.

This is true in .NET Framework but doesn't seem to be true in .NET Core.

Can we update the documentation to highlight this difference? It doesn't matter too much, of course, because the general advice is to not generate lots of System.Randoms, but instead use a single one but 🤷‍♂.

@dotnet-bot dotnet-bot added the untriaged New issue has not been triaged by the area owner label Aug 24, 2019
@rpetrusha rpetrusha self-assigned this Aug 26, 2019
@rpetrusha rpetrusha added 📁 Repo - dotnet-api-docs and removed untriaged New issue has not been triaged by the area owner labels Aug 26, 2019
@rpetrusha rpetrusha added this to the August 2019 milestone Aug 26, 2019
@rpetrusha
Copy link

Thanks for pointing this out, @mlodato517. We've opened #3084 to point out that this limitation applies to .NET Framework only.

@mairaw mairaw added the cross-platform Indicates issues or PRs that relate to content applicable to both Framework and Core. label Aug 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-platform Indicates issues or PRs that relate to content applicable to both Framework and Core.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants