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

[new feature] Random.eachrand #35290

Closed
wants to merge 7 commits into from
Closed

[new feature] Random.eachrand #35290

wants to merge 7 commits into from

Conversation

o314
Copy link
Contributor

@o314 o314 commented Mar 28, 2020

Propose a new Random.eachrand function to bring Iterators to Random in the same vein of eachrow, eachcol.

It currently allow to stream random scalar type, range type.
Stream of random array is not proposed at this stage.

@rfourquet rfourquet added feature Indicates new feature / enhancement requests randomness Random number generation and the Random stdlib labels Mar 28, 2020
@rfourquet
Copy link
Member

rfourquet commented Mar 28, 2020

Note that this idea was slightly discussed in #24912 (named Rand there), which is now part of the RandomExtension package. I would suggest that people interested in this feature try out this package, whose initial raison-d'être was to provide a proof-of-concept for the features proposed in that PR.

@ViralBShah
Copy link
Member

Should we close this in favour of trying these things out in RandomExtension?

@o314
Copy link
Contributor Author

o314 commented May 14, 2020

That's ok for me.

@ViralBShah
Copy link
Member

Thank you!

@ViralBShah ViralBShah closed this May 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Indicates new feature / enhancement requests randomness Random number generation and the Random stdlib
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants