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]: ADI.jl: A Julia Package for High-Contrast Imaging #2843
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @giordano, @semaphoreP 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:
|
|
👋 @semaphoreP, please update us on how your review is going. |
👋 @giordano, please update us on how your review is going. |
@whedon preview |
I'm sorry human, I don't understand that. You can see what commands I support by typing:
|
@whedon generate pdf |
Just finished the checklist. I am done reviewing I believe. Let me know if there's anything else. |
@semaphoreP: Yes it looks like you're all set. Thank you very much for your review!! |
I also just finished the checklist. I'd like to mention that I had left some comments in
All reported issues have been resolved already. I also opened a couple of pull requests with trivial changes:
|
I'll mention, too, I have the working changes on the master branch ready to release, which will change the version to |
@giordano and @semaphoreP: Thanks for reviews! I really appreciate your time and your constructive suggestions. @mileslucas: There are two things for you to do before we do the final processing:
Thanks all!! |
@whedon generate pdf |
@dfm All released and updated, the Zenodo link is https://zenodo.org/deposit/4323554 with the doi |
@mileslucas: Please update the metadata on the Zenodo archive to match the manuscript (as requested above in bold text :D). The title should be |
@dfm I don't know what to say: every time I edit the DOI it says it has the correct information. Saving doesn't change anything and when I try to publish I get a server error |
@mileslucas: Bizarre! It looks like you used some sort of GitHub actions workflow to publish your release? Perhaps it has something to do with that. Can you try uploading a new snapshot of the code with the right metadata to Zenodo directly? Thanks! |
Yeah, I'm using the zenodo feature that automatically updates with each github release. I sent a support request to zenodo to see if that can be resolved before I upload a new version. I'll ping here when I've got it figured out. |
Ok thanks! Ping me when you have it sorted and I'll take it from there. |
I was told to |
@mileslucas: Excellent - Looks good now! |
@whedon check references |
|
@whedon set 10.5281/zenodo.4323554 as archive |
OK. 10.5281/zenodo.4323554 is the archive. |
@whedon set v0.5.3 as version |
OK. v0.5.3 is the version. |
@whedon accept |
|
|
👋 @openjournals/joss-eics, this paper is ready to be accepted and published. Check final proof 👉 openjournals/joss-papers#1992 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#1992, then you can now move forward with accepting the submission by compiling again with the flag
|
@mileslucas: I've handed this off to the Editors-in-Chief to handle the final processing. They might have some last edits for the manuscript and once that's been sorted we'll have this published! @giordano, @semaphoreP: Thanks for your reviews!! I really appreciate the time that you've put in and for your constructive recommendations for the code. Thanks again!! |
Everything looks to be in order! @mileslucas the summary in your paper is excellent! This is uncommon. We can proceed. |
@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 @mileslucas! Thanks to editor @dfm and reviewers @giordano and @semaphoreP for your time 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: @mileslucas (Miles Lucas)
Repository: https://github.com/JuliaHCI/ADI.jl
Version: v0.5.3
Editor: @dfm
Reviewer: @giordano, @semaphoreP
Archive: 10.5281/zenodo.4323554
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
@giordano & @semaphoreP, 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 @dfm 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 @giordano
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @semaphoreP
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
The text was updated successfully, but these errors were encountered: