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

Drop (or fix) POMDPGym.jl dependency #19

Open
zengmao opened this issue Feb 15, 2025 · 3 comments
Open

Drop (or fix) POMDPGym.jl dependency #19

zengmao opened this issue Feb 15, 2025 · 3 comments

Comments

@zengmao
Copy link

zengmao commented Feb 15, 2025

I've made a fork of this repo with the POMDPGym.jl dependency stripped off, so that the rest of the package can be installed without error in the latest Julia 1.11:
https://github.com/zengmao/Crux.jl

POMDPGym.jl is apparently broken, or at least hard to install, in the latest Julia versions due to Python package management issues. If this can be fixed, or if an alternative interface with OpenAI Gym / Gymnasium can be found, it would be the best scenario. An alternative approach, as in my experiment with the forked repo, is to trim down the package to a pure Julia core.

@ancorso
Copy link
Collaborator

ancorso commented Feb 16, 2025

I'd be fine with this! @mossr thoughts? If @mossr is ok with this then go ahead and open a PR if you want.

@mossr
Copy link
Member

mossr commented Feb 16, 2025

Yeah I agree that we can lighten the POMDPGym dependency. I'm all for this.

@zengmao
Copy link
Author

zengmao commented Mar 14, 2025

Looks like @mossr has already made commits to do this. Thanks! Since POMDPGym.jl has also been fixed, in principle it can be treated as Test-specific dependencies supported by Pkg.jl.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants