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]: binomial_cis: A Python Package for Optimal Binomial Confidence Intervals #6949

Closed
editorialbot opened this issue Jul 2, 2024 · 12 comments
Labels
pre-review query-scope Submissions of uncertain scope for JOSS rejected Track: 5 (DSAIS) Data Science, Artificial Intelligence, and Machine Learning

Comments

@editorialbot
Copy link
Collaborator

Submitting author: @Joe-Vincent (Joseph Vincent)
Repository: https://github.com/TRI-ML/binomial_cis
Branch with paper.md (empty if default branch):
Version: 0.0.11
Editor: Pending
Reviewers: Pending
Managing EiC: Chris Vernon

Status

status

Status badge code:

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

Author instructions

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

@Joe-Vincent 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 Jul 2, 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

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

OK DOIs

- None

MISSING DOIs

- No DOI given, and none found for title: \hrefhttps://search.library.berkeley.edu/permalink...
- No DOI given, and none found for title: \hrefhttps://link.springer.com/book/10.1007/978-3-...
- No DOI given, and none found for title: \hrefhttps://www.jstor.org/stable/2331986The Use o...
- No DOI given, and none found for title: \hrefhttps://arxiv.org/abs/2405.05439How Generaliz...
- 10.1136/gut.30.10.1439-a may be a valid DOI for title: Statistics with confidence: confidence intervals a...
- 10.1071/as10046 may be a valid DOI for title: On the estimation of confidence intervals for bino...

INVALID DOIs

- None

@editorialbot
Copy link
Collaborator Author

Software report:

github.com/AlDanial/cloc v 1.90  T=0.04 s (861.6 files/s, 327725.6 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
CSV                              2              0              0           2002
SVG                              1              0             19           1598
Python                           9            403            755            659
Jupyter Notebook                 5              0           7038            349
reStructuredText                 7            154             61            206
YAML                             4             15              1            130
Markdown                         5             67              0            126
TeX                              1             10              0             45
DOS Batch                        1              8              1             26
make                             1              4              7              9
-------------------------------------------------------------------------------
SUM:                            36            661           7882           5150
-------------------------------------------------------------------------------

Commit count by author:

    32	Joe Vincent
    12	Joe-Vincent
    10	Joe
     4	HarukiNishimura-TRI

@editorialbot
Copy link
Collaborator Author

Paper file info:

📄 Wordcount for paper.md is 477

✅ The paper includes a Statement of need section

@editorialbot
Copy link
Collaborator Author

License info:

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

@editorialbot
Copy link
Collaborator Author

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

@editorialbot
Copy link
Collaborator Author

Five most similar historical JOSS papers:

Hypothesize: Robust Statistics for Python
Submitting author: @Alcampopiano
Handling editor: @csoneson (Active)
Reviewers: @martinagvilas, @arbennett
Similarity score: 0.6971

A Python module to combine $p$ values arising from discrete tests.
Submitting author: @Wrzlprmft
Handling editor: @vissarion (Active)
Reviewers: @steppi, @mdhaber
Similarity score: 0.6939

Fast Resampling and Monte Carlo Methods in Python
Submitting author: @mdhaber
Handling editor: @jbytecode (Active)
Reviewers: @SaranjeetKaur, @kose-y
Similarity score: 0.6938

presize: An R-package for precision-based sample size calculation in clinical research
Submitting author: @aghaynes
Handling editor: @majensen (Active)
Reviewers: @amoeba, @TomKellyGenetics
Similarity score: 0.6903

BayesO: A Bayesian optimization framework in Python
Submitting author: @jungtaekkim
Handling editor: @drvinceknight (Active)
Reviewers: @salrm8, @thomaspinder
Similarity score: 0.6853

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

@crvernon
Copy link

crvernon commented Jul 2, 2024

@editorialbot query scope

👋 @Joe-Vincent - thanks for your submission to JOSS! I am going to place this one up for review with our larger editorial board to ensure that it meets our submission requirements for substantial scholarly effort. This process usually takes about 2 weeks so I'll provide feedback here ASAP.

@editorialbot
Copy link
Collaborator Author

Submission flagged for editorial review.

@editorialbot editorialbot added the query-scope Submissions of uncertain scope for JOSS label Jul 2, 2024
@crvernon
Copy link

@editorialbot reject

Thanks for your submission @Joe-Vincent. After further review with our editorial board, we must reject your submission at this time due to the following:

  • The scope of the package is too narrow based on the functionality it provides. Instead it could potentially be a feature addition into an existing package with aligned functionality.
  • Low lines of code count and shorter commit history to meet the JOSS substantial scholarly effort requirement.
  • We do not require novelty in JOSS, but for the author's benefit an editor pointed out that a claim in the paper "The binomial_cis package is the first open-source implementation of these optimal binomial confidence intervals." may need to be reconsidered due to an existing R package being present that may provide similar functionality via a function named binom.optim (see https://cran.r-project.org/package=binom).

Thanks for your interest in publishing in JOSS and we wish you the best with your research!

@editorialbot
Copy link
Collaborator Author

Paper rejected.

@Joe-Vincent
Copy link

Hi @crvernon, thank you for your time reviewing my submission and for your feedback.
I am disappointed in the decision, and will leave a few clarifications here in case others come across this thread,

The commit history starts in May 2024, but the package development started in July 2023.
The package was largely developed in a private Github Enterprise repository, then once ready to be public the code was uploaded to the current repository.
I assumed this to be common practice, and it may benefit the journal to consider software developed in this manner.

  • We do not require novelty in JOSS, but for the author's benefit an editor pointed out that a claim in the paper "The binomial_cis package is the first open-source implementation of these optimal binomial confidence intervals." may need to be reconsidered due to an existing R package being present that may provide similar functionality via a function named binom.optim (see https://cran.r-project.org/package=binom).

The binom.optim function in in the binom R package is quite different than what is provided in our binomial_cis package.
We are aware of this R package, referencing it's main function (binom.confint) in the Background page of our documentation, however, the claim in the paper is unaffected by this R package.
Although the binom.optim function in that package uses optimization to compute confidence intervals, the resulting intervals do not have the property of being uniformly most accurate, which is the standard optimality criterion for confidence intervals.
I plan to update our documentation to make note of this in order to avoid confusion about the binom.optim function in the future.

Thanks again for taking the time to review my submission.

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

No branches or pull requests

3 participants