Open editorialbot opened 3 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.48550/arXiv.1212.5701 is OK
- 10.48550/arXiv.1412.6980 is OK
- 10.48550/arXiv.1711.05101 is OK
- 10.1007/BF01589116 is OK
- 10.48550/arXiv.1908.03265 is OK
MISSING DOIs
- No DOI given, and none found for title: Adaptive Subgradient Methods for Online Learning a...
- No DOI given, and none found for title: On the Convergence of Adam and Beyond
- No DOI given, and none found for title: On the importance of initialization and momentum i...
- No DOI given, and none found for title: Incorporating Nesterov Momentum into Adam
- No DOI given, and none found for title: Neural Networks for Machine Learning
- No DOI given, and none found for title: Candle: a Minimalist ML Framework for Rust
INVALID DOIs
- None
Software report:
github.com/AlDanial/cloc v 1.90 T=0.03 s (1518.4 files/s, 355086.6 lines/s)
-------------------------------------------------------------------------------
Language files blank comment code
-------------------------------------------------------------------------------
Rust 27 701 1674 4819
Jupyter Notebook 1 0 1046 578
Markdown 5 79 0 147
TeX 1 12 0 132
YAML 3 9 0 82
TOML 2 11 1 48
HTML 1 0 0 15
-------------------------------------------------------------------------------
SUM: 40 812 2721 5821
-------------------------------------------------------------------------------
Commit count by author:
142 Kirpal Grewal
1 KGrewal1
Paper file info:
π Wordcount for paper.md
is 259
β
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:
@jatkinson1000 could you please generate your checklist, as explained above? Seeing it partially checked helps us to better understand the state of the review process. Thanks!
@relf and @jatkinson1000 thank you for starting your reviews - partially filled in checklists are very helpful to see where you're approximately with your reviews, and cross-references to issues in the software/paper repository are also useful. Do you think you will be able to complete your reviews soon?
To the best of my ability, I've completed my review of this library. I've installed and run the software test suite and the example successfully. The documentation is in place using standard Rust doc tool and contains algorithms pseudo-code and reference papers.
The paper itself should elaborate on the "far lower overhead for neural network training" claim which is not so obvious to me.
I would not call PyTorch or Tensorflow "full Python frameworks", as actually the training heavylifting is done using
compiled languages under the hood. It should also consolidate the position of the library with regard to the existing
Rust landscape at least by referencing current candle
optimizers and established optimizers crate like argmin
.
Hi All,
In summary, whilst the software repository looks good the paper needs more work and content. I would also like to see more information in the README, though see @relf has started an issue for this.
Submitting author: !--author-handle-->@KGrewal1<!--end-author-handle-- (Kirpal Grewal) Repository: https://github.com/KGrewal1/optimisers Branch with paper.md (empty if default branch): Version: v0.3.1 Editor: !--editor-->@olexandr-konovalov<!--end-editor-- Reviewers: @relf, @jatkinson1000 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
@relf & @jatkinson1000, 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 @olexandr-konovalov 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 @relf
π Checklist for @jatkinson1000