genrl
genrl copied to clipboard
Mujoco, PyBullet support
We should develop an environment module with wrappers. For a starter, I find TF Agents env module pretty good.
I'll work on Atari wrappers.
Ok sure. I'm making a list of some that we should support at the very least, and some that we can think of adding.
- [x] Atari (@Zeus3101 )
- [ ] PyBullet
- [ ] Mujoco
- [x] Generic monitor, etc wrappers around gym (@Zeus3101 )
I'll do generic wrappers too if that's fine
Sure. Just ensure they are compatible with vecenvs which we should also move under this module.
On Wed, 20 May 2020, 04:02 Sampreet, [email protected] wrote:
I'll do generic wrappers too if that's fine
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/SforAiDl/genrl/issues/125#issuecomment-631120450, or unsubscribe https://github.com/notifications/unsubscribe-auth/AH72FJ6IVWR5HFX5PRFE3ZLRSMCGDANCNFSM4NE2KDLQ .
Hmm... Okay. Should I also accommodate for classical models or only the deep models? Depending on that, I'll add a folder called wrappers (or) environments under genrl or genrl/deep.
If you can, thats definitely better
Also, I think it would be a good idea to move VecEnv.py into this folder too. It makes more sense keeping it environments rather than in deep/common
Yeah that makes sense.
On Wed, 20 May 2020, 21:15 Sampreet, [email protected] wrote:
Also, I think it would be a good idea to move VecEnv.py into this folder too. It makes more sense keeping it environments rather than in deep/common
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/SforAiDl/genrl/issues/125#issuecomment-631558568, or unsubscribe https://github.com/notifications/unsubscribe-auth/AH72FJ224FMLW523YDF67WDRSP3KJANCNFSM4NE2KDLQ .
Do we want pybullet and mujoco by v0.1?
I think we should probably focus on getting what we have better structured first (internally) but let anyone else (not a core contributor) work on this.
Stale issue message