Closed whedon closed 6 years ago
Hello human, I'm @whedon. I'm here to help you with some common editorial tasks. @jkmacc-LANL it looks like you're currently assigned as the reviewer for this paper :tada:.
:star: Important :star:
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:
@whedon commands
Attempting PDF compilation. Reticulating splines etc...
https://github.com/openjournals/joss-papers/blob/joss.00530/joss.00530/10.21105.joss.00530.pdf
Hi @jkmacc-LANL – the review will take place in this issue. Please see the instructions and checklist above, and feel free to let me know if you have any questions!
PyDMD is a well-documented, well-tested, easy to install software package. Congratulations! It was a pleasure to review this (...and thanks for your patience). I only had a few minor requests (below). These aren't show-stoppers, so I didn't create issues for them in the project repo. With these minor tweaks, this contribution is easily ready to be accepted.
I installed and ran the tutorial notebooks on OS X 10.12.6 with Python 3.6.4. It went very smoothly.
ffmpeg
. I recommended that this be explicit at the top of the notebook, or in the cell (7) that uses it.@jkmacc-LANL thank you very much! I will proceed in implementing your suggestions in the next few days!
@whedon generate pdf
Attempting PDF compilation. Reticulating splines etc...
https://github.com/openjournals/joss-papers/blob/joss.00530/joss.00530/10.21105.joss.00530.pdf
@jakevdp I have implemented all the suggestions and regenerated the pdf. What do I have to do now?
@mtezzele I see your changes, and they look good. Regarding the middle point, my suggestion was to add a slightly longer description to https://mathlab.github.io/PyDMD/, not on the README. I apologize for the confusion. Everything else is great👍
Done! I'm sorry I misread, my fault.
@jkmacc-LANL, has @mtezzele addressed all your comments?
Yes, they are addressed. Thanks!
@jakevdp what do I have to do now?
@mtezzele - At this point could you make an archive of the reviewed software in Zenodo/figshare/other service and update this thread with the DOI of the archive? I can then move forward with accepting the submission.
@arfon We have upload the package and this is the DOI for all the versions: https://doi.org/10.5281/zenodo.1171787
@whedon set 10.5281/zenodo.1171787 as archive
OK. 10.5281/zenodo.1171787 is the archive.
@jkmacc-LANL - many thanks for your review here and to @jakevdp for editing this submission ✨
@ndem0 @mtezzele - your paper is now accepted in JOSS and your DOI is https://doi.org/10.21105/joss.00530 ⚡️ 🚀 💥
:tada::tada::tada: Congratulations on your paper acceptance! :tada::tada::tada:
If you would like to include a link to your paper from your README use the following code snippet:
[![DOI](http://joss.theoj.org/papers/10.21105/joss.00530/status.svg)](https://doi.org/10.21105/joss.00530)
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:
Thank you very much @jkmacc-LANL, @jakevdp, and @arfon for your time and efforts!
Yes, thanks everyone. 10/10, would review again 😉 !
Submitting author: @mtezzele (Marco Tezzele) Repository: https://github.com/mathLab/PyDMD Version: v0.1.0 Editor: @jakevdp Reviewer: @jkmacc-LANL Archive: 10.5281/zenodo.1171787
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
@jkmacc-LANL, 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 @jakevdp know.
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?