Closed whedon closed 5 years ago
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @henrik227, @SirSharpest it looks like you're currently assigned to review this paper :tada:.
:star: Important :star:
If you haven't already, you should seriously consider unsubscribing from GitHub notifications for this (https://github.com/openjournals/joss-reviews) repository. As a reviewer, you're probably currently watching this repository which means for GitHub's default behaviour you will receive notifications (emails) for all reviews 😿
To fix this do the following two things:
For a list of things I can do to help you, just type:
@whedon commands
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
@whedon generate pdf
Attempting PDF compilation. Reticulating splines etc...
👋Thank you so much @henrik227 and @SirSharpest for agreeing to review this submission! The review checklist is above. Please let me know if there is anything else you need!
Hi, I have rewritten the test script test.py
to better meet criteria described herein.
@whedon generate pdf
Attempting PDF compilation. Reticulating splines etc...
@sohrabtowfighi Regarding the authorship: Are you also using the @pySRURGS account, or is this someone else?
@henrik227, I am also using the @pySRURGS account. There is no other author.
I added API documentation to https://pysrurgs.github.io/pySRURGS/ which is linked from the main pySRURGS github page under the API section.
I have integrated the coveralls package for test-suite coverage reporting - currently at 89%.
I have added an -exhaustive flag which performs brute-force exhaustive search instead of uniform random global search
@whedon generate pdf
Attempting PDF compilation. Reticulating splines etc...
I have added the -deterministic
flag and made other changes to address @SirSharpest' comments.
After acknowledging Dr. Luther Tychonievich, I added a wayback machine archive link to the blog post where Dr. Tychonievich details the algorithm mapping integers to binary trees.
@SirSharpest are you still working on your review? Please let me know!
Yes - sorry for delay. I am happy with the changes made.
@SirSharpest do the changes meet the checkbox criteria for the review?
Yes, all updated now
Ok great thanks. @mbobra please let me know how to proceed when you get the chance.
@whedon generate pdf
Attempting PDF compilation. Reticulating splines etc...
I removed the word 'actually' from the paper.
@sohrabtowfighi We're almost there! Could you please go ahead and create a new release that incorporates all of these fixes? And can you please archive your release on Zenodo to obtain a DOI and then put the Zenodo badge in your README.md file? After that I think we're done 🎉
Thank you for your reviews, @henrik227 and @SirSharpest! We really appreciate it 👏
Done. Thanks! https://zenodo.org/badge/latestdoi/200314164
@whedon set 10.5281/zenodo.3442594 as archive
OK. 10.5281/zenodo.3442594 is the archive.
@whedon set 0.1 as version
OK. 0.1 is the version.
@whedon check references
Attempting to check references...
OK DOIs
- 10.1186/s13040-018-0164-x is OK
- 10.1145/2464576.2482691 is OK
MISSING DOIs
- https://doi.org/10.1016/0303-2647(94)90062-0 may be missing for title: Genetic Programming: On the Programming of Computers by Means of Natural Selection
- https://doi.org/10.1016/0012-365x(71)90023-9 may be missing for title: Enumerating Trees
- https://doi.org/10.1287/moor.6.1.19 may be missing for title: Minimization by Random Search Techniques
INVALID DOIs
- None
@pySRURGS Can you please make sure all the references in the paper.bib file include DOIs? Thanks!
@whedon generate pdf
Attempting PDF compilation. Reticulating splines etc...
@mbobra I have fixed references. The work of Tychonievich has no DOI.
Wait found issue and need to regenerate pdf.
@whedon generate pdf
Attempting PDF compilation. Reticulating splines etc...
@mbobra ok done
@whedon check references
Attempting to check references...
OK DOIs
- 10.1007/BF00175355 is OK
- 10.1186/s13040-018-0164-x is OK
- 10.1145/2464576.2482691 is OK
- 10.1287/moor.6.1.19 is OK
MISSING DOIs
- https://doi.org/10.1016/0012-365x(71)90023-9 may be missing for title: Enumerating Trees
INVALID DOIs
- None
Submitting author: @pySRURGS (Sohrab Towfighi) Repository: https://github.com/pySRURGS/pySRURGS Version: 0.1 Editor: @mbobra Reviewer: @henrik227, @SirSharpest Archive: 10.5281/zenodo.3442594
Status
Status badge code:
Reviewers and authors:
Please avoid lengthy details of difficulties in the review thread. Instead, please create a new issue in the target repository and link to those issues (especially acceptance-blockers) by leaving comments in the review thread below. (For completists: if the target issue tracker is also on GitHub, linking the review thread in the issue or vice versa will create corresponding breadcrumb trails in the link target.)
Reviewer instructions & questions
@henrik227 & @SirSharpest, please carry out your review in this issue by updating the checklist below. If you cannot edit the checklist please:
The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @mbobra know.
✨ Please try and complete your review in the next two weeks ✨
Review checklist for @henrik227
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?Review checklist for @SirSharpest
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?