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]: SPECTPSFToolbox: A Python Toolbox for SPECT Point Spread Function Modeling #7082

Closed
editorialbot opened this issue Aug 7, 2024 · 16 comments
Labels
pre-review Python query-scope Submissions of uncertain scope for JOSS rejected Shell TeX Track: 2 (BCM) Biomedical Engineering, Biosciences, Chemistry, and Materials

Comments

@editorialbot
Copy link
Collaborator

Submitting author: @lukepolson (Lucas Polson)
Repository: https://github.com/PyTomography/SPECTPSFToolbox
Branch with paper.md (empty if default branch):
Version: v0.1.0
Editor: Pending
Reviewers: Pending
Managing EiC: Kevin M. Moerman

Status

status

Status badge code:

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

Author instructions

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

@lukepolson 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 7, 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

- 10.1109/42.363108 is OK
- 10.1109/42.52985 is OK
- 10.1109/TNS.2002.998681 is OK
- 10.1109/NSSMIC.2006.354345 is OK
- 10.1088/1361-6560/aa6911 is OK
- 10.1088/1361-6560/aadac1 is OK
- 10.1109/TMI.2003.812251 is OK
- 10.1007/978-1-4614-7657-3_19 is OK
- 10.1109/NSS/MIC42677.2020.9507966 is OK
- 10.21037/atm-20-5988 is OK
- 10.1007/s00259-022-06092-1 is OK
- 10.1007/s00259-024-06681-2 is OK
- 10.1088/0031-9155/35/1/008 is OK
- 10.1088/0031-9155/43/4/021 is OK
- 10.1088/0031-9155/43/11/013 is OK
- 10.1088/0031-9155/25/6/003 is OK
- 10.1109/TNS.2002.1039547 is OK
- 10.1007/0-387-25444-7_1 is OK
- 10.2967/jnumed.124.267482 is OK

MISSING DOIs

- 10.1097/mnm.0000000000001675 may be a valid DOI for title: Anatomically guided reconstruction improves lesion...
- No DOI given, and none found for title: PyTomography: A Python Library for Quantitative Me...
- No DOI given, and none found for title: Foundations of Image Science
- No DOI given, and none found for title: Targeted α-therapy of metastatic castration-resist...
- No DOI given, and none found for title: Predictors of overall and disease-free survival in...
- No DOI given, and none found for title: First clinical results for PSMA-targeted α-therapy...
- No DOI given, and none found for title: Molecular imaging and biochemical response assessm...

INVALID DOIs

- https://doi.org/10.1016/0169-2607(89)90111-9 is INVALID because of 'https://doi.org/' prefix

@editorialbot
Copy link
Collaborator Author

Software report:

github.com/AlDanial/cloc v 1.90  T=0.03 s (1038.8 files/s, 209643.2 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
Jupyter Notebook                10              0           3043            861
Python                           7            196            518            856
TeX                              1             28              0            298
Markdown                         3             35              0             82
YAML                             2              5              0             44
TOML                             1              8              0             41
DOS Batch                        1              8              1             26
reStructuredText                 2             43            106             15
Bourne Shell                     2              3              2             14
make                             1              4              7              9
CSS                              1              0              0              3
-------------------------------------------------------------------------------
SUM:                            31            330           3677           2249
-------------------------------------------------------------------------------

Commit count by author:

    16	lukepolson
     1	Luke Polson

@editorialbot
Copy link
Collaborator Author

Paper file info:

📄 Wordcount for paper.md is 1874

✅ 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:

PyQMRI: An accelerated Python based Quantitative MRI toolbox
Submitting author: @maieroli2010
Handling editor: @Kevin-Mattheus-Moerman (Active)
Reviewers: @grlee77, @agahkarakuzu, @DARSakthi
Similarity score: 0.7218

elsa: an elegant framework for tomographic reconstruction
Submitting author: @ner0-m
Handling editor: @jbytecode (Active)
Reviewers: @uellue, @DanNixon
Similarity score: 0.7132

PiSCAT: A Python Package for Interferometric Scattering Microscopy
Submitting author: @po60nani
Handling editor: @emdupre (Active)
Reviewers: @ziatdinovmax, @aquilesC
Similarity score: 0.7049

Pylinac: Image analysis for routine quality assurance in radiotherapy
Submitting author: @jrkerns
Handling editor: @kellyrowland (Active)
Reviewers: @ProfLeao, @SimonBiggs
Similarity score: 0.7009

PWSpy: A Python library dedicated to the analysis of Partial Wave Spectroscopic Microscopy data.
Submitting author: @nanthony21
Handling editor: @jgostick (Active)
Reviewers: @pr4deepr, @jgostick
Similarity score: 0.6986

⚠️ 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 query scope

@editorialbot
Copy link
Collaborator Author

Submission flagged for editorial review.

@editorialbot editorialbot added the query-scope Submissions of uncertain scope for JOSS label Aug 8, 2024
@Kevin-Mattheus-Moerman
Copy link
Member

@lukepolson Dear author, thanks for this re-submission (original pre-review here: #7067). I see some of the issues raised were addressed. I present a summary below.

Note also I have just added the query-scope label, to trigger a scope review by the editorial board. This is because of the relatively small size of the project. During this scope review we'll determine if the functionality offered is extensive enough and if the project conforms to our substantial scholarly effort criteria. The scope review should take about 2 weeks to complete. In the meantime, if you can continue to look at/work on the below that would be good.

Points raised at previous PRE-REVIEW:

  • The submission is of a relatively small size (~717 lines of code appear to capture core functionality)

Still remains

  • The contribution history appears to show less than 3 months of work. Although work could have occurred off line, it is typically a sign of low maturity as evident also by the lack of forks/issues/stars/follows and community engagement with the work.

Partially addressed by explaining:

"while the repository is relatively new, it was built using prior code from the PyTomography (https://github.com/PyTomography/PyTomography) library, and has been in development offline for some time. The repository was made separate from PyTomography because we believe it forms an independent function in modeling of gamma camera resolution. "

  • Your project appears to lack contributing guidelines at the moment. Please work to add these and to mention/link to them in your README. You could for instance create a CONTRIBUTING.md file (see here for some examples: https://contributing.md/example/).

Partially addressed, a basic file was added but is not linked to or mentioned in the README

  • The project appears to lack dedicated documentation and automated testing (or at least the README does not link to them)

Potentially addressed assuming documentation covers all functionality.

README somewhat improved. The README is still very short and not like the example READMEs in that it does not link to contributing guidelines, does not mention/link to testing, does not mention/link to license, e.g. it should be more like this one: https://github.com/paucablop/chemotools.

Note that the above python projects also show examples of automated testing/continuous integration.

Potentially/partially addressed since the authors state they've:
"add[ed] a tests/ folder to the repository with some automated testing"

I have not reviewed your paper in detail but I can add the following about the paper:

  • Please study the above paper compilation error, it looks like you need to fix the ORCID id for one author. Check our documentation on this too if needed.

Looks like this was addressed.

  • Please study the above reference check ☝️ and see if you can address any of the reported potential DOI issues

Not completely addressed. Please study the above reference check ☝️ and see if you can address any of the reported potential DOI issues. You can add/amend DOI entries in your .bib file, and call @editorialbot check references here to check them again.

@lukepolson
Copy link

@editorialbot check references

@editorialbot
Copy link
Collaborator Author

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

OK DOIs

- 10.1109/42.363108 is OK
- 10.1109/42.52985 is OK
- 10.1016/0169-2607(89)90111-9 is OK
- 10.1109/TNS.2002.998681 is OK
- 10.1109/NSSMIC.2006.354345 is OK
- 10.1088/1361-6560/aa6911 is OK
- 10.1097/MNM.0000000000001675 is OK
- 10.1088/1361-6560/aadac1 is OK
- 10.1109/TMI.2003.812251 is OK
- 10.1007/978-1-4614-7657-3_19 is OK
- 10.48550/arXiv.2309.01977 is OK
- 10.1109/NSS/MIC42677.2020.9507966 is OK
- 10.21037/atm-20-5988 is OK
- 10.1117/1.1905634 is OK
- 10.2967/jnumed.117.203539 is OK
- 10.2967/jnumed.119.229229 is OK
- 10.2967/jnumed.120.251017 is OK
- 10.7150/thno.56211 is OK
- 10.1007/s00259-022-06092-1 is OK
- 10.1007/s00259-024-06681-2 is OK
- 10.1088/0031-9155/35/1/008 is OK
- 10.1088/0031-9155/43/4/021 is OK
- 10.1088/0031-9155/43/11/013 is OK
- 10.1088/0031-9155/25/6/003 is OK
- 10.1109/TNS.2002.1039547 is OK
- 10.1007/0-387-25444-7_1 is OK
- 10.2967/jnumed.124.267482 is OK

MISSING DOIs

- None

INVALID DOIs

- None

@lukepolson
Copy link

@Kevin-Mattheus-Moerman I recently pushed to the library to enhance the README file and make it more like the other packages you've linked. I've also fixed the DOI issues, as seen above. Thanks again for the in depth recommendations in improving the library!!

@Kevin-Mattheus-Moerman
Copy link
Member

@lukepolson thanks for this re-submission, and the efforts to improve this work prior to resubmission. Unfortunately, the second scope review by the JOSS editorial has once again concluded this work is out of scope. The small size and limited functionality of the package still cause this submission to not pass our substantial scholarly effort criterion, and the work is still deemed too immature at present.

We will now proceed to reject this submission.

One possible alternative to JOSS is to follow GitHub's guide on how to create a permanent archive and DOI for your software. This DOI can then be used by others to cite your work.

@Kevin-Mattheus-Moerman
Copy link
Member

@editorialbot reject

@editorialbot
Copy link
Collaborator Author

Paper rejected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pre-review Python query-scope Submissions of uncertain scope for JOSS rejected Shell TeX Track: 2 (BCM) Biomedical Engineering, Biosciences, Chemistry, and Materials
Projects
None yet
Development

No branches or pull requests

3 participants