Closed whedon closed 6 years ago
Hello human, I'm @whedon. I'm here to help you with some common editorial tasks. @puncan it looks like you're currently assigned as the reviewer for 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
Attempting PDF compilation. Reticulating splines etc...
@whedon assign @Tabeel as reviewer
OK, the reviewer is @Tabeel
@whedon add @puncan as reviewer
OK, @puncan is now a reviewer
Ok, both @puncan and @Tabeel will be reviewers for this submission. Each of you has a review checklist above. Please use the one with your name on it.
Hi @katyhuff, I am unable to make changes to the review checklist. Am I am missing something?
@Tabeel there's a little note about this up at the top of the issue, but it's easy to miss.
It says:
If you cannot edit the checklist please:
Thank you
@puncan - it looks like you're only part way through your review here. Any chance you could complete this in the next week or so?
Yes, my apologies. It got away from me. Certainly within a week.
-Paul
On Fri, Jun 15, 2018 at 1:02 PM Arfon Smith notifications@github.com wrote:
@puncan https://github.com/puncan - it looks like you're only part way through your review here. Any chance you could complete this in the next week or so?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/openjournals/joss-reviews/issues/710#issuecomment-397727647, or mute the thread https://github.com/notifications/unsubscribe-auth/AgSGWflCfQTf2ml09aHLVdk6IpmSBo2Nks5t9BLagaJpZM4TuU9R .
There were a couple I checked off after creating new issues. Am I doing this right?
Thanks @puncan I'll fix up the issues you've raised now.
Also, there is a newer version of MCycle that uses Cython; it has a different build and different directory structure. At the moment it's just in the cython fork, I haven't pulled it to master yet. Could I update my joss submission to consider this version?
I don’t see why not.
Sent from my iPhone
On Jun 21, 2018, at 7:20 PM, Momar Hughes notifications@github.com wrote:
Also, there is a newer version of MCycle that uses Cython; it has a different build and different directory structure. At the moment it's just in the cython fork, I haven't pulled it to master yet. Could I update my joss submission to consider this version?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.
Yes, go right ahead and update to the new version. Sorry for the delay in responding. It looks like all of the issues raised by the reviewers have now been handled. Is that correct @puncan @Tabeel @momargoh ?
If so, then @momargoh at this point could you make an archive of the reviewed software in Zenodo/figshare/other service and update this thread with the DOI of the archive? I can then move forward with accepting the submission.
Sounds good to me.
On Mon, Jul 23, 2018 at 6:22 AM Katy Huff notifications@github.com wrote:
Yes, go right ahead and update to the new version. Sorry for the delay in responding. It looks like all of the issues raised by the reviewers have now been handled. Is that correct @puncan https://github.com/puncan @Tabeel https://github.com/Tabeel @momargoh https://github.com/momargoh ?
If so, then @momargoh https://github.com/momargoh at this point could you make an archive of the reviewed software in Zenodo/figshare/other service and update this thread with the DOI of the archive? I can then move forward with accepting the submission.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/openjournals/joss-reviews/issues/710#issuecomment-407055895, or mute the thread https://github.com/notifications/unsubscribe-auth/AgSGWeaUTRLBzznfPMjqGC8DolRscKqiks5uJc4QgaJpZM4TuU9R .
No, I haven't handled all the issues raised by @puncan yet, doing so now. I'm having trouble committing the new Cython-based version to pip due to problems creating a cross-platform dist wheel. The package can be built from the source using python3 setup.py install
and similarly the docs will not be hosted properly until the package is on pip, so at the moment they can just be built from the source using Sphinx3 make html
.
Is this OK: the package needing to be built from source instead of available from pip?
OK, the docs are building OK https://mcycle.readthedocs.io/index.html. For now the source has to be installed from source as opposed to from pip. I have updated my JOSS submission to v1.0.0: the latest version of the source code.
Zenodo is ready to go. I will wait for @puncan 's approval, regarding the issues he raised, before creating the DOI
It looks to me like @puncan approved this a few comments ago. (https://github.com/openjournals/joss-reviews/issues/710#issuecomment-407145425)
It seems you can go ahead and make the DOI whenever you're ready.
doi:10.5281/zenodo.1324221 https://zenodo.org/record/1324222#.W2BdfxiuY5k
Excellent!
@whedon set 10.5281/zenodo.1324221 as archive
OK. 10.5281/zenodo.1324221 is the archive.
@arfon This submission is ready for acceptance! Congratulations @momargoh , and thank you for your reviews @Tabeel and @puncan !
@Tabeel, @puncan - many thanks for your reviews here and to @katyhuff for editing this submission ✨
@momargoh - your paper is now accepted and your DOI is https://doi.org/10.21105/joss.00710 :zap: :rocket: :boom:
:tada::tada::tada: Congratulations on your paper acceptance! :tada::tada::tada:
If you would like to include a link to your paper from your README use the following code snippets:
Markdown:
[![DOI](http://joss.theoj.org/papers/10.21105/joss.00710/status.svg)](https://doi.org/10.21105/joss.00710)
HTML:
<a style="border-width:0" href="https://doi.org/10.21105/joss.00710">
<img src="http://joss.theoj.org/papers/10.21105/joss.00710/status.svg" alt="DOI badge" >
</a>
This is how it will look in your documentation:
We need your help!
Journal of Open Source Software is a community-run journal and relies upon volunteer effort. If you'd like to support us please consider doing either one (or both) of the the following:
Submitting author: @momargoh (Momar Hughes) Repository: https://github.com/momargoh/MCycle Version: v0.1.2 Editor: @katyhuff Reviewers: @Tabeel, @puncan Archive: 10.5281/zenodo.1324221
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) 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
@puncan, 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.theoj.org/about#reviewer_guidelines. Any questions/concerns please let @katyhuff know.
Review checklist for @puncan
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 @Tabeel
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?