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

Add epsilon greedy policy for DQN #1012

Merged
merged 2 commits into from May 30, 2017

Conversation

Projects
None yet
4 participants
@ShangtongZhang
Member

ShangtongZhang commented May 28, 2017

Add epsilon greedy policy and a simple test case.

@mlpack-jenkins

This comment has been minimized.

Show comment
Hide comment
@mlpack-jenkins

mlpack-jenkins May 28, 2017

Can one of the admins verify this patch?

mlpack-jenkins commented May 28, 2017

Can one of the admins verify this patch?

@zoq zoq merged commit a76fe5e into mlpack:master May 30, 2017

3 checks passed

Style Checks Build finished.
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@zoq

This comment has been minimized.

Show comment
Hide comment
@zoq

zoq May 30, 2017

Member

Thanks, looking forward to review the next PR.

Member

zoq commented May 30, 2017

Thanks, looking forward to review the next PR.

@rcurtin

This comment has been minimized.

Show comment
Hide comment
@rcurtin

rcurtin May 30, 2017

Member

I made some really minor style changes in 48676e8. In future PRs, the style checker should give useful results now that we have merged #1008. :)

Member

rcurtin commented May 30, 2017

I made some really minor style changes in 48676e8. In future PRs, the style checker should give useful results now that we have merged #1008. :)

@ShangtongZhang ShangtongZhang deleted the ShangtongZhang:rl-policy branch May 30, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment