Skip to content
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.

Sign up for GitHub

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

Jump to bottom

submission for #67 #160

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

submission for #67 #160

wants to merge 1 commit into from

Conversation

mhtsbt
Copy link

@mhtsbt mhtsbt commented Jan 11, 2019

submission for #67

@reproducibility-org reproducibility-org added the checks-complete Submission criteria checks complete label Jan 11, 2019
@reproducibility-org reproducibility-org added reviewer-assigned Reviewer has been assigned and removed reviewer-assigned Reviewer has been assigned labels Feb 21, 2019
@reproducibility-org
Copy link
Collaborator

Hi, please find below a review submitted by one of the reviewers:

Score: 5
Reviewer 1 comment : Successor Options is a novel strategy to discover options for RL algorithms. The authors have reproduced a subset of the original experiments proposed in the main paper. Since they considered the simplest environment, the scope of reproducibility study is quite limited. Further, the report does not direct compare with the results reported in the paper. The overall writing is easy to follow though there are some grammatical mistake (eg good canidate subgoals)

Confidence : 3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
checks-complete Submission criteria checks complete
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mhtsbt @reproducibility-org