Closed editorialbot closed 1 year ago
Hello humans, I'm @editorialbot, a robot that can help you with some common editorial tasks.
For a list of things I can do to help you, just type:
@editorialbot commands
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
@editorialbot generate pdf
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.1137/22M1503002 is OK
- 10.1017/pasa.2020.8 is OK
- 10.1093/mnras/stac1532 is OK
- 10.1093/mnras/stac1531 is OK
- 10.1093/mnras/stz1154 is OK
- 10.1093/mnras/staa1584 is OK
- 10.1093/mnras/stab229 is OK
MISSING DOIs
- None
INVALID DOIs
- None
Software report:
github.com/AlDanial/cloc v 1.88 T=0.28 s (142.2 files/s, 21916.8 lines/s)
-------------------------------------------------------------------------------
Language files blank comment code
-------------------------------------------------------------------------------
Python 17 717 1353 2365
Markdown 9 339 0 904
YAML 6 6 17 122
TeX 1 7 0 107
reStructuredText 5 58 60 56
DOS Batch 1 8 1 26
make 1 4 7 9
-------------------------------------------------------------------------------
SUM: 40 1139 1438 3589
-------------------------------------------------------------------------------
gitinspector failed to run statistical information for the repository
Wordcount for paper.md
is 654
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@apoudel2014 @Jammy2211 thanks again for agreeing to review this submission! When you are ready to begin your review, please comment @editorialbot generate my checklist
on this issue. As you work your way through the checklist, you may create issues on the GitHub repo for the software being reviewed. Please see the JOSS reviewer guidelines for more information. Let me know if you have any questions!
All done, I've put up 3 issues on the GitHub which should be straight forward enough to address.
Overall, popkinmocks
is an awesome little package with clear docs, easy-to-follow examples and it serves a need and purpose not currently filled elsewhere in the literature.
You may hear from me one day about simulating IFU observations of strong lenses with the tool!
@Jammy2211 thanks very much for your review!
@prashjet please begin addressing the issues relevant to the review (https://github.com/prashjet/popkinmocks/issues/75, https://github.com/prashjet/popkinmocks/issues/76, https://github.com/prashjet/popkinmocks/issues/77). You can reply to this thread when you feel you've addressed them.
@apoudel2014 just checking in on how the review is going. As you make your way through the checklist, please update your comment to indicate whether the software addresses the different points or not. Let me know if you have any questions!
I am working on it. Hopefully I will have something by next week. Will let you know if I have any questions.
Thanks, Amit
On Thu, Apr 6, 2023 at 9:01 AM Paul La Plante @.***> wrote:
@apoudel2014 https://github.com/apoudel2014 just checking in on how the review is going. As you make your way through the checklist, please update your comment to indicate whether the software addresses the different points or not. Let me know if you have any questions!
— Reply to this email directly, view it on GitHub https://github.com/openjournals/joss-reviews/issues/5225#issuecomment-1499296197, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIT5V6DL5P4BNXTLJNDB7HTW73SHNANCNFSM6AAAAAAVR3QMUM . You are receiving this because you were mentioned.Message ID: @.***>
-- Best Regards, Amit Poudel
I've completed my review and submitted 1 issue on GitHub, which is a minor comment. Regarding the paper, the Popkinmocks package is quite useful, and the paper does an excellent job of presenting the information in a clear and concise manner.
@apoudel2014 thanks very much for your review!
@prashjet please address the issue relevant to the review (https://github.com/prashjet/popkinmocks/issues/78). You can reply to this thread when you feel it's been addressed.
Hi all - many thanks for the positive reviews and useful suggestions. The latest merge to the main branch addresses all your comments (except the pypi suggestion, which I will come back to in the future).
@Jammy2211
You may hear from me one day about simulating IFU observations of strong lenses with the tool!
I look forward to it!
FYI - had some problems with readthedocs
updating to the latest release, but sorted now - the latest stable
docs are the corrected version
@prashjet thanks for addressing the reviewers' comments!
@Jammy2211 when you get a chance, please check to make sure all of your initial concerns have been addressed, and update your checklist accordingly. Thanks again for reviewing!
Done
@Jammy2211 thanks very much for your review!
@prashjet both reviewers have recommended acceptance of the submission, so we can move forward with the final acceptance checks. If the software has changed in the course of the review, please make a new tagged release of the repository. After that, please archive it (on Zenodo, figshare, or some other long-term hosting platform). Once those are done, please post the version number and DOI of the archive in this thread. Let me know if you have any questions!
@editorialbot generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@editorialbot check references
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.1137/22M1503002 is OK
- 10.1017/pasa.2020.8 is OK
- 10.1093/mnras/stac1532 is OK
- 10.1093/mnras/stac1531 is OK
- 10.1093/mnras/stz1154 is OK
- 10.1093/mnras/staa1584 is OK
- 10.1093/mnras/stab229 is OK
- 10.1111/j.1365-2966.2007.11752.x is OK
- 10.1111/j.1365-2966.2011.18496.x is OK
- 10.1088/2041-8205/764/1/L1 is OK
- 10.1038/s41550-017-0348-1 is OK
MISSING DOIs
- None
INVALID DOIs
- None
Once those are done, please post the version number and DOI of the archive in this thread. Let me know if you have any questions!
popkinmocks (1.0.1) https://doi.org/10.5281/zenodo.7890628
Thanks!
@prashjet thank you!
@editorialbot set 10.5281/zenodo.7890628 as archive
Done! archive is now 10.5281/zenodo.7890628
@editorialbot set v1.0.1 as version
Done! version is now v1.0.1
@editorialbot recommend-accept
Attempting dry run of processing paper acceptance...
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.1137/22M1503002 is OK
- 10.1017/pasa.2020.8 is OK
- 10.1093/mnras/stac1532 is OK
- 10.1093/mnras/stac1531 is OK
- 10.1093/mnras/stz1154 is OK
- 10.1093/mnras/staa1584 is OK
- 10.1093/mnras/stab229 is OK
- 10.1111/j.1365-2966.2007.11752.x is OK
- 10.1111/j.1365-2966.2011.18496.x is OK
- 10.1088/2041-8205/764/1/L1 is OK
- 10.1038/s41550-017-0348-1 is OK
MISSING DOIs
- None
INVALID DOIs
- None
:wave: @openjournals/aass-eics, this paper is ready to be accepted and published.
Check final proof :point_right::page_facing_up: Download article
If the paper PDF and the deposit XML files look good in https://github.com/openjournals/joss-papers/pull/4195, then you can now move forward with accepting the submission by compiling again with the command @editorialbot accept
@prashjet — I've opened a tiny PR to your paper fixing some typographical issues in the bibliography. One last thing I'll need from you: Can you update the Zenodo record metadata (there should be an "Edit" button in the top right corner of this page when you're logged in) so that the title is "popkinmocks: mock IFU datacubes for modelling stellar populations and kinematics" to match the paper? Then I'm happy to continue with publication!
@dfm - thanks Dan, both just done.
Then I'm happy to continue with publication!
Sweet!
@editorialbot generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@editorialbot recommend-accept
Attempting dry run of processing paper acceptance...
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.1137/22M1503002 is OK
- 10.1017/pasa.2020.8 is OK
- 10.1093/mnras/stac1532 is OK
- 10.1093/mnras/stac1531 is OK
- 10.1093/mnras/stz1154 is OK
- 10.1093/mnras/staa1584 is OK
- 10.1093/mnras/stab229 is OK
- 10.1111/j.1365-2966.2007.11752.x is OK
- 10.1111/j.1365-2966.2011.18496.x is OK
- 10.1088/2041-8205/764/1/L1 is OK
- 10.1038/s41550-017-0348-1 is OK
MISSING DOIs
- None
INVALID DOIs
- None
:wave: @openjournals/aass-eics, this paper is ready to be accepted and published.
Check final proof :point_right::page_facing_up: Download article
If the paper PDF and the deposit XML files look good in https://github.com/openjournals/joss-papers/pull/4201, then you can now move forward with accepting the submission by compiling again with the command @editorialbot accept
@editorialbot accept
Doing it live! Attempting automated processing of paper acceptance...
Ensure proper citation by uploading a plain text CITATION.cff file to the default branch of your repository.
If using GitHub, a Cite this repository menu will appear in the About section, containing both APA and BibTeX formats. When exported to Zotero using a browser plugin, Zotero will automatically create an entry using the information contained in the .cff file.
You can copy the contents for your CITATION.cff file here:
``` cff-version: "1.2.0" authors: - family-names: Jethwa given-names: Prashin orcid: "https://orcid.org/0000-0003-0010-8129" doi: 10.5281/zenodo.7890628 message: If you use this software, please cite our article in the Journal of Open Source Software. preferred-citation: authors: - family-names: Jethwa given-names: Prashin orcid: "https://orcid.org/0000-0003-0010-8129" date-published: 2023-05-05 doi: 10.21105/joss.05225 issn: 2475-9066 issue: 85 journal: Journal of Open Source Software publisher: name: Open Journals start: 5225 title: "popkinmocks: mock IFU datacubes for modelling stellar populations and kinematics" type: article url: "https://joss.theoj.org/papers/10.21105/joss.05225" volume: 8 title: "popkinmocks: mock IFU datacubes for modelling stellar populations and kinematics" ```
If the repository is not hosted on GitHub, a .cff file can still be uploaded to set your preferred citation. Users will be able to manually copy and paste the citation.
🐦🐦🐦 👉 Tweet for this paper 👈 🐦🐦🐦
🐘🐘🐘 👉 Toot 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...
Many thanks to @apoudel2014 and @Jammy2211 for reviewing and to @plaplant for editing! JOSS relies upon the volunteer effort of people like you and we simply wouldn't be able to do this without you!!
@prashjet — Your paper is now accepted and published in JOSS! :zap::rocket::boom:
: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.05225/status.svg)](https://doi.org/10.21105/joss.05225)
HTML:
<a style="border-width:0" href="https://doi.org/10.21105/joss.05225">
<img src="https://joss.theoj.org/papers/10.21105/joss.05225/status.svg" alt="DOI badge" >
</a>
reStructuredText:
.. image:: https://joss.theoj.org/papers/10.21105/joss.05225/status.svg
:target: https://doi.org/10.21105/joss.05225
This is how it will look in your documentation:
We need your help!
The 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: !--author-handle-->@prashjet<!--end-author-handle-- (Prashin Jethwa) Repository: https://github.com/prashjet/popkinmocks Branch with paper.md (empty if default branch): Version: v1.0.1 Editor: !--editor-->@plaplant<!--end-editor-- Reviewers: @apoudel2014, @Jammy2211 Archive: 10.5281/zenodo.7890628
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
@apoudel2014 & @Jammy2211, your review will be checklist based. Each of you will have a separate checklist that you should update when carrying out your review. First of all you need to run this command in a separate comment to create the checklist:
The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @plaplant 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 ✨
Checklists
📝 Checklist for @apoudel2014
📝 Checklist for @Jammy2211