Skip to content

R2d1#139

Closed
kargarisaac wants to merge 30 commits intoastooke:masterfrom
deepdrive:r2d1
Closed

R2d1#139
kargarisaac wants to merge 30 commits intoastooke:masterfrom
deepdrive:r2d1

Conversation

@kargarisaac
Copy link

Hi,
I added some examples that may be helpful for those who want to start with rlpyt.

  • example for dqn algorithm in environments with non-frame state spaces such as CartPole-v0.
  • loading and evaluation of the trained model
  • Tested some algorithms like DDPG, SAC, DQN, R2D1 in the DeepDrive-Zero environment.
    It would be great if you can take a look and comment.

@astooke
Copy link
Owner

astooke commented Apr 7, 2020

Hi, thanks for all this!

I like several of the pieces such as an example DQN without frame-based buffer (although this can already be done by passing in the replay buffer class) and possibly evaluating a previously trained agent. Overall this PR has too many pieces in it though and would need to be broken down. It also has several customizations which wouldn't be considered for the main repo. If you break it down into bite-sized pieces I can take a look again.

@kargarisaac
Copy link
Author

You mean I need to remove codes for deepdrive-zero environment or keep them?
I'll create a new branch for dqn codes and make a new pull request then.

@kargarisaac kargarisaac closed this Apr 7, 2020
@kargarisaac
Copy link
Author

I created a new pull request #142

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

Successfully merging this pull request may close these issues.

3 participants