Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PRE REVIEW]: PeakPerformance - A tool for Bayesian inference-based fitting of LC-MS/MS peaks #7141

Closed
editorialbot opened this issue Aug 26, 2024 · 40 comments
Assignees
Labels
pre-review Python TeX Track: 2 (BCM) Biomedical Engineering, Biosciences, Chemistry, and Materials

Comments

@editorialbot
Copy link
Collaborator

editorialbot commented Aug 26, 2024

Submitting author: @MicroPhen (Stephan Noack)
Repository: https://github.com/JuBiotech/peak-performance/
Branch with paper.md (empty if default branch): peak-performance-paper
Version: v0.7.0
Editor: @csoneson
Reviewers: @Adafede, @lazear
Managing EiC: Kevin M. Moerman

Status

status

Status badge code:

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

Author instructions

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

@MicroPhen 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 editorialbot added pre-review Track: 2 (BCM) Biomedical Engineering, Biosciences, Chemistry, and Materials labels Aug 26, 2024
@editorialbot
Copy link
Collaborator Author

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
Copy link
Collaborator Author

Software report:

github.com/AlDanial/cloc v 1.90  T=0.27 s (182.5 files/s, 476210.6 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
SVG                             10              1            128         108888
Python                          10            366           1162           2309
Markdown                         7            233              0            921
YAML                             9             25             30            224
TeX                              1             16              0            197
Jupyter Notebook                 5              0          12948            193
TOML                             1             10              1             49
DOS Batch                        1              8              1             26
reStructuredText                 4             25             32             22
make                             1              4              7              9
-------------------------------------------------------------------------------
SUM:                            49            688          14309         112838
-------------------------------------------------------------------------------

Commit count by author:

   343	j.niesser
    35	Michael Osthege
    32	Osthege, Michael
    25	Jochen Nießer
     5	dependabot[bot]

@editorialbot
Copy link
Collaborator Author

Paper file info:

📄 Wordcount for paper.md is 5787

✅ The paper includes a Statement of need section

@editorialbot
Copy link
Collaborator Author

License info:

🟡 License found: GNU Affero General Public License v3.0 (Check here for OSI approval)

@editorialbot
Copy link
Collaborator Author

👉📄 Download article proof 📄 View article proof on GitHub 📄 👈

@editorialbot
Copy link
Collaborator Author

Five most similar historical JOSS papers:

APAV: An Open-Source Python Package for Mass Spectrum Analysis in Atom Probe Tomography
Submitting author: @jdasm
Handling editor: @rkurchin (Active)
Reviewers: @ziatdinovmax, @mkuehbach
Similarity score: 0.6794

PyAutoFit: A Classy Probabilistic Programming Language for Model Composition and Fitting
Submitting author: @Jammy2211
Handling editor: @dfm (Active)
Reviewers: @arm61, @karllark
Similarity score: 0.6767

Fitspy: A Python package for spectral decomposition
Submitting author: @patquem
Handling editor: @phibeck (Active)
Reviewers: @maurov, @FCMeng
Similarity score: 0.6733

PCRedux: A Quantitative PCR Machine Learning Toolkit
Submitting author: @devSJR
Handling editor: @csoneson (Active)
Reviewers: @jaybee84, @markziemann
Similarity score: 0.6630

pocoMC: A Python package for accelerated Bayesian inference in astronomy and cosmology
Submitting author: @minaskar
Handling editor: @dfm (Active)
Reviewers: @kazewong, @marylou-gabrie
Similarity score: 0.6609

⚠️ 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.

@Kevin-Mattheus-Moerman
Copy link
Member

@editorialbot check references

1 similar comment
@Kevin-Mattheus-Moerman
Copy link
Member

@editorialbot check references

@Kevin-Mattheus-Moerman
Copy link
Member

@editorialbot invite @csoneson as editor

@editorialbot
Copy link
Collaborator Author

Invitation to edit this submission sent!

@csoneson
Copy link
Member

@editorialbot assign @csoneson as editor

@editorialbot
Copy link
Collaborator Author

Assigned! @csoneson is now the editor

@csoneson
Copy link
Member

👋🏻 @MicroPhen - I will handle your paper, and will start by looking for a couple of suitable reviewers. Before that, however, I would like to ask you to significantly cut down on the length of your paper - it's currently almost 6,000 words, and the target is 250-1,000 (see the JOSS guidelines for more details). Once this is done, please let me know here (you can generate a new proof by typing @editorialbot generate pdf in a comment in this issue). Thanks!

@Y0dler
Copy link

Y0dler commented Sep 24, 2024

Hello @csoneson, thanks for handling our paper.

Regarding the page number, I must admit we apparently overlooked that part. The problem is just that we realized during an earlier attempt to publish this content that a) most people in our primary target audience of biologists and chemists, who usually deal with chromatographic peak data, have very little to no experience with Bayesian statistics and b) we have to explain the model structure in order to set ourselves apart from other instances where Bayesian statistics were employed in one way or another to peak recognition or related topics. We will do our best to shorten the paper and shift some sections from the manuscript to the documentation but it would be very difficult to arrive at < 1000 words and still address the initially mentioned problems.
Is this an immovable upper limit or is it just the case that 6000 words are too far off from the target?

@csoneson
Copy link
Member

Hi @Y0dler - 6,000 words is definitely too far from the target, we can accept a bit over 1,000 words but I think the goal should be to get reasonably close. As mentioned in the guidelines, JOSS papers are not intended to be "regular" research papers, and longer descriptions of models etc may fit better in the documentation. Thanks!

@Y0dler
Copy link

Y0dler commented Sep 26, 2024

Hello again, @csoneson, I removed many sections and trimmed the remaining ones down to arrive at 1693 words (counting from the beginning of summary to the end of conclusion). Is this acceptable?

@csoneson
Copy link
Member

@editorialbot generate pdf

@csoneson
Copy link
Member

@editorialbot check repository

@editorialbot
Copy link
Collaborator Author

Software report:

github.com/AlDanial/cloc v 1.90  T=0.27 s (181.5 files/s, 472768.5 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
SVG                             10              1            128         108888
Python                          10            366           1162           2309
Markdown                         7            185              0            747
YAML                             9             25             30            224
TeX                              1             16              0            197
Jupyter Notebook                 5              0          12948            193
TOML                             1             10              1             49
DOS Batch                        1              8              1             26
reStructuredText                 4             25             32             22
make                             1              4              7              9
-------------------------------------------------------------------------------
SUM:                            49            640          14309         112664
-------------------------------------------------------------------------------

Commit count by author:

   343	j.niesser
    35	Michael Osthege
    32	Jochen Nießer
    32	Osthege, Michael
     5	dependabot[bot]

@editorialbot
Copy link
Collaborator Author

Paper file info:

📄 Wordcount for paper.md is 1955

✅ The paper includes a Statement of need section

@editorialbot
Copy link
Collaborator Author

License info:

🟡 License found: GNU Affero General Public License v3.0 (Check here for OSI approval)

@editorialbot
Copy link
Collaborator Author

👉📄 Download article proof 📄 View article proof on GitHub 📄 👈

@editorialbot
Copy link
Collaborator Author

Five most similar historical JOSS papers:

PyAutoFit: A Classy Probabilistic Programming Language for Model Composition and Fitting
Submitting author: @Jammy2211
Handling editor: @dfm (Active)
Reviewers: @arm61, @karllark
Similarity score: 0.7087

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

Fitspy: A Python package for spectral decomposition
Submitting author: @patquem
Handling editor: @phibeck (Active)
Reviewers: @maurov, @FCMeng
Similarity score: 0.7033

pocoMC: A Python package for accelerated Bayesian inference in astronomy and cosmology
Submitting author: @minaskar
Handling editor: @dfm (Active)
Reviewers: @kazewong, @marylou-gabrie
Similarity score: 0.6984

UltraNest - a robust, general purpose Bayesian inference engine
Submitting author: @JohannesBuchner
Handling editor: @fboehm (Active)
Reviewers: @mattpitkin, @ziatdinovmax
Similarity score: 0.6891

⚠️ 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.

@csoneson
Copy link
Member

Hi @Y0dler - yes, I think we can leave it like this for now. I will start by looking for a couple of suitable reviewers - if you have suggestions (e.g. from the list linked in the first post in this issue), feel free to let me know.

@Y0dler
Copy link

Y0dler commented Sep 27, 2024

Hello @csoneson, I'm glad to hear we're moving on for now :)

Regarding reviewer suggestions, we would put forward Prof. Aljoscha Wahl from the FAU Erlangen-Nürnberg and Virgile Andreani from Boston University.

@csoneson
Copy link
Member

👋🏻 @Armavica, @Adafede, @bittremieux - would you be interested in reviewing this submission for the Journal of Open Source Software (JOSS)?

PeakPerformance - A tool for Bayesian inference-based fitting of LC-MS/MS peaks
#7141

The checklist-based review is carried out on GitHub, more details can be found here.

Thanks in advance!

@bittremieux
Copy link

With a full teaching schedule and several conference travels in the next few weeks, unfortunately I can't dedicate the necessary time to this at the moment.

@Adafede
Copy link

Adafede commented Oct 1, 2024

Hi,
Similar to Wout, next weeks are rather busy. For when would you expect the first reviews?

@csoneson
Copy link
Member

csoneson commented Oct 1, 2024

@Adafede - we aim to have comments for the authors within 2-4 weeks after the review has started.

@Adafede
Copy link

Adafede commented Oct 1, 2024

Seems doable, you can count on me! 👍

@csoneson
Copy link
Member

csoneson commented Oct 1, 2024

Perfect, thanks @Adafede! I will assign you now, and open the actual review issue (where you will have your checklist etc) as soon as we have secured one more reviewer.

@csoneson
Copy link
Member

csoneson commented Oct 1, 2024

@editorialbot add @Adafede as reviewer

@editorialbot
Copy link
Collaborator Author

@Adafede added to the reviewers list!

@csoneson
Copy link
Member

csoneson commented Oct 3, 2024

👋🏻 @lazear, @hechth - would one of you be interested in reviewing this submission for JOSS?

PeakPerformance - A tool for Bayesian inference-based fitting of LC-MS/MS peaks
#7141

Thanks in advance!

@lazear
Copy link

lazear commented Oct 3, 2024

I am willing to review it

@csoneson
Copy link
Member

csoneson commented Oct 3, 2024

Great, thank you @lazear! I will assign you and open the review issue

@csoneson
Copy link
Member

csoneson commented Oct 3, 2024

@editorialbot add @lazear as reviewer

@editorialbot
Copy link
Collaborator Author

@lazear added to the reviewers list!

@csoneson
Copy link
Member

csoneson commented Oct 3, 2024

@editorialbot start review

@editorialbot
Copy link
Collaborator Author

OK, I've started the review over in #7313.

@github-staff github-staff deleted a comment Mar 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pre-review Python TeX Track: 2 (BCM) Biomedical Engineering, Biosciences, Chemistry, and Materials
Projects
None yet
Development

No branches or pull requests

9 participants
@csoneson @Kevin-Mattheus-Moerman @bittremieux @lazear @Adafede @editorialbot @Y0dler and others