Closed whedon closed 3 years ago
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @seisman, @ThomasLecocq it looks like you're currently assigned to review this paper :tada:.
:warning: JOSS reduced service mode :warning:
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.
: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
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
@whedon generate pdf
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.25080/Majora-92bf1922-00a is OK
MISSING DOIs
- 10.1088/1749-4699/8/1/014003 may be a valid DOI for title: ObsPy: A bridge for seismology into the scientific Python ecosystem
INVALID DOIs
- None
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
π @d-chambers @seisman @ThomasLecocq 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 with the JOSS requirements. 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 and pull requests on the software repository. When doing so, please mention openjournals/joss-reviews#2236
so that a link is created to this thread (and I can keep an eye on what is happening). Please also feel free to comment and ask questions on this thread. In my experience, it is better to post comments/questions/suggestions as you come across them instead of waiting until you've reviewed the entire package.
We aim for reviews to be completed within about 4 weeks. Please let me know if any of you require some more time. We can also use @whedon (our bot) to set automatic reminders if you know you'll be away for a known period of time.
Please feel free to ping me here (@leouieda) or email me privately if you have any questions/concerns.
@d-chambers please have a look at the missing DOI report from Whedon above and add the missing DOIs to your BibTex file.
@leouieda my apologies, I missed this last request. I just added the correct DOI to the Bib Tex file.
Hi @ThomasLecocq @seisman just checking in on the status of this review π Let me know if you have any questions or concerns.
Sorry for forgetting this paper. I'll review it in the following week.
Thank you for the update @seisman π
ππΌ Hi @seisman @ThomasLecocq just checking in to see how things are going with the reviews. It would be nice to get some progress on this soon but let me know if you require more time (weβre all living in strange times).
Hi @leouieda, sorry for the delay. I'm a little too busy with job interviews and preparing for AGU meeting recently. I may have more time in December.
@seisman thanks for the update. I realize that the earlier than usual AGU deadlines are causing problems. Good luck on the job interviews and let me know when you have some more time.
@d-chambers apologies for the unusually long delays with this paper. I really appreciate your patience in these strange times we live in π
Hi @leouieda,
No worries, I understand the circumstances.
Hi @seisman @ThomasLecocq are there any updates on this review?
I'll start to review the submission in the following week.
:wave: @ThomasLecocq @leouieda happy new year! Just checking in here on the status of this review?
Sorry to pester, but @leouieda are the any updates on this? I'm keen to have this out so that I can cite it for some upcoming papers for which ObsPlus had been very useful!
It's all good for me
I would love to get this wrapped up soon. I addressed all of @seisman's comments thus far, but I am happy to make any additional improvements that might be required.
I reviewed the original v0.1.0 version. It's good for me.
I can't check the boxes above, but I have no further remarks other than the citation for Pandas seem buggy in the latest rendered PDF. Other than that, it's good to go
@whedon re-invite @ThomasLecocq as reviewer
OK, the reviewer has been re-invited.
@thomaslecocq please accept the invite by clicking this link: https://github.com/openjournals/joss-reviews/invitations
This should let you check the boxes
@whedon generate pdf
PDF failed to compile for issue #2696 with the following error:
Error reading bibliography file paper.bib:
(line 7, column 9):
unexpected "Y"
expecting space, ",", white space or "}"
Looks like we failed to compile the PDF
All π’ for me, just the missing Pandas citation in the PDF available above.
Ok the pdf now compiles and I think the citation looks correct.
@whedon generate pdf
@whedon generate pdf
(whedon commands need to be on the first line of a comment)
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
All good now!
Hi @d-chambers, I'm the AEIC on duty this week, and stepping in here to help get this past the finish line. It looks like the reviewers have both recommended this for acceptance.
Some final steps needed:
@whedon generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
Current-version DOI: http://doi.org/10.5281/zenodo.4544008
All-version DOI: http://doi.org/10.5281/zenodo.3228432
ok @kyleniemeyer, let me know if anything else is required. Also, thanks for your work on JOSS :smile:
Thanks @d-chambers - can you clean up the author list on the current-version Zenodo record to match that of the paper?
Done. My apologies for not doing it originally, I am new to Zenodo.
@whedon set 10.5281/zenodo.4544008 as archive
OK. 10.5281/zenodo.4544008 is the archive.
@whedon accept
Attempting dry run of processing paper acceptance...
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.1088/1749-4699/8/1/014003 is OK
- 10.25080/Majora-92bf1922-00a is OK
MISSING DOIs
- None
INVALID DOIs
- None
:wave: @openjournals/joss-eics, this paper is ready to be accepted and published.
Check final proof :point_right: https://github.com/openjournals/joss-papers/pull/2237
If the paper PDF and Crossref deposit XML look good in https://github.com/openjournals/joss-papers/pull/2237, then you can now move forward with accepting the submission by compiling again with the flag deposit=true
e.g.
@whedon accept deposit=true
@d-chambers sorry, looks like I missed something - the QuakeML reference is actually missing some key citation details, including a few authors, the volume (54) and number (1), and DOI: 10.4401/ag-4874
Can you add these?
Ok, I have added the volume, number and DOI.
The missing authors issue is interesting. The authors are included in the bib but don't show up in the PDF. Same issue for the ObsPy reference. I will dig into it more.
@whedon generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
Sorry haven't merged the changes to master yet, still trying to figure out why the middle authors aren't showing.
Ah, I think the issue related to the commas in the author
fields.
For the ObsPy reference, it should be {Krischer, L. and Megies, T. and Barsch, R. and Beyreuther, M. and Lecocq, T. and Caudron, C. and Wassermann, J.}
(you want "and" rather than commas between the authors)
Also, you also do not need brackets around all the surnames, except in uncommon cases.
Submitting author: @d-chambers (Derrick Chambers) Repository: https://github.com/niosh-mining/obsplus Version: v0.1.0 Editor: @leouieda Reviewer: @seisman, @ThomasLecocq Archive: 10.5281/zenodo.4544008
:warning: JOSS reduced service mode :warning:
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
@seisman & @ThomasLecocq, 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 @leouieda 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 @seisman
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @ThomasLecocq
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper