-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
Maintainers #44
Comments
I am having write access as of now. But I am not having option to give you write access. Maybe @DhairyaLGandhi can do something about it? |
@darsnack Even I share the same interest :). If you have some experience maybe you could tell me something about how and where to extend this project?(add more envs or something like that). I am relatively new to julia |
I would suggest getting in touch with @findmyway. I saw you posted in the FastAI port channel on Slack, and this issue was part of ML community effort along the RL thrust. Since then, we haven't focused on RL as much, but that is mostly a bandwidth issue. I think Jun could probably give you the best advice (you can find him on Slack or Zulip) on what's the next step. Based on previous discussions, I think the best next step is to port missing functionality from this repo to JuliaReinforcementLearning packages and not add more environments here. |
@RajGhugare19 has already finished several meaningful PRs in JuliaReinforcementLearning/GridWorlds.jl#49 and JuliaReinforcementLearning/GridWorlds.jl#46 . 👍 If you are interested in adding more environments, you and take a look at the unmerged PR #37 and maybe port some of them into https://github.com/JuliaReinforcementLearning/ReinforcementLearningEnvironments.jl ? |
Who is currently maintaining this package? I have an interest in seeing it through, and I have been helping/following JuliaReinforcementLearning/CommonRLInterface.jl. If no one is actively maintaining it, can I have write access?
I'll be doing my development in an alternate branch and PR everything for the community to review. That way any stability isn't broken (even though this package is not registered or marked as stable).
The text was updated successfully, but these errors were encountered: