Closed tbreloff closed 7 years ago
Looks good to me. I've been wanting to reproduce the basic examples from Sutton and Barto book as a learning exercise - it would be great if I could use what you're working on!
Edit: After giving a little more thought... I like this API, but can't think of a reason why they would have to live in LearnBase rather than have Reinforce as a standalone package. I guess it doesn't hurt to have it in LearnBase though.
I'm going to hold off on this, and keep everything in Reinforce for now. The interface isn't stable enough and anyone using it would want other things in Reinforce anyways.
I'm starting to work on a framework for reinforcement learning in https://github.com/tbreloff/Reinforce.jl, and I was thinking it would be nice to include the core abstractions in LearnBase so that I could have standalone environments or agents without requiring a dependency on Reinforce.jl.
I propose adding this to LearnBase:
Any thoughts/objections?