Closed editorialbot closed 4 months 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/141000671 is OK
- 10.5334/jors.151 is OK
- 10.5281/zenodo.7738525 is OK
- 10.11588/heidok.00002980 is OK
- 10.1007/978-3-319-93031-2_27 is OK
- 10.1016/S0168-9274(99)00020-3 is OK
- 10.48550/arXiv.2103.05244 is OK
- 10.1137/110835098 is OK
- 10.1007/s10107-004-0559-y is OK
MISSING DOIs
- None
INVALID DOIs
- None
Software report:
github.com/AlDanial/cloc v 1.90 T=0.02 s (1292.5 files/s, 113404.6 lines/s)
-------------------------------------------------------------------------------
Language files blank comment code
-------------------------------------------------------------------------------
Julia 14 262 168 1395
Markdown 7 143 0 421
YAML 6 9 4 141
TeX 1 8 0 118
TOML 3 5 0 46
-------------------------------------------------------------------------------
SUM: 31 427 172 2121
-------------------------------------------------------------------------------
Commit count by author:
61 JuliusMartensen
48 Christoph Plate
Paper file info:
📄 Wordcount for paper.md
is 1253
✅ The paper includes a Statement of need
section
License info:
✅ License found: MIT License
(Valid open source OSI approved license)
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
Had an issue on installation: https://github.com/mathopt/DynamicOED.jl/issues/28#issue-2269343131
Resolved problem was on my end. Looks good from my perspective. One minor note that I would add how to add while not in the registry: ] add https://github.com/mathopt/DynamicOED.jl
and keep it even when in the registry just as a helpful redundancy
Hi @KBodolai, @imciner2 how is your review going?
Hi @KBodolai, @imciner2 how is your review going?
Hey @diehlpk . Thanks for your efforts!
If needed, I asked two people I know of if they could jump in for a review:
Let me know if I can provide any more help on this.
Hi @diehlpk , apologies for the delay, finally moving forward with it!
Overall I'm quite happy with the submission, the text in the paper reads well and the software looks solid!
A couple of small things that I noticed while reviewing, preventing me from ticking the last two boxes:
paper/figures/lotka.pdf
as a png and replacing it. (I'd do a pull request with it, but it's probably best to export it directly as a png to preserve image quality)Thanks for the review and the approval!
- I can't seem to reproduce the examples other than the example in the home section, more info in MethodError in examples. mathopt/DynamicOED.jl#30.
- (A bit more of a nitpick) As for the community guidelines tickbox, I've not seen any guidelines in the documentation referring to contributions. (issues and support can easily be reported through the gh issues). @AlCap23 , could it be possible to add a contributing.md file, or anything in the docs clarifying this?
- There seems to be an error rendering the figure in the paper. I'm not entirely sure, but it may be because it's a pdf. I did a quick test with a screenshot and it renders well saved as a png, so I'd suggest exporting
paper/figures/lotka.pdf
as a png and replacing it. (I'd do a pull request with it, but it's probably best to export it directly as a png to preserve image quality)
Should be handled via https://github.com/mathopt/DynamicOED.jl/pull/31 Instead of a seperate contribution guide, we choose to follow the ColPrac given by SciML ( as we are already closely related to this part of Julias Ecosystem ).
fantastic, thanks for checking so quickly! I've verified that the examples are now working and provide the expected results.
Good to hear about the separate contribution guide, it could be a good idea to mention it in the docs though (thinking of people that may be newer to the Julia ecosystem).
Hi @imciner2 the other two reviewers finished their reviews. Do you think you can finish your review in the next two weeks? If not I will go ahead with the other two reviews.
Hi @KBodolai and @joshuaeh. Thanks for finishing your reviews.
Hi @imciner2 can you please estimate when you will finish your review?
@editorialbot remove @imciner2 as reviewer
@imciner2 removed from the reviewers list!
@editorialbot check references
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.1137/141000671 is OK
- 10.5334/jors.151 is OK
- 10.5281/zenodo.7738525 is OK
- 10.11588/heidok.00002980 is OK
- 10.1007/978-3-319-93031-2_27 is OK
- 10.1016/S0168-9274(99)00020-3 is OK
- 10.48550/arXiv.2103.05244 is OK
- 10.1137/110835098 is OK
- 10.1007/s10107-004-0559-y is OK
MISSING DOIs
- None
INVALID DOIs
- None
@editorialbot commands
Hello @diehlpk, here are the things you can ask me to do:
# List all available commands
@editorialbot commands
# Add to this issue's reviewers list
@editorialbot add @username as reviewer
# Remove from this issue's reviewers list
@editorialbot remove @username from reviewers
# Get a list of all editors's GitHub handles
@editorialbot list editors
# Assign a user as the editor of this submission
@editorialbot assign @username as editor
# Remove the editor assigned to this submission
@editorialbot remove editor
# Remind an author, a reviewer or the editor to return to a review after a
# certain period of time (supported units days and weeks)
@editorialbot remind @reviewer in 2 weeks
# Adds a checklist for the reviewer using this command
@editorialbot generate my checklist
# Set a value for version
@editorialbot set v1.0.0 as version
# Set a value for branch
@editorialbot set joss-paper as branch
# Set a value for repository
@editorialbot set https://github.com/organization/repo as repository
# Set a value for the archive DOI
@editorialbot set 10.5281/zenodo.6861996 as archive
# Mention the EiCs for the correct track
@editorialbot ping track-eic
# Run checks and provide information on the repository and the paper file
@editorialbot check repository
# Check the references of the paper for missing DOIs
@editorialbot check references
# Generates the pdf paper
@editorialbot generate pdf
# Recommends the submission for acceptance
@editorialbot recommend-accept
# Generates a LaTeX preprint file
@editorialbot generate preprint
# Flag submission with questionable scope
@editorialbot query scope
# Get a link to the complete list of reviewers
@editorialbot list reviewers
# Creates a post-review checklist with editor and authors tasks
@editorialbot create post-review checklist
# Open the review issue
@editorialbot start review
@editorialbot set <DOI here> as archive
@editorialbot set <version here> as version
@editorialbot generate pdf
@editorialbot check references
and ask author(s) to update as needed@editorialbot recommend-accept
@editorialbot generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
Hi @AlCap23 I took a look at the paper as editor and here are my comments:
Time
is it seconds or milliseconds?@AlCap23 if you could address these comments, we could proceed with the paper.
@AlCap23 if you could address these comments, we could proceed with the paper.
@diehlpk I've added the space. Unfortunately, time has no unit in given that it is from our viewpoint just a "scaling" factor ( see also the original publication ), so I would just leave it blank or add days
if strictly necessary.
@diehlpk We archived the version v0.1.2 on Zenodo and are good to go 👍 License, Orcid IDs and authors check out.
One minor comment based on my copy-pasta mistake : There is a 1
in the title of the paper hidden, would you be so kind to remove it? Thanks!
@editorialbot set v0.1.2 as version
Done! version is now v0.1.2
@AlCap23 can you please post the DOI for Zenodo?
@editorialbot generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@AlCap23 One minor comment based on my copy-pasta mistake : There is a 1 in the title of the paper hidden, would you be so kind to remove it? Thanks!
Not sure where this is happening. I think you should be able to fix that too.
@AlCap23 can you please post the DOI for Zenodo?
@AlCap23
One minor comment based on my copy-pasta mistake : There is a 1 in the title of the paper hidden, would you be so kind to remove it? Thanks!
Not sure where this is happening. I think you should be able to fix that too.
I've tried, but not found a way ( maybe it is irrelevant ). This happened during the initial submission
@editorialbot set 10.5281/zenodo.12084055 as archive
Done! archive is now 10.5281/zenodo.12084055
@AlCap23 The title is wrong on Zendodo. Can you please change the tittle to match with the paper?
@AlCap23 if you log in to Zenodo, you should be able to edit the title and other meta data. See the screenshot below with one from my projects. I do not have access to your project to change that.
I've changed the title to
DynamicOED.jl: A Julia package for solving optimum1 experimental design problems
without the typo :) . Does this work for you @diehlpk ?
@editorialbot check references
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.1137/141000671 is OK
- 10.5334/jors.151 is OK
- 10.5281/zenodo.7738525 is OK
- 10.11588/heidok.00002980 is OK
- 10.1007/978-3-319-93031-2_27 is OK
- 10.1016/S0168-9274(99)00020-3 is OK
- 10.48550/arXiv.2103.05244 is OK
- 10.1137/110835098 is OK
- 10.1007/s10107-004-0559-y is OK
MISSING DOIs
- None
INVALID DOIs
- None
@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/141000671 is OK
- 10.5334/jors.151 is OK
- 10.5281/zenodo.7738525 is OK
- 10.11588/heidok.00002980 is OK
- 10.1007/978-3-319-93031-2_27 is OK
- 10.1016/S0168-9274(99)00020-3 is OK
- 10.48550/arXiv.2103.05244 is OK
- 10.1137/110835098 is OK
- 10.1007/s10107-004-0559-y is OK
MISSING DOIs
- None
INVALID DOIs
- None
Submitting author: !--author-handle-->@AlCap23<!--end-author-handle-- (Julius Martensen) Repository: https://github.com/mathopt/DynamicOED.jl Branch with paper.md (empty if default branch): Version: v0.1.2 Editor: !--editor-->@diehlpk<!--end-editor-- Reviewers: @KBodolai, @joshuaeh Archive: 10.5281/zenodo.12084055
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
@KBodolai & @imciner2 & @joshuaeh, 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 @diehlpk 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 @joshuaeh
📝 Checklist for @KBodolai