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.

Sign up for GitHub

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

Jump to bottom

Make all POMDPTools distributions into Distributions.jl distributions #490

Open
zsunberg opened this issue May 14, 2023 · 1 comment · May be fixed by #495
Open

Make all POMDPTools distributions into Distributions.jl distributions #490

zsunberg opened this issue May 14, 2023 · 1 comment · May be fixed by #495
Assignees
@WhiffleFish

Comments

@zsunberg
Copy link
Member

In #473 we decided it would make sense to make all distributions into Distributions.jl distributions. @WhiffleFish volunteered to do this - just making it an official issue. @WhiffleFish , when do you think you can have it done by?
@zsunberg zsunberg linked a pull request May 23, 2023 that will close this issue
Open
@zsunberg
Copy link
Member Author

zsunberg commented May 23, 2023 •
edited

I started implementing this, but it is quite unsatisfying. Distributions.jl is very focused on numerical distributions, for example, rand actually falls back to quantile in many cases and has a rigid parameter-centric type hierarchy. See also comments in #473

@zsunberg zsunberg mentioned this issue May 23, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@WhiffleFish WhiffleFish

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

WIP: convert distributions to Distributions.jl distributions JuliaPOMDP/POMDPs.jl
2 participants
@zsunberg @WhiffleFish