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]: AtlasReader: A Python package to generate coordinate tables, region labels, and informative figures from statistical MRI images #1257
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @andrewheusser it looks like you're currently assigned as the reviewer for this paper 🎉. ⭐ 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:
|
|
@andrewheusser, thank you for the prompt and thorough review! It looks like you're all done. For @miykael, the next steps are to merge these changes and confirm that you think it's ready to be accepted. Then I'll look everything over and check that I agree. If that's the case, I'll then ask for the DOI of an archived copy of the code and what version of the software corresponds to the code after the revisions related to the review--and then we're all set! |
Thank you @andrewheusser for the very nice and helpful feedback! Your comments were much appreciated and helped to improve the toolbox. @cMadan - I've merged the changes, created a new GitHub release and a new PyPi release. So, the current version number is now v0.1.0. In my opinion (and this of the co-authors), the toolbox is ready to be accepted. Can I update the version number in the JOSS publication or is that up to you? Is there anything else that I can help with? |
@whedon generate pdf |
|
@whedon generate pdf |
|
I apologize for the repeated |
@whedon generate pdf |
|
Sorry, found a small typo in the references. But otherwise, everything is good to go. |
@whedon generate pdf |
|
my pleasure, nice job on the package! |
@whedon set v0.1.0 as version |
OK. v0.1.0 is the version. |
@whedon set 10.5281/zenodo.2575731 as archive |
OK. 10.5281/zenodo.2575731 is the archive. |
@whedon accept |
|
Check final proof 👉 openjournals/joss-papers#518 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#518, then you can now move forward with accepting the submission by compiling again with the flag
|
|
I think those missing DOIs aren't correct here, they are related but more tutorial to the packages than for the packages per se. As discussed in the review (miykael/atlasreader#105), I think there aren't DOIs for those references. @miykael, you're all set congratulations! @andrewheusser, thank you again for the thorough review. @arfon, can you do the honors? |
Done. The proof I generated locally ended up doing a slightly better job with figure layout so I processed the paper manually. @andrewheusser - many thanks for your review here and to @cMadan for editing this submission ✨ @miykael - your paper is now accepted into JOSS ⚡🚀💥 |
🎉🎉🎉 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:
|
@andrewheusser - thank you again for your great review! @cMadan - thank you for your work and accepting the paper. @arfon - thank you for changing the figure layout. It's now exactly how I wanted it. |
👍 my pleasure |
Submitting author: @miykael (Michael Philipp Notter)
Repository: https://github.com/miykael/atlasreader
Version: v0.1.0
Editor: @cMadan
Reviewer: @andrewheusser
Archive: 10.5281/zenodo.2575731
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
@andrewheusser, 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 @cMadan know.
✨ Please try and complete your review in the next two weeks ✨
Review checklist for @andrewheusser
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?The text was updated successfully, but these errors were encountered: