-
Notifications
You must be signed in to change notification settings - Fork 40
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
Added Report submission for Issue 43 #146
Added Report submission for Issue 43 #146
Conversation
@reproducibility-org complete |
We made a modification to the final report submission. Thanks |
Hi, please find below a review submitted by one of the reviewers: Score: 9
Confidence : 5 |
Hi, please find below a review submitted by one of the reviewers: Score: 9 CODE COMMUNICATION WITH THE ORIGINAL AUTHORS HYPERPARAMETER SEARCH DISCUSSION ON RESULTS RECOMMENDATIONS FOR REPRODUCIBILITY I also appreciated that the authors of the report included a computational cost in their analysis, details that were missing in the original ICLR submission. DISCUSSION ON RESULTS As a minor issue, I do not understand why the plots on the first row in Figure 4 have a different style than the ones on the second and third row. I assume it is not relevant, but using the same style for plots would be required in the final submission. OVERALL ORGANIZATION AND CLARITY As minor issues: Confidence : 4 |
Hi, please find below a review submitted by one of the reviewers: Score: 9 Problem statement Code Communication with original authors Hyperparameter search Ablaton study Discussion on results Overall organization and clarity |
Meta Reviewer Decision: Accept |
#43