Skip to content

Improve the algorithm list on README - #509

Merged
muupan merged 10 commits into
masterfrom
improve-algorithm-list
Jul 17, 2019
Merged

Improve the algorithm list on README#509
muupan merged 10 commits into
masterfrom
improve-algorithm-list

Conversation

@muupan

@muupan muupan commented Jul 17, 2019

Copy link
Copy Markdown
Member

This PR improve the algorithm list on README by adding links to papers and examples. The paper list is removed. You can check the new README here: https://github.com/chainer/chainerrl/tree/improve-algorithm-list

I removed PGT because it does not follow any paper and I think it's not worth mentioning in README. I'm not sure if the technique list covers all the techniques worth mentioning, but we can add later.

@prabhatnagarajan
prabhatnagarajan self-requested a review July 17, 2019 09:32
@muupan muupan changed the title [WIP] Improve the algorithm list on README Improve the algorithm list on README Jul 17, 2019

@prabhatnagarajan prabhatnagarajan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I checked the links. They're correct. Only two small comments.

Comment thread README.md Outdated
Comment thread README.md
Co-Authored-By: Prabhat Nagarajan <prabhat.nagarajan@gmail.com>

@prabhatnagarajan prabhatnagarajan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM;

@muupan
muupan merged commit 04fdf6b into master Jul 17, 2019
@muupan
muupan deleted the improve-algorithm-list branch July 17, 2019 13:42
@muupan muupan added this to the v0.8 milestone Feb 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants