Open editorialbot opened 3 weeks 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.03 s (806.1 files/s, 185859.1 lines/s)
-------------------------------------------------------------------------------
Language files blank comment code
-------------------------------------------------------------------------------
Python 9 309 331 1243
Jupyter Notebook 2 0 1705 477
XML 1 0 0 417
Markdown 4 50 0 123
TeX 1 8 0 72
YAML 2 0 1 62
make 1 9 1 32
TOML 1 0 0 2
-------------------------------------------------------------------------------
SUM: 21 376 2038 2428
-------------------------------------------------------------------------------
Commit count by author:
5 Jacob
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.1109/TEMC.2017.2692962 is OK
- 10.1109/TASL.2008.919073 is OK
- 10.1109/TSG.2018.2883795 is OK
MISSING DOIs
- No DOI given, and none found for title: Black - The Uncompromising Code Formatter
- No DOI given, and none found for title: Flake8 - Your Tool For Style Guide Enforcement
- No DOI given, and none found for title: pre-commit - A framework for managing and maintain...
- No DOI given, and none found for title: NILMTK: An open source toolkit for non-intrusive l...
- No DOI given, and none found for title: Large Scale Radio Frequency Signal Classification
- No DOI given, and none found for title: source-separation-AHS
INVALID DOIs
- None
Paper file info:
π Wordcount for paper.md
is 1247
β
The paper includes a Statement of need
section
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:
@kellyrowland do you know if the github preview for joss is broken? I've seen it failing fairly consistency recently with the error
Error rendering embedded code invalid pdf
hmm, for viewing the article proof on github?
I'm seeing a rendered PDF at https://github.com/openjournals/joss-papers/blob/joss.06866/joss.06866/10.21105.joss.06866.pdf , but please let me know if you're referring to something else.
yeah, I'm referring to the article preview, clicking your link I get
I'm on chrome 114.0.5735.198
gotcha, thanks. I haven't seen that as a known or widespread issue, so you might try the usual clearing cookies/cache or even updating Chrome (I'm using Chrome 125.0.6422.142 with the PDF showing up in-browser). less convenient, but the PDF should be viewable via download either way.
I couldn't confirm Contribution and authorship and I have created this issue on the source repo: https://github.com/ORNL/dasp-stacker/issues/1
@samiralavi thanks for opening that issue, it looks like it's been resolved. just a ping here on the review status, please let me know if you need to set things down.
likewise @lucaferranti - thanks for getting started on your review, please let me know if you need to set things down.
Thanks @kellyrowland for the reminder. I can see that the issue is resolved and I continued with the rest of the review. @jcupsmith I have created the following issues to be addressed based on the review criteria mentioned above:
https://github.com/ORNL/dasp-stacker/issues/2 https://github.com/ORNL/dasp-stacker/issues/3 https://github.com/ORNL/dasp-stacker/issues/4
I am happy to recommend the paper after these issues are resolved.
Submitting author: !--author-handle-->@jcupsmith<!--end-author-handle-- (Jacob Smith) Repository: https://github.com/ORNL/dasp-stacker Branch with paper.md (empty if default branch): Version: v0.1.1 Editor: !--editor-->@kellyrowland<!--end-editor-- Reviewers: @samiralavi, @lucaferranti 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
@samiralavi & @lucaferranti, 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 @kellyrowland 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 @lucaferranti
π Checklist for @samiralavi