Open editorialbot opened 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
Software report:
github.com/AlDanial/cloc v 1.90 T=0.14 s (1023.5 files/s, 314809.1 lines/s)
-------------------------------------------------------------------------------
Language files blank comment code
-------------------------------------------------------------------------------
HTML 37 6489 182 18202
SVG 6 6 6 4476
Python 22 1051 1446 3640
Jupyter Notebook 14 0 3223 951
CSS 6 185 42 842
JavaScript 8 123 183 778
reStructuredText 34 485 943 276
Markdown 5 102 0 183
YAML 5 9 12 139
TeX 1 12 0 138
make 2 16 9 54
DOS Batch 1 8 1 26
Bourne Shell 1 8 10 20
JSON 1 0 0 8
TOML 1 1 0 8
-------------------------------------------------------------------------------
SUM: 144 8495 6057 29741
-------------------------------------------------------------------------------
Commit count by author:
136 Martindale, Nathan
2 Nathan Martindale
Paper file info:
š Wordcount for paper.md
is 1543
ā
The paper includes a Statement of need
section
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.1145/3185517 is OK
- 10.1145/3593013.3594087 is OK
- 10.1609/aimag.v35i4.2513 is OK
- 10.1109/TAMD.2010.2051030 is OK
- 10.1145/3241379 is OK
- 10.1145/604045.604056 is OK
MISSING DOIs
- No DOI given, and none found for title: Machine Teaching: A New Paradigm for Building Mach...
- No DOI given, and none found for title: Panel: The powerful data exploration & web app fra...
- No DOI given, and none found for title: ipyvuetify: Jupyter widgets based on vuetify UI co...
- No DOI given, and none found for title: Scikit-learn: Machine Learning in Python
INVALID DOIs
- None
License info:
ā
License found: BSD 3-Clause "New" or "Revised" 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 @WarmCyan! Would you please be able to address the missing DOIs in the references?
Hello @JBorrow! I've added DOIs for all except ipyvuetify, I couldn't find anything for that one
@SamHames thanks for agreeing to review this! Just so you're aware if you start playing with the software soon, there's a couple weird issues going on right now with dependencies that I'm still trying to resolve, if you run into problems during setup I'm tracking them over here: https://github.com/ORNL/icat/issues/44
Hi all, I am in contact with the authors. They have been actively working to fix some issues in the repository (broken package dependencies die to updates). I'll investigate the fixes and latest end of next week. I provide my checklist here then too. Thanks for your patience.
This review is still in process. Issues with the code repository still have to be analyzed and fixed, see https://github.com/ORNL/icat/issues/44.
Submitting author: !--author-handle-->@WarmCyan<!--end-author-handle-- (Nathan Martindale) Repository: https://github.com/ORNL/icat Branch with paper.md (empty if default branch): Version: v0.7.2 Editor: !--editor-->@JBorrow<!--end-editor-- Reviewers: @SamHames, @jhagerer Archive: Pending
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
@SamHames & @jhagerer, 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 @JBorrow 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 @SamHames