openjournals / joss-reviews

Reviews for the Journal of Open Source Software
Creative Commons Zero v1.0 Universal
714 stars 38 forks source link

[PRE REVIEW]: squallms: Squashing qualms with speedy quality assurance via lasso labeling for untargeted mass spectrometry data #6955

Closed editorialbot closed 2 months ago

editorialbot commented 3 months ago

Submitting author: !--author-handle-->@wkumler<!--end-author-handle-- (William Kumler) Repository: https://github.com/wkumler/squallms Branch with paper.md (empty if default branch): manuscript Version: 0.99.6 Editor: Pending Reviewers: Pending Managing EiC: Kevin M. Moerman

Status

status

Status badge code:

HTML: <a href="https://joss.theoj.org/papers/9cd8011375df0f02bfb2196fae2b098b"><img src="https://joss.theoj.org/papers/9cd8011375df0f02bfb2196fae2b098b/status.svg"></a>
Markdown: [![status](https://joss.theoj.org/papers/9cd8011375df0f02bfb2196fae2b098b/status.svg)](https://joss.theoj.org/papers/9cd8011375df0f02bfb2196fae2b098b)

Author instructions

Thanks for submitting your paper to JOSS @wkumler. Currently, there isn't a JOSS editor assigned to your paper.

@wkumler if you have any suggestions for potential reviewers then please mention them here in this thread (without tagging them with an @). You can search the list of people that have already agreed to review and may be suitable for this submission.

Editor instructions

The JOSS submission bot @editorialbot is here to help you find and assign reviewers and start the main review. To find out what @editorialbot can do for you type:

@editorialbot commands
editorialbot commented 3 months ago

Hello human, 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
editorialbot commented 3 months ago
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

OK DOIs

- 10.1016/j.jchromb.2019.04.009 is OK
- 10.1021/acs.analchem.1c02220 is OK
- 10.1021/acs.analchem.9b02983 is OK
- 10.32614/RJ-2022-050 is OK
- 10.1186/s12859-023-05533-4 is OK
- 10.1021/ac800692c is OK
- 10.3390/metabo12020137 is OK
- 10.1201/9780429447273 is OK
- 10.1021/ac051437y is OK

MISSING DOIs

- No DOI given, and none found for title: Shiny: Web Application Framework for R
- No DOI given, and none found for title: Squallms: Speedy Quality Assurance via Lasso Label...

INVALID DOIs

- None
editorialbot commented 3 months ago

Software report:

github.com/AlDanial/cloc v 1.90  T=0.01 s (1108.6 files/s, 166910.2 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
R                                9            105            489            835
YAML                             2             49             78            237
TeX                              1             10              0            147
Rmd                              1             80            113            135
Markdown                         3             41              0             90
-------------------------------------------------------------------------------
SUM:                            16            285            680           1444
-------------------------------------------------------------------------------

Commit count by author:

   131  wkumler
     2  William Kumler
     1  William
editorialbot commented 3 months ago

Paper file info:

📄 Wordcount for paper.md is 1013

✅ The paper includes a Statement of need section

editorialbot commented 3 months ago

License info:

🟡 License found: Other (Check here for OSI approval)

editorialbot commented 3 months ago

:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:

editorialbot commented 3 months ago

Five most similar historical JOSS papers:

proteoDA: a package for quantitative proteomics Submitting author: @ByrumLab Handling editor: @jmschrei (Active) Reviewers: @shahmoradi, @MohmedSoudy Similarity score: 0.7837

RIAssigner: A package for gas chromatographic retention index calculation Submitting author: @hechth Handling editor: @dhhagan (Retired) Reviewers: @yguitton, @davidegraff Similarity score: 0.7255

matchms - processing and similarity evaluation of mass spectrometry data. Submitting author: @florian-huber Handling editor: @arfon (Active) Reviewers: @bittremieux, @kaibioinfo Similarity score: 0.7231

Viime: Visualization and Integration of Metabolomics Experiments Submitting author: @waxlamp Handling editor: @lpantano (Active) Reviewers: @joannawolthuis, @rowlandm, @jkanche Similarity score: 0.7029

patRoon 2.0: Improved non-target analysis workflows including automated transformation product screening Submitting author: @rickhelmus Handling editor: @KristinaRiemer (Retired) Reviewers: @wesleyburr, @hechth Similarity score: 0.7026

⚠️ Note to editors: If these papers look like they might be a good match, click through to the review issue for that paper and invite one or more of the authors before considering asking the reviewers of these papers to review again for JOSS.

wkumler commented 3 months ago

Listing a few potential reviewers from the list and some thoughts about their relevant expertise:

  1. hechth: Previous experience reviewing metabolomics R packages in JOSS (patRoon)
  2. adafefe: Chromatography and metabolomics experience plus R
  3. joewandy: Metabolomics and R experience
  4. luciorq: Broader expertise with Shiny apps and R packages in bioinformatics
  5. salix-d: Broader expertise with Shiny apps and R packages in bioinformatics
Kevin-Mattheus-Moerman commented 2 months ago

@wkumler thanks for this submission. As AEiC I'm here to help with initial steps. Note that last week I assigned the query-scope label. This triggers a scope review by the editorial board. I've done this because of the relatively small size of the project. The scope review will determine if this work conforms to our substantial scholarly effort criteria, and will take about 2 weeks to complete.

Kevin-Mattheus-Moerman commented 2 months ago

@wkumler the editorial board has concluded the scope review for this submission. Unfortunately it is not deemed in scope as it stands. Besides the fact that the code base is rather small and the functionality relatively narrow, there were also concerns raised in terms of "maturity". It was recommended that the authors first push for acceptance in Bioconductor. This will likely require various improvements, such as a reduction of the number of dependencies, and also reduction in the use of global variables. We'd welcome a potential re-submission at such a later point, especially when accompanied by a potential expansion of the functionality.

We will now proceed to reject this submission. Note however, this rejection does not mean the work is not useful or not well put together, in simply means that for the above reasons it is not in scope for JOSS as it stands.

Kevin-Mattheus-Moerman commented 2 months ago

@editorialbot reject

editorialbot commented 2 months ago

Paper rejected.

wkumler commented 2 months ago

@Kevin-Mattheus-Moerman Thank you for the feedback, and thank you as well to the editorial board for considering the submission! I am interested in resubmitting at a future point and so could use some additional comments about what else JOSS is looking for in terms of maturity.

My current plan with this project is to wait to resubmit to JOSS until Bioconductor 3.20 releases (in October) with squallms included, during which I'll work on resolving some of the feature-requests I've got on the Github page. My colleagues and I are already using the package (and have seen interest from other institutions despite the relatively narrow scope) and the goal behind submitting it to JOSS was to avoid having to document the functions encapsulated here in the Methods sections, so perhaps I'll identify additional areas for improvement as we continue to use it. I can also expand the documentation and vignette selection to better emphasize how squallms interfaces with existing MS software such as Skyline, MSDIAL, and MzMine. Would it be better suited for JOSS if that's done?