Closed editorialbot closed 4 months ago
@wathom Thanks for the updated manuscript. @danielskatz All checkboxes ticked now.
👋 @wscullin - would you be willing to review this submission for JOSS? We carry out our checklist-driven reviews here in GitHub issues and follow these guidelines: https://joss.readthedocs.io/en/latest/review_criteria.html (Two of my three reviewers seem to have vanished, and I need at least two reviews to move this forward.)
Thanks @wscullin - I didn't get a notification for your reaction, so missed this until now, but will assume your 👍 means you are willing, so I will add you now
@editorialbot add @wscullin as reviewer
@wscullin added to the reviewers list!
👋 @wscullin, your review will be checklist based. You will have a separate checklist that you should update when carrying out your review. First of all you need to run this command in a separate comment to create the checklist:
@editorialbot generate my checklist
As you go over the submission, please check any items that you feel have been satisfied. The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html.
The JOSS review is different from most other journals. Our goal is to work with the authors to help them meet our criteria instead of merely passing judgment on the submission. As such, reviewers are encouraged to submit issues and pull requests on the software repository. When doing so, please mention openjournals/joss-reviews#6017
so that a link is created to this thread (and I can keep an eye on what is happening). Please also feel free to comment and ask questions on this thread. In my experience, it is better to post comments/questions/suggestions as you come across them instead of waiting until you've reviewed the entire package.
We aim for reviews to be completed within about 2-4 weeks. Please let me know if either of you require some more time. We can also use editorialbot (our bot) to set automatic reminders if you know you'll be away for a known period of time.Any questions/concerns please let @danielskatz know.
👋 @wscullin - if you can go ahead and generate your checklist as above, and then at least check off the COI and CoC items, I would appreciate it, in part to make sure all the permissions are working ok.
👋 @wscullin - if you can go ahead and generate your checklist as above, and then at least check off the COI and CoC items, I would appreciate it, in part to make sure all the permissions are working ok.
👋 @wscullin - if you can go ahead and generate your checklist as above, and then at least check off the COI and CoC items, I would appreciate it, in part to make sure all the permissions are working ok.
👋 @wscullin - one last ping before I look for another reviewer - I do still hope you have time to do this though
We have another reviewer who has agreed to review this, so I'm going to add him, and remove the reviewers who haven't appeared...
@editorialbot add @phargogh as reviewer
@phargogh added to the reviewers list!
@editorialbot remove @wscullin as reviewer
@wscullin removed from the reviewers list!
@editorialbot remove @apjez as reviewer
@apjez removed from the reviewers list!
@editorialbot remove verolero86 as reviewer
I can't add that reviewer: verolero86 is not a username
👋 @phargogh, thanks for agreeing to review this.
Your review will be checklist based. You will have a separate checklist that you should update when carrying out your review. First of all you need to run this command in a separate comment to create the checklist:
@editorialbot generate my checklist
As you go over the submission, please check any items that you feel have been satisfied. The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html.
The JOSS review is different from most other journals. Our goal is to work with the authors to help them meet our criteria instead of merely passing judgment on the submission. As such, reviewers are encouraged to submit issues and pull requests on the software repository. When doing so, please mention openjournals/joss-reviews#6017
so that a link is created to this thread (and I can keep an eye on what is happening). Please also feel free to comment and ask questions on this thread. In my experience, it is better to post comments/questions/suggestions as you come across them instead of waiting until you've reviewed the entire package.
We aim for reviews to be completed within about 2-4 weeks. Please let me know if either of you require some more time. We can also use editorialbot (our bot) to set automatic reminders if you know you'll be away for a known period of time.Any questions/concerns please let @danielskatz know.
@editorialbot remove @verolero86 as reviewer
@verolero86 removed from the reviewers list!
Thanks for getting started so quickly @phargogh
👋 @phargogh - how is your review coming along? Is there anything blocking you from making more progress?
Thanks for the ping @danielskatz ! Still in progress and I believe I'll have more to report back on in the next day or two.
@phargogh - just checking on this again...
Thanks @danielskatz , and I'm sorry all for the delay.
@danielskatz I see that other JOSS papers have a specific References section of the paper in addition to Acknowledgements. This paper has an explicit Acknowledgements section that includes the references. Is this a normal format for listing references for JOSS?
@wathom I noticed that your CONTRIBUTIONS.md
file makes reference to GitLab quite a number of times, yet the code for this repo is hosted here on GitHub. There are a few other minor differences of language, like that GitLab uses Merge Requests instead of Pull Requests. To avoid any confusion here, perhaps it'd be worth updating the language to be consistent with GitHub?
Thanks @phargogh for catching this. The paper needs to have a References header added at the end, so that the JOSS processsing that adds the actual references at the end of paper will fill them in under this header, not the Acknowledgements one.
@wathom - please see the example paper and change your paper to fix this.
Thanks @danielskatz , and I'm sorry all for the delay.
@danielskatz I see that other JOSS papers have a specific References section of the paper in addition to Acknowledgements. This paper has an explicit Acknowledgements section that includes the references. Is this a normal format for listing references for JOSS?
Many thanks @phargogh for reviewing the paper. Dear @danielskatz, I changed the CONTRIBUTIONS.md and the paper.md files accordingly.
@editorialbot generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@wathom thanks for the changes. The proof looks complete now, with the correct references header, and I see the updates in the contribution guidelines doc now refer to github instead of gitlab. I think it's fine to leave references to "merge requests", since folks familiar with either platform should understand well enough.
@danielskatz all boxes are now checked. Is there anything else I need to do?
Thanks @phargogh - no, nothing more is needed from you
@wathom - at this point, I'll proofread the paper and let you know the next steps. This might take me a little bit due to some other activities.
@wathom - I'm suggesting some small changes to the paper in https://github.com/Bioinformatics-Munich/scas_dashboard/pull/3 Please merge this or let me know what you disagree with, then we can finish the process.
Dear @danielskatz, many thanks for proofreading the paper, I merged the changes.
@editorialbot generate pdf
I'm sorry human, I don't understand that. You can see what commands I support by typing:
@editorialbot commands
@editorialbot generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
👋 @wathom - At this point could you:
I can then move forward with accepting the submission.
Dear @danielskatz,
@editorialbot set v1.0.0 as version
Done! version is now v1.0.0
@editorialbot set 10.5281/zenodo.10064783 as archive
Done! archive is now 10.5281/zenodo.10064783
@editorialbot recommend-accept
Attempting dry run of processing paper acceptance...
Submitting author: !--author-handle-->@wathom<!--end-author-handle-- (Thomas W.) Repository: https://github.com/Bioinformatics-Munich/scas_dashboard Branch with paper.md (empty if default branch): Version: v1.0.0 Editor: !--editor-->@danielskatz<!--end-editor-- Reviewers: @aturner-epcc, @phargogh Archive: 10.5281/zenodo.10064783
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
@aturner-epcc & @apjez & @verolero86, your review will be checklist based. Each of you will have a separate checklist that you should update when carrying out your review. First of all you need to run this command in a separate comment to create the checklist:
The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @danielskatz know.
✨ Please start on your review when you are able, and be sure to complete your review in the next six weeks, at the very latest ✨
Checklists
📝 Checklist for @aturner-epcc