-
Notifications
You must be signed in to change notification settings - Fork 3
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
API for objects that support rand? #16
Comments
Haven't thought much about it yet, but my initial feeling is that this is a bit orthogonal to the density aspect. Maybe it would be better to add traits (if needed/useful) to Random or RandomExtensions? |
I agree. Adding it to Random would take time though, and we'd have to have the API "perfect" from the get-go because it would be frozen immediately. Maybe a separate package? |
IIRC RandomExtensions was supposed to be some experimental "playground" for Random? |
Oh, completely overlooked RandomExtensions, lol. Maybe that ties into JuliaStats/Distributions.jl#1139 as well? |
In relation to the discussion on DiffEqBase and QuasiMonteCarlo in JuliaStats/Distributions.jl#1139 - should have a trait based API for (density-like) objects that support IID sampling, i.e. that support
rand
(and possiblyrand!
as well)? This could be part of DensityInterface or it could go into a separate package, but it might be nice to have a community-standard way to ask something "do you support rand?", or maybe even "what do you produce on rand?".CC @ChrisRackauckas , @devmotion
The text was updated successfully, but these errors were encountered: