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
Software report:
github.com/AlDanial/cloc v 1.88 T=0.50 s (410.9 files/s, 179188.4 lines/s)
-------------------------------------------------------------------------------
Language files blank comment code
-------------------------------------------------------------------------------
Python 80 3920 7577 18171
JavaScript 17 4510 4494 17123
HTML 11 1369 33 8098
TeX 25 885 1158 7321
reStructuredText 11 911 3729 2783
SVG 1 0 0 2671
YAML 32 95 845 1151
CSS 6 294 52 1106
Markdown 8 199 0 554
Bourne Shell 5 11 53 60
JSON 2 0 0 60
make 2 22 11 50
DOS Batch 2 17 2 47
INI 2 0 0 8
TOML 1 0 0 6
-------------------------------------------------------------------------------
SUM: 205 12233 17954 59209
-------------------------------------------------------------------------------
gitinspector failed to run statistical information for the repository
Wordcount for paper.md
is 1344
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.5281/zenodo.6925534 is OK
- 10.1016/j.energy.2016.08.060 is OK
- 10.1016/j.esd.2016.09.002 is OK
- 10.1016/j.renene.2020.03.002 is OK
- 10.1016/j.egypro.2017.09.416 is OK
- 10.1016/j.rset.2021.100003 is OK
- 10.1016/j.seta.2021.101095 is OK
- 10.1016/j.esd.2021.11.008 is OK
- 10.1016/j.erss.2022.102636 is OK
MISSING DOIs
- 10.1891/9780826190123.ap02 may be a valid DOI for title: Transforming our world: the 2030 Agenda for Sustainable Development
INVALID DOIs
- None
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@phil-sandwell : Please update us about the open issues
@fneum : Any updates on how the review is going would be great :)
@fraukewiese : I am waiting with my further review on the already opened issues.
@fraukewiese My review is complete. The library CLOVER meets all criteria for me and I can recommend a publication in JOSS.
Thank you very much @EwaGomez for your thorough review!
@fneum : Could you give us a brief update on how the review is going or when you would be able to do it. Thank you very much :)
Sorry for not getting to this earlier. I have two main comments regarding the documentation and README. Otherwise, everything looks good or has already been addressed through @EwaGomez comments.
pip
route. Here, I think python -m src.clover ....
should be replaced with python -m clover. ....
.pandoc
can help you convert large chunks of the .tex
to .rst
format. Is this still up-to-date since it contains a 2020 date on the title page?Thank you very much @fneum ! @phil-sandwell : Please give us an update as soon as you have addressed the comments by @fneum
Thank you very much for your review @fneum
An issue was raised here which was merged here with updates to address how to run the clover-energy
package if installed via pip
and with directions to the wiki pages which recently replaced the .pdf
userguide.
Please let me know if there's anything else that you need!
Great @BenWinchester! Sorry that I didn't suspect the documentation to be placed on the Wiki Pages.
I have one mini suggestion remaining, then good to go!
In the README, it would be good to hit the keyword "documentation" somehow, in case someone is Ctrl + F
ing for it. Either as a header or just in-text where you incorporated the reference to the wiki.
@EwaGomez and @fneum : Thank you very much for your effort, your thorough review and your support in improving the submission!
@editorialbot generate pdf
@editorialbot check references
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.5281/zenodo.6925534 is OK
- 10.21105/joss.00884 is OK
- 10.20900/jsr20200009 is OK
- 10.5334/jors.188 is OK
- 10.1016/j.energy.2016.08.060 is OK
- 10.1016/j.esd.2016.09.002 is OK
- 10.1016/j.renene.2020.03.002 is OK
- 10.1016/j.egypro.2017.09.416 is OK
- 10.1016/j.rset.2021.100003 is OK
- 10.1016/j.seta.2021.101095 is OK
- 10.1016/j.esd.2021.11.008 is OK
- 10.1016/j.erss.2022.102636 is OK
- 10.1088/2515-7655/ac9dc0 is OK
- 10.25561/77296 is OK
MISSING DOIs
- 10.1891/9780826190123.ap02 may be a valid DOI for title: Transforming our world: the 2030 Agenda for Sustainable Development
INVALID DOIs
- None
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@BenWinchester and @phil-sandwell Two minor points: As raised by fneum: "In the README, it would be good to hit the keyword "documentation" somehow, in case someone is Ctrl + Fing for it. Either as a header or just in-text where you incorporated the reference to the wiki." Please add the DOI for Howells et al 2011: https://doi.org/10.1016/j.enpol.2011.06.033
@phil-sandwell : Could you then please:
I can then move forward with accepting the submission.
@fraukewiese , the two points raised above were addressed in this issue and then merged into our master
branch here and published on pypi here.
v5.0.6
, was made of the archived software,
The tag is available here;v5.0.6
available at 10.5281/zenodo.7575877 and the latest doi at 10.5281/zenodo.6925534;Please let me know if there's anything else needed!
@editorialbot set v5.0.6 as version
Done! version is now v5.0.6
@BenWinchester and @phil-sandwell The author list in the paper should exactly match the one in the archival deposit. Could you please adjust this?
Hi @fraukewiese ,
This has now been fixed here with the archive updated to match the paper.
We also spotted that bullet-point lists within the paper, which were compiling in our markdown editors, weren't being displayed properly by the action, so fixed this here . The paper.md
is now up-to-date on both our publish
(stand-alone) and master
branches. (Annoyingly, black
and pylint
, used for automated checking and master-branch protection, were updated in the last two days, which resulted in our automated checks failing, so there have been formatting changes to the code to pass these but no functional changes. If need be, we can trigger a new tagged release?)
Let me know if there's anything else that needs doing :smile:
@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 set 10.5281/zenodo.7575877 as archive
Done! Archive is now 10.5281/zenodo.7575877
@editorialbot recommend-accept
Attempting dry run of processing paper acceptance...
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.5281/zenodo.6925534 is OK
- 10.21105/joss.00884 is OK
- 10.20900/jsr20200009 is OK
- 10.1016/j.enpol.2011.06.033 is OK
- 10.5334/jors.188 is OK
- 10.1016/j.energy.2016.08.060 is OK
- 10.1016/j.esd.2016.09.002 is OK
- 10.1016/j.renene.2020.03.002 is OK
- 10.1016/j.egypro.2017.09.416 is OK
- 10.1016/j.rset.2021.100003 is OK
- 10.1016/j.seta.2021.101095 is OK
- 10.1016/j.esd.2021.11.008 is OK
- 10.1016/j.erss.2022.102636 is OK
- 10.1088/2515-7655/ac9dc0 is OK
- 10.25561/77296 is OK
MISSING DOIs
- 10.1891/9780826190123.ap02 may be a valid DOI for title: Transforming our world: the 2030 Agenda for Sustainable Development
INVALID DOIs
- None
:wave: @openjournals/pe-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/3950, then you can now move forward with accepting the submission by compiling again with the command @editorialbot accept
I'm sorry @phil-sandwell, I'm afraid I can't do that. That's something only eics are allowed to do.
Hi @phil-sandwell, just a final check to do: is 10.5281/zenodo.7612815 the correct DOI for the reviewed version? I followed the link for the DOI reported above and Zenodo said a newer one was available.
Hi @kyleniemeyer , the DOI 10.5281/zenodo.7575877 is correct for the reviewed version (v5.0.6
). 10.5281/zenodo.7612815 is a "beta" (pre-release) version, v5.0.7b1
released to deal with an issue that a researcher raised with respect to their modelling, merged here and released here :smiley:
@BenWinchester got it, thanks
@editorialbot accept
Doing it live! Attempting automated processing of paper acceptance...
π¦π¦π¦ π 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...
Congratulations @phil-sandwell & @BenWinchester et al. on your article's publication in JOSS!
Many thanks to @EwaGomez and @fneum for reviewing this, and @fraukewiese for editing.
: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.04799/status.svg)](https://doi.org/10.21105/joss.04799)
HTML:
<a style="border-width:0" href="https://doi.org/10.21105/joss.04799">
<img src="https://joss.theoj.org/papers/10.21105/joss.04799/status.svg" alt="DOI badge" >
</a>
reStructuredText:
.. image:: https://joss.theoj.org/papers/10.21105/joss.04799/status.svg
:target: https://doi.org/10.21105/joss.04799
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-->@phil-sandwell<!--end-author-handle-- (Philip Sandwell) Repository: https://github.com/CLOVER-energy/CLOVER Branch with paper.md (empty if default branch): publish Version: v5.0.6 Editor: !--editor-->@fraukewiese<!--end-editor-- Reviewers: @EwaGomez, @fneum Archive: 10.5281/zenodo.7575877
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
@EwaGomez & @fneum, 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 @fraukewiese 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 @EwaGomez
π Checklist for @fneum