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]: MIRTorch: A Differentiable Medical Image Reconstruction Toolbox #7253

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

Comments

@editorialbot
Copy link
Collaborator

editorialbot commented Sep 20, 2024

Submitting author: @guanhuaw (Guanhua Wang)
Repository: https://github.com/guanhuaw/MIRTorch
Branch with paper.md (empty if default branch): feature/joss
Version: v0.1.2
Editor: @ymzayek
Reviewers: @paquiteau, @jonbmartin
Managing EiC: Kevin M. Moerman

Status

status

Status badge code:

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

Author instructions

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

@ GuanhuaW 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 Sep 20, 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.08 s (1306.8 files/s, 176821.0 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
Python                          44           1435           2496           6130
Jupyter Notebook                 6              0           2343            631
Markdown                         4            111              0            236
YAML                             6             27             18            187
reStructuredText                47            547            496            111
TeX                              1             10              0            104
TOML                             1              8              0             74
DOS Batch                        1              8              1             26
make                             1              4              7              9
-------------------------------------------------------------------------------
SUM:                           111           2150           5361           7508
-------------------------------------------------------------------------------

Commit count by author:

    39	m5520
    36	neelsh
    29	Guanhua
    29	guanhuaw
    18	Keyue Zhu
     5	quickstep
     4	SoniaMinseoKim
     3	Zongyu Li
     1	GuanhuaW
     1	Jeff Fessler
     1	nnmurthy
     1	ray@omni

@editorialbot
Copy link
Collaborator Author

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

✅ OK DOIs

- 10.1109/TMI.2018.2833635 is OK
- 10.1109/TMI.2018.2865356 is OK
- 10.1002/mrm.29645 is OK

🟡 SKIP DOIs

- No DOI given, and none found for title: Michigan Image Reconstruction Toolbox
- No DOI given, and none found for title: SigPy: a python package for high performance itera...
- No DOI given, and none found for title: The BART toolbox for computational magnetic resona...

❌ MISSING DOIs

- 10.1201/9781351069397-24 may be a valid DOI for title: Methods of conjugate gradients for solving linear ...
- 10.1137/080716542 may be a valid DOI for title: A fast iterative shrinkage-thresholding algorithm ...
- 10.1007/s10851-010-0251-1 may be a valid DOI for title: A first-order primal-dual algorithm for convex pro...
- 10.1561/9781601987174 may be a valid DOI for title: Proximal algorithms
- 10.1002/mrm.24389 may be a valid DOI for title: Gadgetron: an open source framework for medical im...

❌ INVALID DOIs

- None

@editorialbot
Copy link
Collaborator Author

Paper file info:

📄 Wordcount for paper.md is 717

✅ The paper includes a Statement of need section

@editorialbot
Copy link
Collaborator Author

License info:

🟡 License found: Other (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:

DIRECT: Deep Image REConstruction Toolkit
Submitting author: @georgeyiasemis
Handling editor: @osorensen (Active)
Reviewers: @estenhl, @sairamgeethanath
Similarity score: 0.7507

Radial Interstices Enable Speedy Low-volume Imaging
Submitting author: @spinicist
Handling editor: @emdupre (Active)
Reviewers: @uecker, @MartinK84
Similarity score: 0.7473

ivadomed: A Medical Imaging Deep Learning Toolbox
Submitting author: @charleygros
Handling editor: @cMadan (Retired)
Reviewers: @NMontanaBrown, @justusschock, @lbugnon
Similarity score: 0.7420

MRdataset : A unified and user-friendly interface to medical imaging datasets
Submitting author: @sinhaharsh
Handling editor: @mstimberg (Active)
Reviewers: @htwangtw, @djmannion
Similarity score: 0.7300

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

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

@guanhuaw Dear author, thanks for this submission. I am the AEiC on this track and here to help process the initial steps. Before we proceed, please can you have a look at the following points:

  • 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.
  • 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/).

@Kevin-Mattheus-Moerman Kevin-Mattheus-Moerman added the waitlisted Submissions in the JOSS backlog due to reduced service mode. label Sep 23, 2024
@Kevin-Mattheus-Moerman
Copy link
Member

@guanhuaw I have just waitlisted this submission since many of the editors in this domain are currently handling other submissions. I aim to assign an editor as soon as possible.

@Kevin-Mattheus-Moerman
Copy link
Member

@ymzayek I see you may be nearly done with one of you submissions. Do you think you could take on this one as editor once you are available?

@Kevin-Mattheus-Moerman
Copy link
Member

@editorialbot invite @ymzayek as editor

@editorialbot
Copy link
Collaborator Author

Invitation to edit this submission sent!

@ymzayek
Copy link

ymzayek commented Sep 26, 2024

Hi sorry for the late reply. Yes I'll take this

@Kevin-Mattheus-Moerman Kevin-Mattheus-Moerman removed the waitlisted Submissions in the JOSS backlog due to reduced service mode. label Sep 27, 2024
@Kevin-Mattheus-Moerman
Copy link
Member

@editorialbot assign @ymzayek as editor

@editorialbot
Copy link
Collaborator Author

Assigned! @ymzayek is now the editor

@ymzayek
Copy link

ymzayek commented Sep 28, 2024

@editorialbot generate pdf

@ymzayek
Copy link

ymzayek commented Sep 28, 2024

@editorialbot check references

@editorialbot
Copy link
Collaborator Author

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

✅ OK DOIs

- 10.1109/TMI.2018.2833635 is OK
- 10.1109/TMI.2018.2865356 is OK
- 10.1002/mrm.29645 is OK

🟡 SKIP DOIs

- No DOI given, and none found for title: Michigan Image Reconstruction Toolbox
- No DOI given, and none found for title: SigPy: a python package for high performance itera...
- No DOI given, and none found for title: The BART toolbox for computational magnetic resona...

❌ MISSING DOIs

- 10.1201/9781351069397-24 may be a valid DOI for title: Methods of conjugate gradients for solving linear ...
- 10.1137/080716542 may be a valid DOI for title: A fast iterative shrinkage-thresholding algorithm ...
- 10.1007/s10851-010-0251-1 may be a valid DOI for title: A first-order primal-dual algorithm for convex pro...
- 10.1561/9781601987174 may be a valid DOI for title: Proximal algorithms
- 10.1002/mrm.24389 may be a valid DOI for title: Gadgetron: an open source framework for medical im...

❌ INVALID DOIs

- None

@editorialbot
Copy link
Collaborator Author

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

@editorialbot
Copy link
Collaborator Author

Five most similar historical JOSS papers:

DIRECT: Deep Image REConstruction Toolkit
Submitting author: @georgeyiasemis
Handling editor: @osorensen (Active)
Reviewers: @estenhl, @sairamgeethanath
Similarity score: 0.7495

Radial Interstices Enable Speedy Low-volume Imaging
Submitting author: @spinicist
Handling editor: @emdupre (Active)
Reviewers: @uecker, @MartinK84
Similarity score: 0.7474

ivadomed: A Medical Imaging Deep Learning Toolbox
Submitting author: @charleygros
Handling editor: @cMadan (Retired)
Reviewers: @NMontanaBrown, @justusschock, @lbugnon
Similarity score: 0.7412

MRdataset : A unified and user-friendly interface to medical imaging datasets
Submitting author: @sinhaharsh
Handling editor: @mstimberg (Active)
Reviewers: @htwangtw, @djmannion
Similarity score: 0.7307

Scientific Computational Imaging Code (SCICO)
Submitting author: @bwohlberg
Handling editor: @danielskatz (Active)
Reviewers: @vitorsr, @DanNixon, @lucaferranti
Similarity score: 0.7276

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

@ymzayek
Copy link

ymzayek commented Sep 28, 2024

Hi @guanhuaw, I am the handling editor for this submission. Can you please check the reference summary for the skipped and missing DOIs in #7253 (comment) and update them or provide a reason for their omission?

Also let me know if you already have suggestions for reviewers. You can use the potential reviewers list provided at the top of this issue. You can also have a look at the authors in similar papers in JOSS #7253 (comment). If you have suggestions, please provide a list of potential reviewers here but do not use the @ character to avoid receiving unwanted notifications. Thank you in advance.

@guanhuaw
Copy link

Hi @ymzayek and @Kevin-Mattheus-Moerman,

Thank you both for your assistance with my submission. I apologize for not thoroughly checking the DOI and for directly using Zotero without careful review. We updated the paper.bib and added a contribution guideline as contribution.md. (in branch feature/joss)

Regarding potential reviewers, I believe individuals with expertise in inverse problems and medical imaging would be a good fit. From the list, the following may be suitable candidates: Pierre-Antoine Comby, Jonathan Bach Martin, Haykel Snoussi, Ajinkya Kadu, and sairamgeethanath.

Thanks again for your help!

Best,
Contributors of MIRTorch

@ymzayek
Copy link

ymzayek commented Oct 3, 2024

@editorialbot check references

@editorialbot
Copy link
Collaborator Author

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

✅ OK DOIs

- 10.1109/TMI.2018.2833635 is OK
- 10.1109/TMI.2018.2865356 is OK
- 10.1137/080716542 is OK
- 10.1007/s10851-010-0251-1 is OK
- 10.1561/2400000003 is OK
- 10.1002/mrm.29645 is OK
- 10.1002/mrm.24389 is OK

🟡 SKIP DOIs

- No DOI given, and none found for title: Michigan Image Reconstruction Toolbox
- No DOI given, and none found for title: SigPy: a python package for high performance itera...
- No DOI given, and none found for title: The BART toolbox for computational magnetic resona...

❌ MISSING DOIs

- None

❌ INVALID DOIs

- 10.6028/igres.049.044 is INVALID

@ymzayek
Copy link

ymzayek commented Oct 3, 2024

Hello @paquiteau, @jonbmartin and @H-Snoussi,

Would you be willing to assist in reviewing this submission for JOSS (Journal of Open Source Software)?

JOSS publishes articles about open source research software. The submission I'd like you to review is titled:

MIRTorch: A Differentiable Medical Image Reconstruction Toolbox

You can find more information at the top of this Github issue.

The review process at JOSS is unique: it takes place in a GitHub issue, is open, and author-reviewer-editor conversations are encouraged. If you have any questions please let me know.

This is the pre-review issue. After setting at least 2 reviewers we will start the review process in a separate thread. In that thread, there will be 23 check items for each single reviewer.

Thank you in advance!

@paquiteau
Copy link

Hello @ymzayek 1

Yes I can review it :)

@guanhuaw: shameless plug, but you can check out our work on Non-Cartesian MRI https://github.com/mind-inria/mri-nufft, that should a nice addition for your MRI linear operators (with performance improvements compared to torchkbnufft 2)

Footnotes

  1. (nice to see you here btw !)

  2. https://github.com/mind-inria/mri-nufft-benchmark

@jonbmartin
Copy link

jonbmartin commented Oct 3, 2024 via email

@H-Snoussi
Copy link

H-Snoussi commented Oct 3, 2024 via email

@ymzayek
Copy link

ymzayek commented Oct 9, 2024

@paquiteau thanks for accepting. Good to hear from you as well :)

@jonbmartin thanks for agreeing to review this and I'm glad it if you can find it useful.

I think we can get this started then. The review will start in a separate thread.

@guanhuaw you can follow the review in the new review issue that will be opened.

@ymzayek
Copy link

ymzayek commented Oct 9, 2024

@editorialbot start review

@editorialbot
Copy link
Collaborator Author

Can't start a review without reviewers

@ymzayek
Copy link

ymzayek commented Oct 9, 2024

@editorialbot add @paquiteau as reviewer

@editorialbot
Copy link
Collaborator Author

@paquiteau added to the reviewers list!

@ymzayek
Copy link

ymzayek commented Oct 9, 2024

@editorialbot add @jonbmartin as reviewer

@editorialbot
Copy link
Collaborator Author

@jonbmartin added to the reviewers list!

@ymzayek
Copy link

ymzayek commented Oct 9, 2024

@editorialbot start review

@editorialbot
Copy link
Collaborator Author

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

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

7 participants