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
[REVIEW]: Syn-CpG-Spacer: A Panel web app for synonymous recoding of viral genomes with CpG dinucleotides #6332
Comments
Hello humans, I'm @editorialbot, a robot that can help you with some common editorial tasks. For a list of things I can do to help you, just type:
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
|
|
Wordcount for |
|
Review checklist for @babinyuriiConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Hi, sorry to be slow, I should be able to look at this on Friday. |
No problem, @KatyBrown . Please let us know if you encounter any difficulties as you work through the review. |
@babinyurii - thank you for the thorough review. It looks like you have left unchecked a couple of the review checklist items. Are there associated tasks or edits needed from teh authors before you can check these last items? Or did you just now have time yet to assess them? |
Hi, @fboehm ! Should I address other issues here which are not covered by these checkboxes? I've got some. |
Review checklist for @KatyBrownConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Thank you @babinyurii - yes, please do offer suggestions that may improve the product. Please feel free to open new issues on the software repo's github page to describe the issues and offer suggestions. |
I agree with this and similarly comments would be useful in the JS wrapper in index.js |
I'm not sure if this is outside of my scope as a reviewer, but I think it would be preferable to read the code in index.py - either directly or into a string - and then process this within index.js, if that's possible, rather than pasting it across. It seems risky to have to remember to paste the code and it brings the Python code which is actually running outside of the CI testing. |
Hi @KatyBrown, thank you for your feedback. The code in index.py is transferred to index.js when @babinyurii could you please share the other issues? I will start working on your suggestions then. |
@oleksulkowski apologies, I missed that - in that case I think it’s fine as it is. |
I’d just like a last quick look through tomorrow - I’ll comment when I’m done. |
@fboehm OK, my review is complete now. Apart from extending the in-code documentation as @babinyurii discussed above, I think it looks good and meets all the requirements. It will be a useful tool and is well written and documented. |
Thank you, @KatyBrown! @babinyurii - did you have a chance to open issues in the software repository? If so, can you link to the issues in this thread? Thanks again! |
Here are the links to the issues: |
@oleksulkowski - do you have questions about the issues and how to resolve them? Please feel free to link here to your proposed resolutions for the issues. And please let me know if you have any questions. Thanks :) |
Hi, @babinyurii @KatyBrown, I have applied changes based on your suggestions - could you please check the issue pages and referenced commits? I added the docstrings in commit oleksulkowski/Syn-CpG-Spacer@dc58e2a |
Hi, @oleksulkowski ! Everything is ok. I've closed the issues. |
@oleksulkowski - are you using the proper citation for Bokeh? I found this: https://docs.bokeh.org/en/1.0.4/docs/citation.html#:~:text=To%20cite%20Bokeh%20in%20publications,.bokeh.pydata.org. It looks my link here is to an older version, maybe, of Bokeh? It seems like the citation in your pdf should include which version you use, too. |
@oleksulkowski - I can't quickly find another way to cite Progressive Web Apps, so, unless you know of one, we can leave it as is... although, if it has a version number, or a date on which you downloaded it or used it, you might want to add that to the citation. |
I changed the Bokeh reference to include the version we used now. Regarding the PWA reference, the date accessed is there in the paper.bib file ( |
Excellent, @oleksulkowski ! |
@editorialbot check references |
@editorialbot generate pdf |
@editorialbot commands |
|
Hello @fboehm, here are the things you can ask me to do:
|
@editorialbot recommend-accept |
|
|
👋 @openjournals/bcm-eics, this paper is ready to be accepted and published. Check final proof 👉📄 Download article If the paper PDF and the deposit XML files look good in openjournals/joss-papers#5192, then you can now move forward with accepting the submission by compiling again with the command |
@oleksulkowski as AEiC for JOSS I will now help to process this submission for acceptance in JOSS. I have checked this review, your repository, the archive link, and the paper. Everything seems in order, I only have the below point that requires your attention:
|
Thanks @Kevin-Mattheus-Moerman, it's fixed now. |
@editorialbot generate pdf |
@oleksulkowski Everything seems to be in order so we will now proceed to process this work for acceptance. |
@editorialbot accept |
|
Ensure proper citation by uploading a plain text CITATION.cff file to the default branch of your repository. If using GitHub, a Cite this repository menu will appear in the About section, containing both APA and BibTeX formats. When exported to Zotero using a browser plugin, Zotero will automatically create an entry using the information contained in the .cff file. You can copy the contents for your CITATION.cff file here: CITATION.cff
If the repository is not hosted on GitHub, a .cff file can still be uploaded to set your preferred citation. Users will be able to manually copy and paste the citation. |
🐘🐘🐘 👉 Toot for this paper 👈 🐘🐘🐘 |
🚨🚨🚨 THIS IS NOT A DRILL, YOU HAVE JUST ACCEPTED A PAPER INTO JOSS! 🚨🚨🚨 Here's what you must now do:
Any issues? Notify your editorial technical team... |
@oleksulkowski congratulations on this publication !! @fboehm Thank you for editing this one! And a special thanks to the reviewers: @babinyurii, @KatyBrown !!! 🥳 |
🎉🎉🎉 Congratulations on your paper acceptance! 🎉🎉🎉 If you would like to include a link to your paper from your README use the following code snippets:
This is how it will look in your documentation: We need your help! The 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: @oleksulkowski (Aleksander Sułkowski)
Repository: https://github.com/oleksulkowski/Syn-CpG-Spacer
Branch with paper.md (empty if default branch):
Version: v1.1.0
Editor: @fboehm
Reviewers: @babinyurii, @KatyBrown
Archive: 10.5281/zenodo.10781374
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
@babinyurii & @KatyBrown, 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 @fboehm 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 @babinyurii
📝 Checklist for @KatyBrown
The text was updated successfully, but these errors were encountered: