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]: Generating fragment density plots in R/Bioconductor with VplotR #3009
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @henrykironde, @fgeier it looks like you're currently assigned to review this paper 🎉. Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post. ⭐ Important ⭐ 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:
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
|
|
PDF failed to compile for issue #3009 with the following error: Can't find any papers to compile :-( |
👋🏼 @js2264, @henrykironde, @fgeier - this is the review thread for the paper. All of our communications will happen here from now on. Both reviewers have checklists at the top of this thread. As you go over the submission, please check any items that you feel have been satisfied. There are also links to the JOSS reviewer guidelines. 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, the reviewers are encouraged to submit issues directly in the software repository. If you do so, please mention this thread so that a link is created (and I can keep an eye on what is happening). Please also feel free to comment and ask questions in this thread. It is often easier to post comments/questions/suggestions as you come across them instead of waiting until you've reviewed the entire package. Please feel free to ping me (@csoneson) if you have any questions/concerns. |
@whedon generate pdf from branch JOSS |
|
@js2264, I have made some recommendations as issues on the repo. js2264/VplotR#4. Please let me know if there is something not clear as I continue to review. |
@henrykironde I have addressed your initial issues. Please do let me know if you feel like I have not addressed some of them in enough depth! |
@csoneson I have merged JOSS branch to master branch. Should I do anything for whedon to know about this change? |
@whedon generate pdf |
No, I think this should be all good. And whedon seems to find the paper in the master branch |
@henrykironde FYI I have closed the second round of issues you opened. Let me know if anything else needs to be changed! |
Thanks for the update. I checked the issues and they look good. I am trying to evaluate the paper at the moment. I know the total number of words exceed the maximum (1000) by ~120 words. I will have to get some advice on that. For now, I will go through the writing and get back to the checkboxes. |
@henrykironde Let me know if there is anything related to the paper I can do (I can try to shorten it if the length is an issue). |
@js2264 thanks for the great science. I have finished the review and the submission meets the criteria for acceptance. |
Thanks @henrykironde! |
Thank you very much for your insightful review of the package @henrykironde! |
👋 @fgeier, please update us on how your review is going (this is an automated reminder). |
@fgeier - could you give us a quick update about the status of your review? Thanks! |
@whedon generate pdf |
@whedon set 10.5281/zenodo.4562440 as archive |
OK. 10.5281/zenodo.4562440 is the archive. |
@whedon set v1.2.0 as version |
OK. v1.2.0 is the version. |
Alright, looks good! I'm going to hand over now to the associate editor in chief on rotation this week for the final steps. |
@whedon accept |
|
|
👋 @openjournals/joss-eics, this paper is ready to be accepted and published. Check final proof 👉 openjournals/joss-papers#2112 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#2112, then you can now move forward with accepting the submission by compiling again with the flag
|
@fgeier We need all of the boxes checked to finish up reviews. This is our metric for the review and want them ticked off for the review to be finished. For
if there are no claims, you can check this off. For
If this is not done in the paper, the author should fix this. If there is no comparable software, that should be stated and then the box can be checked off. |
Looks like everything else is in order to finalize publication (version updated, Zenodo archive looks good, and paper is fine)! |
@js2264 no you don't need to update the archive. |
@whedon generate pdf |
@fgeier It looks like you should be able to finalize your review by checking off the final two boxes now, but either way please let us know. |
@kthyng I check them off. Sorry for the confusion on my side. |
@whedon accept deposit=true |
|
🐦🐦🐦 👉 Tweet 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... |
Congrats on your new publication @js2264! Thanks to editor @csoneson and reviewers @henrykironde and @fgeier for your time, hard work, and expertise! 🎉 |
🎉🎉🎉 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! 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: @js2264 (Jacques Serizay)
Repository: https://github.com/js2264/VplotR/
Version: v1.2.0
Editor: @csoneson
Reviewer: @henrykironde, @fgeier
Archive: 10.5281/zenodo.4562440
Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post.
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
@henrykironde & @fgeier, 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 @csoneson 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 ✨
Review checklist for @henrykironde
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @fgeier
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
The text was updated successfully, but these errors were encountered: