Closed editorialbot closed 1 month 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.1016/j.cageo.2022.105270 is OK
- 10.1016/j.cageo.2015.09.015 is OK
- 10.1111/j.1365-246x.1997.tb05663.x is OK
- 10.1023/A:1015012821040 is OK
- 10.1111/rssa.12315 is OK
- 10.1190/geo2018-0679.1 is OK
- 10.1093/gji/ggz197 is OK
- 10.2481/dsj.ss_igy-009 is OK
- 10.3389/feart.2020.00296 is OK
MISSING DOIs
- No DOI given, and none found for title: BIRRP: Bounded influence, remote reference process...
- No DOI given, and none found for title: Continuous integration: improving software quality...
- No DOI given, and none found for title: Mod3DMT and EMTF: Free Software for MT Data Proces...
- No DOI given, and none found for title: Resistics: A versatile native Python 3 package for...
- No DOI given, and none found for title: The Magnetotelluric Method
- No DOI given, and none found for title: MTHotel
INVALID DOIs
- None
Software report:
github.com/AlDanial/cloc v 1.90 T=0.16 s (778.1 files/s, 278718.8 lines/s)
-------------------------------------------------------------------------------
Language files blank comment code
-------------------------------------------------------------------------------
Python 97 2503 5840 7867
XML 2 22 0 2440
Jupyter Notebook 11 0 24668 848
JSON 2 0 0 547
TeX 1 14 0 138
Markdown 1 44 0 103
reStructuredText 8 112 136 73
CSV 1 0 0 46
YAML 2 1 4 32
DOS Batch 1 8 1 26
make 1 4 7 9
-------------------------------------------------------------------------------
SUM: 127 2708 30656 12129
-------------------------------------------------------------------------------
Commit count by author:
1187 Karl N. Kappler
93 JP
13 kkappler
12 Lindsey Heagy
9 Joseph Capriotti
5 timronan
3 Laura Keyson
2 Karl Kappler
1 Seogi Kang
Paper file info:
π Wordcount for paper.md
is 1754
β
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:
Hi @blsqr, @sinanozaydin how is your review going?
@diehlpk Thank you for checking in. I continued my review today and opened two issues in the Aurora repository:
Hi @blsqr, @sinanozaydin how is your review going?
Hello @diehlpk, as i mentioned before i can only start reviewing this after 11 July since I'm at fieldwork right now until then.
Cheers,
Hi @blsqr, @sinanozaydin how is your review going?
Hello @diehlpk, as i mentioned before i can only start reviewing this after 11 July since I'm at fieldwork right now until then.
Cheers,
Sorry, I forgot about that.
@kkappler @diehlpk Sorry for not having continued with the review yet; I will do so next week.
@diehlpk I have opened an issue for my review:
@kkappler could you please have a look?
Hi @diehlpk ,
I have started a branch fix_issue_337, and have a Draft PR in place to address Aurora issue 337. Currently I am working on documentation (making each module start with "This module contains ... ", and adding a descriptive sentence at the top of each method).
This response is started in a google doc. I will tag @sinanozaydin once all the comments are addressed. Lets give this a week or two for next status update.
Hello @diehlpk. With the revisions made by @kkappler (link above in their comment), I updated my checklist.
Cheers,
Sinan
@editorialbot generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@sinanozaydin thanks for finishing your review.
@blsqr could you please have a look if all your concerns were addressed?
@diehlpk I have also concluded my checklist and support publication of the submitted software paper in JOSS.
The aurora package seems to address an important use case in magnetotelluric data processing, building on and extending the existing ecosystem. The submitted software paper gives a good overview of the main ideas and provides examples of how to use aurora
for data analysis, starting from common dataset formats; while I'm not from this domain and probably can't assess shortcomings, the approaches taken seem convenient and well-implemented. Furthermore, it seems like this project is under very active development; the planned co-development of aurora with key packages of the domain seems like a very good idea.
Thanks to @kkappler for quickly addressing the issues raised during this review. If there are any other questions, feel free to reach out.
@blsqr, @sinanozaydin Thanks for finishing your review.
@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
@kkappler if you are done with 346, please work on the Additional Author Tasks After Review is Complete above to proceed with the publication process.
Hi @diehlpk ,
Re issue 346, as blsqr states, that task is on hold until the publication process is complete
The "Additional Author Tasks" have been addressed, can you please take a look?
Double check authors and affiliations (including ORCIDs)
Make a release of the software with the latest changes from the review and post the version number here. This is the version that will be used in the JOSS paper.
joss_2024
Archive the release on Zenodo/figshare/etc and post the DOI here.
Make sure that the title and author list (including ORCIDs) in the archive match those in the JOSS paper.
Make sure that the license listed for the archive is the same as the software license.
@editorialbot add joss_2024 as version
I'm sorry human, I don't understand that. You can see what commands I support by typing:
@editorialbot commands
@editorialbot set joss_2024 as version
Done! version is now joss_2024
@editorialbot set 10.5281/zenodo.13334589 as archive
Done! archive is now 10.5281/zenodo.13334589
@editorialbot recommend-accept
Attempting dry run of processing paper acceptance...
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
β
OK DOIs
- 10.1016/j.cageo.2022.105270 is OK
- 10.1016/j.cageo.2015.09.015 is OK
- 10.1111/j.1365-246x.1997.tb05663.x is OK
- 10.1023/A:1015012821040 is OK
- 10.1111/rssa.12315 is OK
- 10.5334/jors.148 is OK
- 10.1190/geo2018-0679.1 is OK
- 10.1093/gji/ggz197 is OK
- 10.2481/dsj.ss_igy-009 is OK
- 10.3389/feart.2020.00296 is OK
- 10.1190/1.9781560802686.ch8 is OK
π‘ SKIP DOIs
- No DOI given, and none found for title: BIRRP: Bounded influence, remote reference process...
- No DOI given, and none found for title: Continuous integration: improving software quality...
- No DOI given, and none found for title: Mod3DMT and EMTF: Free Software for MT Data Proces...
- No DOI given, and none found for title: Resistics: A versatile native Python 3 package for...
- No DOI given, and none found for title: MTHotel
β MISSING DOIs
- None
β INVALID DOIs
- https://doi.org/10.1016/j.cageo.2022.105102 is INVALID because of 'https://doi.org/' prefix
@kkappler please fix the invalid DOI and I will run the command again.
:warning: Error preparing paper acceptance. The generated XML metadata file is invalid.
Element doi: [facet 'pattern'] The value 'https://doi.org/10.1016/j.cageo.2022.105102' is not accepted by the pattern '10\.[0-9]{4,9}/.{1,200}'.
@editorialbot check references
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
β
OK DOIs
- 10.1016/j.cageo.2022.105270 is OK
- 10.1016/j.cageo.2015.09.015 is OK
- 10.1111/j.1365-246x.1997.tb05663.x is OK
- 10.1023/A:1015012821040 is OK
- 10.1111/rssa.12315 is OK
- 10.5334/jors.148 is OK
- 10.1190/geo2018-0679.1 is OK
- 10.1093/gji/ggz197 is OK
- 10.2481/dsj.ss_igy-009 is OK
- 10.3389/feart.2020.00296 is OK
- 10.1190/1.9781560802686.ch8 is OK
π‘ SKIP DOIs
- No DOI given, and none found for title: BIRRP: Bounded influence, remote reference process...
- No DOI given, and none found for title: Continuous integration: improving software quality...
- No DOI given, and none found for title: Mod3DMT and EMTF: Free Software for MT Data Proces...
- No DOI given, and none found for title: Resistics: A versatile native Python 3 package for...
- No DOI given, and none found for title: MTHotel
β MISSING DOIs
- None
β INVALID DOIs
- https://doi.org/10.1016/j.cageo.2022.105102 is INVALID because of 'https://doi.org/' prefix
@editorialbot check references
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
β
OK DOIs
- 10.1016/j.cageo.2022.105270 is OK
- 10.1016/j.cageo.2015.09.015 is OK
- 10.1111/j.1365-246x.1997.tb05663.x is OK
- 10.1023/A:1015012821040 is OK
- 10.1111/rssa.12315 is OK
- 10.5334/jors.148 is OK
- 10.1190/geo2018-0679.1 is OK
- 10.1016/j.cageo.2022.105102 is OK
- 10.1093/gji/ggz197 is OK
- 10.2481/dsj.ss_igy-009 is OK
- 10.3389/feart.2020.00296 is OK
- 10.1190/1.9781560802686.ch8 is OK
π‘ SKIP DOIs
- No DOI given, and none found for title: BIRRP: Bounded influence, remote reference process...
- No DOI given, and none found for title: Continuous integration: improving software quality...
- No DOI given, and none found for title: Mod3DMT and EMTF: Free Software for MT Data Proces...
- No DOI given, and none found for title: Resistics: A versatile native Python 3 package for...
- No DOI given, and none found for title: MTHotel
β MISSING DOIs
- None
β INVALID DOIs
- None
@diehlpk The doi is fixed.
@editorialbot check references
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
β
OK DOIs
- 10.1016/j.cageo.2022.105270 is OK
- 10.1016/j.cageo.2015.09.015 is OK
- 10.1111/j.1365-246x.1997.tb05663.x is OK
- 10.1023/A:1015012821040 is OK
- 10.1111/rssa.12315 is OK
- 10.5334/jors.148 is OK
- 10.1190/geo2018-0679.1 is OK
- 10.1016/j.cageo.2022.105102 is OK
- 10.1093/gji/ggz197 is OK
- 10.2481/dsj.ss_igy-009 is OK
- 10.3389/feart.2020.00296 is OK
- 10.1190/1.9781560802686.ch8 is OK
π‘ SKIP DOIs
- No DOI given, and none found for title: BIRRP: Bounded influence, remote reference process...
- No DOI given, and none found for title: Continuous integration: improving software quality...
- No DOI given, and none found for title: Mod3DMT and EMTF: Free Software for MT Data Proces...
- No DOI given, and none found for title: Resistics: A versatile native Python 3 package for...
- No DOI given, and none found for title: MTHotel
β 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.1016/j.cageo.2022.105270 is OK
- 10.1016/j.cageo.2015.09.015 is OK
- 10.1111/j.1365-246x.1997.tb05663.x is OK
- 10.1023/A:1015012821040 is OK
- 10.1111/rssa.12315 is OK
- 10.5334/jors.148 is OK
- 10.1190/geo2018-0679.1 is OK
- 10.1016/j.cageo.2022.105102 is OK
- 10.1093/gji/ggz197 is OK
- 10.2481/dsj.ss_igy-009 is OK
- 10.3389/feart.2020.00296 is OK
- 10.1190/1.9781560802686.ch8 is OK
π‘ SKIP DOIs
- No DOI given, and none found for title: BIRRP: Bounded influence, remote reference process...
- No DOI given, and none found for title: Continuous integration: improving software quality...
- No DOI given, and none found for title: Mod3DMT and EMTF: Free Software for MT Data Proces...
- No DOI given, and none found for title: Resistics: A versatile native Python 3 package for...
- No DOI given, and none found for title: MTHotel
β MISSING DOIs
- None
β INVALID DOIs
- None
Submitting author: !--author-handle-->@kkappler<!--end-author-handle-- (karl kappler) Repository: https://github.com/simpeg/aurora Branch with paper.md (empty if default branch): joss Version: joss_2024 Editor: !--editor-->@diehlpk<!--end-editor-- Reviewers: @blsqr, @sinanozaydin Archive: 10.5281/zenodo.13334589
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
@blsqr & @sinanozaydin, 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 @sinanozaydin
π Checklist for @blsqr