Closed whedon closed 4 years ago
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @KedoKudo 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:
OK DOIs
- 10.5281/ZENODO.11813 is OK
- 10.1016/j.msea.2008.05.017 is OK
- 10.3390/met5042252 is OK
- 10.1016/j.msea.2016.01.059 is OK
- 10.1016/j.actamat.2017.04.028 is OK
- 10.1107/s1600576717004708 is OK
- 10.1107/s1600576715004306 is OK
- 10.1107/s1600577515002283 is OK
- 10.1107/s1600576715010390 is OK
- 10.1080/08957959608201408 is OK
MISSING DOIs
- None
INVALID DOIs
- None
@jgostick - you can add a new reviewer here (when you've identified one) with @whedon add @reviewer2 as reviewer
. Unfortunately you'll need to do a small amount of manual work to add a checklist for the second reviewer (Whedon doesn't know how to do this yet). Any issues, let me know :-)
@whedon add @mikapfl as reviewer
OK, @mikapfl is now a reviewer
@jgostick - I finished my review of xrdfit
and submitted my feedbacks to the developers on the corresponding repo (https://github.com/LightForm-group/xrdfit/issues/39, https://github.com/LightForm-group/xrdfit/issues/38).
@merrygoat and @christopher-daniel - please feel free to ping me if you have any questions about the feedbacks.
@KedoKudo Thank you very much for your feedback. I will address the feedback items individually on the issues you have raised in the xrdfit
repository.
Thanks @KedoKudo! @merrygoat can you let me know in this issue when you've made the changes?
@mikapfl, how is your review coming along? It's only been 3 weeks, but things seem to move so fast at JOSS.
Hi,
I've been down with a cold the last week, but hope to do some progress with the review this weekend.
Cheers
Mika
Thanks @jgostick. Done the easy changes already. My colleague @christopher-daniel is on holiday this week but we will meet to resolve the outstanding issues next week.
Hi,
I've looked into the paper itself. I think it is a nice and short description of the the capabilities of xrdfit
and motivates well what the purpose of xrdfit
is. I found only a couple of minor things which could be clearer:
The experiment requires a beamline, such as Diamond Light Source […]
However, the Diamond Light Source is a synchrotron radiation facility comprising many beamlines, so this is not quite correct. Maybe something along the lines "The experiment requires a beamline at a synchrotron radiation facily such as […]" or such might be better.
However, since many intensity profiles are collected during each experiment, with detectors recording at speeds of up to 250 Hz (Diamond Light Source Ltd, 2020b),
Here, I think the web page of the I12 beamline at Diamond might not be an ideal citation. Maybe https://doi.org/10.1109/NSSMIC.2012.6551180 would be the best citation for the recording speeds of PILATUS3 X CdTe detectors? This is the best description of the PILATUS3 ASIC I know of. They specify a maximum frame rate of 500 Hz, which is not realized in the commercial 2M detector, which is described in detail at the web page of dectris: https://www.dectris.com/products/pilatus3/pilatus3-x-cdte-for-synchrotron/pilatus3-x-cdte-2m/ .
Best Regards,
Mika
Hi @merrygoat, did you notice the comments from @mikapfl above? He has a few 'unchecked' boxes for you to address, but this is almost over the finish line!
That's it - I think we have addressed all of the outstanding issues. Let me know if I have missed anything!
@merrygoat Here is a list of what to do next:
@jgostick Thank you.
@whedon set 10.5281/zenodo.3991638 as archive
OK. 10.5281/zenodo.3991638 is the archive.
@whedon accept
Attempting dry run of processing paper acceptance...
Reference check summary:
OK DOIs
- 10.5281/ZENODO.11813 is OK
- 10.1016/j.msea.2008.05.017 is OK
- 10.3390/met5042252 is OK
- 10.1016/j.msea.2016.01.059 is OK
- 10.1016/j.actamat.2017.04.028 is OK
- 10.1107/s1600576717004708 is OK
- 10.1107/s1600576715004306 is OK
- 10.1107/s1600577515002283 is OK
- 10.1107/s1600576715010390 is OK
- 10.1080/08957959608201408 is OK
- 10.1107/S1600576716000455 is OK
- 10.1107/S0021889869006558 is OK
- 10.1017/S0885715613001346 is OK
- 10.1109/NSSMIC.2012.6551180 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/1663
If the paper PDF and Crossref deposit XML look good in https://github.com/openjournals/joss-papers/pull/1663, then you can now move forward with accepting the submission by compiling again with the flag deposit=true
e.g.
@whedon accept deposit=true
Hi @merrygoat this paper is now acceptable, and an Editor in Chief will come make the final acceptance. Thanks to the reviewers their for time!
In the generated pdf (https://github.com/openjournals/joss-papers/blob/5c52f575ebfaac4c6c44846620f181dd6859ae5a/joss.02381/10.21105.joss.02381.pdf) the formatting of the citations at the end of the first paragraph looks funny. It somehow uses []
instead of ()
and the Hammersley_2016
also doesn't look right.
@mikapfl Thank you - you are right, we had missed an @ symbol there. Corrected and released as version v1.1.2.
@jgostick Is it possible to tell whedon to use another version? The updated doi is: 10.5281/zenodo.3997251
We would also like to thank both reviewers - your comments were insightful and lead to significant improvements.
@whedon set v1.1.2 as version
OK. v1.1.2 is the version.
@whedon set 10.5281/zenodo.3997251 as archive
OK. 10.5281/zenodo.3997251 is the archive.
@whedon accept
Attempting dry run of processing paper acceptance...
: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/1672
If the paper PDF and Crossref deposit XML look good in https://github.com/openjournals/joss-papers/pull/1672, then you can now move forward with accepting the submission by compiling again with the flag deposit=true
e.g.
@whedon accept deposit=true
Reference check summary:
OK DOIs
- 10.5281/ZENODO.11813 is OK
- 10.1016/j.msea.2008.05.017 is OK
- 10.3390/met5042252 is OK
- 10.1016/j.msea.2016.01.059 is OK
- 10.1016/j.actamat.2017.04.028 is OK
- 10.1107/s1600576717004708 is OK
- 10.1107/s1600576715004306 is OK
- 10.1107/s1600577515002283 is OK
- 10.1107/s1600576715010390 is OK
- 10.1080/08957959608201408 is OK
- 10.1107/S1600576716000455 is OK
- 10.1107/S0021889869006558 is OK
- 10.1017/S0885715613001346 is OK
- 10.1109/NSSMIC.2012.6551180 is OK
MISSING DOIs
- None
INVALID DOIs
- None
@merrygoat These are some required steps before we proceed:
@Kevin-Mattheus-Moerman Ah yes - thank you. I forgot I would have to update the metadata again with the new version. I have done that now.
I am also happy with the proof.
@whedon accept
Attempting dry run of processing paper acceptance...
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.5281/ZENODO.11813 is OK
- 10.1016/j.msea.2008.05.017 is OK
- 10.3390/met5042252 is OK
- 10.1016/j.msea.2016.01.059 is OK
- 10.1016/j.actamat.2017.04.028 is OK
- 10.1107/s1600576717004708 is OK
- 10.1107/s1600576715004306 is OK
- 10.1107/s1600577515002283 is OK
- 10.1107/s1600576715010390 is OK
- 10.1080/08957959608201408 is OK
- 10.1107/S1600576716000455 is OK
- 10.1107/S0021889869006558 is OK
- 10.1017/S0885715613001346 is OK
- 10.1109/NSSMIC.2012.6551180 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/1687
If the paper PDF and Crossref deposit XML look good in https://github.com/openjournals/joss-papers/pull/1687, then you can now move forward with accepting the submission by compiling again with the flag deposit=true
e.g.
@whedon accept deposit=true
@whedon accept deposit=true
Doing it live! Attempting automated processing of paper acceptance...
🐦🐦🐦 👉 Tweet for this paper 👈 🐦🐦🐦
🚨🚨🚨 THIS IS NOT A DRILL, YOU HAVE JUST ACCEPTED A PAPER INTO JOSS! 🚨🚨🚨
Here's what you must now do:
Party like you just published a paper! 🎉🌈🦄💃👻🤘
Any issues? Notify your editorial technical team...
: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 snippets:
Markdown:
[![DOI](https://joss.theoj.org/papers/10.21105/joss.02381/status.svg)](https://doi.org/10.21105/joss.02381)
HTML:
<a style="border-width:0" href="https://doi.org/10.21105/joss.02381">
<img src="https://joss.theoj.org/papers/10.21105/joss.02381/status.svg" alt="DOI badge" >
</a>
reStructuredText:
.. image:: https://joss.theoj.org/papers/10.21105/joss.02381/status.svg
:target: https://doi.org/10.21105/joss.02381
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:
Congratulations @merrygoat.
@KedoKudo, @mikapfl thanks for reviewing this work for JOSS!
Submitting author: @merrygoat (Peter Crowther) Repository: https://github.com/LightForm-group/xrdfit Version: v1.1.2 Editor: @jgostick Reviewers: @KedoKudo, @mikapfl Archive: 10.5281/zenodo.3997251
: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
@KedoKudo, 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 @jgostick know.
✨ Please try and complete your review in the next six weeks ✨
Review checklist for @KedoKudo
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @mikapfl
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper