Closed whedon closed 2 years ago
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @kasbohm, @ethanknights it looks like you're currently assigned to review this paper :tada:.
:warning: JOSS reduced service mode :warning:
Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post.
:star: Important :star:
If you haven't already, you should seriously consider unsubscribing from GitHub notifications for this (https://github.com/openjournals/joss-reviews) repository. As a reviewer, you're probably currently watching this repository which means for GitHub's default behaviour you will receive notifications (emails) for all reviews 😿
To fix this do the following two things:
For a list of things I can do to help you, just type:
@whedon commands
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
@whedon generate pdf
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.1016/j.media.2020.101759 is OK
- 10.1038/s41586-020-2649-2 is OK
- 10.12688/f1000research.29032.1 is OK
- 10.5281/zenodo.5043456 is OK
- 10.1016/j.zemedi.2018.11.002 is OK
MISSING DOIs
- None
INVALID DOIs
- None
Wordcount for paper.md
is 737
Software report (experimental):
github.com/AlDanial/cloc v 1.88 T=0.08 s (167.7 files/s, 39064.9 lines/s)
-------------------------------------------------------------------------------
Language files blank comment code
-------------------------------------------------------------------------------
Python 9 447 1099 1510
Markdown 2 32 0 74
TeX 1 0 0 70
YAML 1 2 3 19
TOML 1 0 0 6
-------------------------------------------------------------------------------
SUM: 14 481 1102 1679
-------------------------------------------------------------------------------
Statistical information for the repository '239126222a9a1a05aba9e912' was
gathered on 2021/08/10.
The following historical commit information, by author, was found:
Author Commits Insertions Deletions % of changes
Ravi Bandaru 31 6005 2952 100.00
Below are the number of rows from each author that have survived and are still
intact in the current revision:
Author Rows Stability Age % in comments
Ravi Bandaru 3056 50.9 0.2 3.99
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@ethanknights and @kasbohm thanks for agreeing to review this submission! The guidelines and checklist above should get you started, but please ask me in this thread if I can be of further help in the process.
:wave: @ethanknights, please update us on how your review is going (this is an automated reminder).
:wave: @kasbohm, please update us on how your review is going (this is an automated reminder).
@ethanknights and @kasbohm, how is it going with your reviews? If you have any particular issues or suggestions, please feel free to eleborate here or start an issue in the source repository.
Dear @ravibandaru-lab After reading through the paper once more, and testing the submitted software, I am not sure whether the software is within the scope of JOSS. I will therefore pause this review for now, and discuss with the other JOSS editors' whether the submission is within scope.
In particular, whereas the submitted manuscript describes that 'mednoise' contains several algorithms for analyzing medical imaging data, I have not been able to find any demonstrations of such algorithms in the source repository.
@kasbohm and @ethanknights, I pause this review for now, and will get back to you in due time.
Hello @osorensen!
Hopefully I can clear up some misconceptions/misunderstandings surrounding mednoise.
mednoise is not a tool to analyze medical imaging data. I've read over my paper once more to make sure that I didn't make any claims to mednoise's functionality as tool to analyze medical imaging data. mednoise is simply a tool to process medical images to improve the accuracy of ML and DL models (by removing noise/unneccesary features).
I did, however, make claims and demonstrations to mednoise's algorithms. At the time of submission to this journal, mednoise has three algorithms: hotspot, branch, and manual. The source code for these algorithms are present in my source directory as branch.py
, hotspot.py
, and manual.py
. I've linked them below, as well as their associated testcases below.
Manual Algorithm Source Code - https://github.com/mednoise/mednoise/blob/main/src/mednoise/manual.py
Hotspot Algorithm Source Code - https://github.com/mednoise/mednoise/blob/main/src/mednoise/hotspot.py
Branch Algorithm Source Code - https://github.com/mednoise/mednoise/blob/main/src/mednoise/branch.py
Manual Algorithm Testcases - https://github.com/mednoise/mednoise/blob/main/tests/tests_manual.py
Hotspot Algorithm Testcases - https://github.com/mednoise/mednoise/blob/main/tests/tests_hotspot.py
Branch Algorithm Testcases - https://github.com/mednoise/mednoise/blob/main/tests/tests_branch.py
I've also included a description of the functionality of these algorithms in my documentation here: https://mednoise.github.io/documentation/api_reference/explanation.html. You can also learn more about the algorithms by viewing the API usage guide for them here: https://mednoise.github.io/documentation/api_reference/usage.html
Finally, I've included a sample input and output of the branch algorithm in my paper, which hopefully shows exactly what mednoise does.
Let me know if there is anything else you would like clarification on. Hopefully we can resolve this issue soon.
Thanks for explaining @ravibandaru-lab. I will bring it on to the other editors, and you can expect to hear back from us in a week or two.
Hi @ravibandaru-lab! Thanks for your submission. Unfortunately, the editorial board has determined that this submission is not currently in scope primarily due to not passing the substantial scholarly effort bar and confusion around what exactly the package does. We do, however, encourage you to improve your work over time and consider submitting it again in the future after more development.
Here is some specific feedback:
@whedon reject
Paper rejected.
I just realized that I forgot to acknowledge our reviewers @kasbohm and @ethanknights! (Usually the scope checks happen before there are reviewers.) Thank you for being willing to review this submission and for your time.
Thanks @kthyng!
Submitting author: @ravibandaru-lab (Ravi Bandaru) Repository: https://github.com/mednoise/mednoise Version: v.1.2.4 Editor: @osorensen Reviewer: @kasbohm, @ethanknights Archive: Pending
:warning: JOSS reduced service mode :warning:
Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post.
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
@kasbohm & @ethanknights, please carry out your review in this issue by updating the checklist below. If you cannot edit the checklist please:
The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @osorensen 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 ✨
Review checklist for @kasbohm
✨ Important: Please do not use the Convert to issue functionality when working through this checklist, instead, please open any new issues associated with your review in the software repository associated with the submission. ✨
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @ethanknights
✨ Important: Please do not use the Convert to issue functionality when working through this checklist, instead, please open any new issues associated with your review in the software repository associated with the submission. ✨
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper