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]: jax-smfsb: A python library for stochastic systems biology modelling and inference #7329

Closed
editorialbot opened this issue Oct 8, 2024 · 37 comments
Assignees
Labels
Makefile pre-review Python TeX Track: 5 (DSAIS) Data Science, Artificial Intelligence, and Machine Learning

Comments

@editorialbot
Copy link
Collaborator

editorialbot commented Oct 8, 2024

Submitting author: @darrenjw (Darren James Wilkinson)
Repository: https://github.com/darrenjw/jax-smfsb
Branch with paper.md (empty if default branch):
Version: v1.0.0
Editor: @jromanowska
Reviewers: @antoineruzette, @cthoyt
Managing EiC: Chris Vernon

Status

status

Status badge code:

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

Author instructions

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

@darrenjw 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: 5 (DSAIS) Data Science, Artificial Intelligence, and Machine Learning labels Oct 8, 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.03 s (1617.4 files/s, 120350.1 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
Python                          40            556           1140           1614
TeX                              1             23              0            128
Markdown                         2             29              0             69
make                             3             26             11             37
TOML                             1              4              0             29
reStructuredText                 3             25             42             29
YAML                             1              6             17             10
-------------------------------------------------------------------------------
SUM:                            51            669           1210           1916
-------------------------------------------------------------------------------

Commit count by author:

    89	Darren Wilkinson

@editorialbot
Copy link
Collaborator Author

Paper file info:

📄 Wordcount for paper.md is 682

✅ The paper includes a Statement of need section

@editorialbot
Copy link
Collaborator Author

License info:

✅ License found: Apache License 2.0 (Valid open source OSI approved license)

@editorialbot
Copy link
Collaborator Author

@editorialbot
Copy link
Collaborator Author

Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

✅ OK DOIs

- 10.1111/j.1467-9868.2009.00736.x is OK
- 10.1016/j.cpc.2022.108527 is OK
- 10.1093/bioinformatics/btn051 is OK
- 10.1098/rsfs.2011.0047 is OK
- 10.1007/s11222-011-9288-2 is OK
- 10.1038/s41586-020-2649-2 is OK
- 10.15252/msb.20199110 is OK
- 10.1088/1742-5468/ac3ae9 is OK
- 10.1038/s41592-019-0686-2 is OK
- 10.1098/rsif.2008.0172 is OK
- 10.1038/nrg2509 is OK

🟡 SKIP DOIs

- No DOI given, and none found for title: JAX: composable transformations of Python+NumPy pr...
- No DOI given, and none found for title: On Neural Differential Equations
- No DOI given, and none found for title: An overview of the Scala programming language
- No DOI given, and none found for title: Python 3 Reference Manual
- No DOI given, and none found for title: R: A Language and Environment for Statistical Comp...
- No DOI given, and none found for title: scala-smfsb
- No DOI given, and none found for title: python-smfsb

❌ MISSING DOIs

- 10.1201/9781351000918 may be a valid DOI for title: Stochastic Modelling for Systems Biology, third ed...
- 10.32614/cran.package.smfsb may be a valid DOI for title: smfsb: Stochastic Modelling for Systems Biology

❌ INVALID DOIs

- None

@crvernon
Copy link

crvernon commented Oct 8, 2024

@editorialbot invite @jromanowska as editor

👋 @jromanowska - are you available to take this one on?

@editorialbot
Copy link
Collaborator Author

Invitation to edit this submission sent!

@jromanowska
Copy link

@crvernon , I think I can take it :)

@jromanowska
Copy link

@editorialbot assign me as editor

@editorialbot
Copy link
Collaborator Author

Assigned! @jromanowska is now the editor

@darrenjw
Copy link

Searching the review database (for Python and Systems Biology), it seems like Manuel Razo-Mejia and Griffin Chure would be appropriate, along with Antoine Ruzette, Abhishek Tiwari, Yafei Wang, and many others.

@jromanowska
Copy link

Thank you, @darrenjw ! I'm sorry, I didn't have time to look at it yet. I'll come back to you this week.

@jromanowska
Copy link

@editorialbot generate pdf

@editorialbot
Copy link
Collaborator Author

@jromanowska
Copy link

Before we proceed, I've created an issue in your repo because the paper.md does not compile, please fix that.
Also, I'd suggest to extend the paper just a bit by mentioning any software that has similar functionality to yours. You mention in the first paragraph of 'Statement of need' that there are some, perhaps it would be good to cite some of them that you think are worth mentioning.

@darrenjw
Copy link

@editorialbot generate pdf

@editorialbot
Copy link
Collaborator Author

@darrenjw
Copy link

@editorialbot generate pdf

@editorialbot
Copy link
Collaborator Author

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

@editorialbot
Copy link
Collaborator Author

Five most similar historical JOSS papers:

GPJax: A Gaussian Process Framework in JAX
Submitting author: @thomaspinder
Handling editor: @dfm (Active)
Reviewers: @gpleiss, @theorashid
Similarity score: 0.7394

PxMCMC: A Python package for proximal Markov Chain Monte Carlo
Submitting author: @auggiemarignier
Handling editor: @mstimberg (Active)
Reviewers: @jeremiecoullon, @mattpitkin
Similarity score: 0.7139

flowMC: Normalizing flow enhanced sampling package for probabilistic inference in JAX
Submitting author: @kazewong
Handling editor: @rkurchin (Active)
Reviewers: @matt-graham, @Daniel-Dodd
Similarity score: 0.7076

SiSyPHE: A Python package for the Simulation of Systems of interacting mean-field Particles with High Efficiency
Submitting author: @antoinediez
Handling editor: @pdebuyl (Active)
Reviewers: @lorenzo-rovigatti, @junghans
Similarity score: 0.7046

PyNumDiff: A Python package for numerical differentiation of noisy time-series data
Submitting author: @florisvb
Handling editor: @christinahedges (Retired)
Reviewers: @pmli, @billtubbs
Similarity score: 0.7040

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

@darrenjw
Copy link

Thanks. The build is fixed and I've added references to a couple of existing systems biology simulation libraries.

@jromanowska
Copy link

👋 @cthoyt, @benlansdell, @ortega2247 & @antoineruzette, would any of you be willing to review this submission for JOSS? We carry out our checklist-driven reviews here in GitHub issues and follow these guidelines: https://joss.readthedocs.io/en/latest/review_criteria.html

@cthoyt
Copy link

cthoyt commented Oct 25, 2024

I'd made some quick suggestions to @darrenjw to try and update the package to conform to Python community code standards (darrenjw/jax-smfsb#7, darrenjw/jax-smfsb#8) and to add some documentation for new users (darrenjw/jax-smfsb#9)

I will consider reviewing the code further and the manuscript if these are addressed

@jromanowska
Copy link

I'd made some quick suggestions to @darrenjw to try and update the package to conform to Python community code standards (darrenjw/jax-smfsb#7, darrenjw/jax-smfsb#8) and to add some documentation for new users (darrenjw/jax-smfsb#9)

I will consider reviewing the code further and the manuscript if these are addressed

Thank you so much for looking into that so fast! We'll wait for the author to address these issues. 👍

@benlansdell
Copy link

Hi @jromanowska , I'm happy to act as a reviewer, sure!

@jromanowska
Copy link

jromanowska commented Oct 29, 2024

Thank you, @benlansdell. As you can see from previous comments in this thread, we're currently waiting for the author to address some issues raised by @cthoyt.

@antoineruzette
Copy link

Hi @jromanowska, happy to help reviewing this repo. Cheers!

@jromanowska
Copy link

Thank you, @antoineruzette. We'll wait some days for the response from @darrenjw to issues raised by @cthoyt.

In the meantime, if you or @benlansdell have any other comments, please write them here. Note that this is not yet the official review thread, so you don't need to look into the code yet.

@jromanowska
Copy link

@darrenjw, I can see you're working on the code - great work! Let us know when you feel you've addressed the issues so that we can start the real review process. @cthoyt, please, keep your expectations low because the review has not yet started.

@darrenjw
Copy link

The suggestions from @cthoyt are all good - just time-consuming. I've had a reasonable go at addressing most of them, so I think it's probably fine to start the real review process now. Thanks,

@jromanowska
Copy link

@editorialbot add @antoineruzette as reviewer

@editorialbot
Copy link
Collaborator Author

@antoineruzette added to the reviewers list!

@jromanowska
Copy link

@editorialbot add @cthoyt as reviewer

@editorialbot
Copy link
Collaborator Author

@cthoyt added to the reviewers list!

@jromanowska
Copy link

@editorialbot start review

@editorialbot
Copy link
Collaborator Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Makefile pre-review Python TeX Track: 5 (DSAIS) Data Science, Artificial Intelligence, and Machine Learning
Projects
None yet
Development

No branches or pull requests

7 participants