search

chandar-lab / RLHive

MIT License
100 stars 9 forks source link

Stateless agent #316

Closed dapatil211 closed 1 year ago

dapatil211 commented 1 year ago

Note, docstrings still need to be updated. Please review other things in the meantime.

mrsamsami commented 1 year ago

Except for the naming, which Hadi also mentioned, everything looks good. I'd recommend changing the state variable name to memory because it basically stores information about the past and present (not only for DRQN).