Skip to content

[PRE REVIEW]: A MATLAB-based Instrument Control (MIC) package for fluorescence imaging #7121

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

Closed
editorialbot opened this issue Aug 20, 2024 · 33 comments
Assignees
Labels
C++ C Matlab pre-review Track: 2 (BCM) Biomedical Engineering, Biosciences, Chemistry, and Materials

Comments

@editorialbot
Copy link
Collaborator

editorialbot commented Aug 20, 2024

Submitting author: @sajjad88 (Sajjad Khan)
Repository: https://github.com/LidkeLab/matlab-instrument-control
Branch with paper.md (empty if default branch):
Version: v0.1.0
Editor: @adamltyson
Reviewers: @bencardoen, @raacampbell
Managing EiC: Kevin M. Moerman

Status

status

Status badge code:

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

Author instructions

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

@sajjad88 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 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.31 s (1303.3 files/s, 183511.4 lines/s)
---------------------------------------------------------------------------------------
Language                             files          blank        comment           code
---------------------------------------------------------------------------------------
MATLAB                                 140           3471           6950          17329
XML                                     88              0              0          14990
C++                                     90           1049            856           3823
Markdown                                70            657              0           2489
C/C++ Header                             7            436            324           1599
Python                                   7            414            416           1574
Visual Studio Solution                   4              4              4           1006
TeX                                      1             16              0            179
Windows Module Definition                1              0              0              2
JSON                                     1              0              0              1
---------------------------------------------------------------------------------------
SUM:                                   409           6047           8550          42992
---------------------------------------------------------------------------------------

Commit count by author:

   580	dschodt
    91	Hanieh
    70	Sandeep
    48	sajjad88
    28	ellyse-taylor
    18	Michael Wester
    12	HMFarsibaf
    12	kiwibogo
    11	Keith Lidke
     8	Sandeep Pallikkuth
     8	TIRF
     7	Keith A. Lidke
     5	MohamadFazel
     3	kalidke
     2	Hanieh Mazloom Farsibaf
     2	Sajjad Khan
     2	Sheng Liu
     1	Ali
     1	Gert-Jan Bakker
     1	MJWester

@editorialbot
Copy link
Collaborator Author

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

✅ OK DOIs

- 10.1364/OPEX.13.007052 is OK
- 10.1126/science.1127344 is OK
- 10.1038/nmeth929 is OK
- 10.1126/science.1137395 is OK
- 10.1021/acs.chemrev.6b00815 is OK
- 10.1016/j.bpj.2017.11.2912 is OK
- 10.21105/joss.05563 is OK
- 10.1371/journal.pone.0203291 is OK
- 10.1038/s41467-022-34894-2 is OK
- 10.1002/0471142727.mb1420s92 is OK

🟡 SKIP DOIs

- No DOI given, and none found for title: The PYthon Microscopy Environment

❌ MISSING DOIs

- None

❌ INVALID DOIs

- https://doi.org/10.1016/j.celrep.2023.113603 is INVALID because of 'https://doi.org/' prefix

@editorialbot
Copy link
Collaborator Author

Paper file info:

📄 Wordcount for paper.md is 1124

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

ImSwitch: Generalizing microscope control in Python
Submitting author: @kasasxav
Handling editor: @Kevin-Mattheus-Moerman (Active)
Reviewers: @uellue, @beniroquai, @untzag
Similarity score: 0.7264

Madym: A C++ toolkit for quantitative DCE-MRI analysis
Submitting author: @michaelberks
Handling editor: @emdupre (Active)
Reviewers: @agahkarakuzu, @matteomancini
Similarity score: 0.7060

MAHOS: Measurement Automation Handling and Orchestration System
Submitting author: @ktahar
Handling editor: @arfon (Active)
Reviewers: @sidihamady, @aquilesC
Similarity score: 0.6887

qMRLab: Quantitative MRI analysis, under one umbrella
Submitting author: @agahkarakuzu
Handling editor: @Kevin-Mattheus-Moerman (Active)
Reviewers: @grlee77, @mfroeling, @62442katieb
Similarity score: 0.6872

LiberTEM: Software platform for scalable multidimensional data processing in transmission electron microscopy
Submitting author: @uellue
Handling editor: @majensen (Active)
Reviewers: @alvarolopez, @fedorov
Similarity score: 0.6814

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

Kevin-Mattheus-Moerman commented Aug 20, 2024

@sajjad88, 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/).
  • Does the project feature automated testing? If so I recommend adding links to this to your README.
  • Does the project have dedicated documentation? I recommend adding links to this to your README.
  • Can you please confirm if you feel the above code report (showing 17329 lines of MATLAB) is accurate? If so, can you clarify what proportion is truly your original contribution in terms of code? I ask this as MATLAB submission can sometimes feature large amounts of automatically generated code (e.g. for APP/GUI creation).

@sajjad88
Copy link

@editorialbot commands

@editorialbot
Copy link
Collaborator Author

Hello @sajjad88, here are the things you can ask me to do:


# List all available commands
@editorialbot commands

# Get a list of all editors's GitHub handles
@editorialbot list editors

# Adds a checklist for the reviewer using this command
@editorialbot generate my checklist

# Set a value for branch
@editorialbot set joss-paper as branch

# Run checks and provide information on the repository and the paper file
@editorialbot check repository

# Check the references of the paper for missing DOIs
@editorialbot check references

# Generates the pdf paper
@editorialbot generate pdf

# Generates a LaTeX preprint file
@editorialbot generate preprint

# Get a link to the complete list of reviewers
@editorialbot list reviewers

@sajjad88
Copy link

Names of potential reviewers:

  1. Ben Cardoen (https://reviewers.joss.theoj.org/reviewers/2046).
  2. caldarolamartin (https://reviewers.joss.theoj.org/reviewers/781).

@sajjad88
Copy link

@Kevin-Mattheus-Moerman thank you for helping in the initial steps. We have fixed the DOI issues, added documentation and contribution guidelines in main README.md.

When changes are made to the matlab-instrument-class, continuous integration tests are performed on the abstract classes, which are further inherited by the other instruments classes but cannot be applied to any instance class unless the corresponding instrument is actually hooked up.

There is no automatically generated code (e.g., .mlapp) included, such as code from MATLAB App Designer or other GUI creation tools. All of the MATLAB code, including any functions related to user interfaces or graphical components, has been manually written, ensuring that it reflects original contributions and not auto-generated templates or GUI elements. The 17,329 lines of MATLAB code in the report are entirely the result of manual development.

With these changes we created a new release: matlab-instrument-control v0.1.1.

@Kevin-Mattheus-Moerman
Copy link
Member

@editorialbot generate pdf

@Kevin-Mattheus-Moerman
Copy link
Member

@editorialbot check references

@editorialbot
Copy link
Collaborator Author

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

✅ OK DOIs

- 10.1364/OPEX.13.007052 is OK
- 10.1126/science.1127344 is OK
- 10.1038/nmeth929 is OK
- 10.1126/science.1137395 is OK
- 10.1021/acs.chemrev.6b00815 is OK
- 10.1016/j.bpj.2017.11.2912 is OK
- 10.21105/joss.05563 is OK
- 10.1371/journal.pone.0203291 is OK
- 10.1016/j.celrep.2023.113603 is OK
- 10.1038/s41467-022-34894-2 is OK
- 10.1002/0471142727.mb1420s92 is OK
- 10.5281/zenodo.4289803 is OK

🟡 SKIP DOIs

- None

❌ MISSING DOIs

- None

❌ INVALID DOIs

- None

@editorialbot
Copy link
Collaborator Author

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

@Kevin-Mattheus-Moerman
Copy link
Member

@editorialbot invite @adamltyson as editor

@editorialbot
Copy link
Collaborator Author

Invitation to edit this submission sent!

@editorialbot
Copy link
Collaborator Author

Five most similar historical JOSS papers:

ImSwitch: Generalizing microscope control in Python
Submitting author: @kasasxav
Handling editor: @Kevin-Mattheus-Moerman (Active)
Reviewers: @uellue, @beniroquai, @untzag
Similarity score: 0.7215

Madym: A C++ toolkit for quantitative DCE-MRI analysis
Submitting author: @michaelberks
Handling editor: @emdupre (Active)
Reviewers: @agahkarakuzu, @matteomancini
Similarity score: 0.7113

MAHOS: Measurement Automation Handling and Orchestration System
Submitting author: @ktahar
Handling editor: @arfon (Active)
Reviewers: @sidihamady, @aquilesC
Similarity score: 0.6886

qMRLab: Quantitative MRI analysis, under one umbrella
Submitting author: @agahkarakuzu
Handling editor: @Kevin-Mattheus-Moerman (Active)
Reviewers: @grlee77, @mfroeling, @62442katieb
Similarity score: 0.6884

LiberTEM: Software platform for scalable multidimensional data processing in transmission electron microscopy
Submitting author: @uellue
Handling editor: @majensen (Active)
Reviewers: @alvarolopez, @fedorov
Similarity score: 0.6854

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

@adamltyson would you be able to help edit this one? It may be a tricky one in terms finding reviewers able to test the software that links to/needs particular hardware/instruments.

@adamltyson
Copy link
Member

Hi @Kevin-Mattheus-Moerman, yes happy to take this on. I'll see what I can do.

@adamltyson
Copy link
Member

@editorialbot assign @adamltyson as editor

@editorialbot
Copy link
Collaborator Author

Assigned! @adamltyson is now the editor

@adamltyson
Copy link
Member

@bencardoen, @caldarolamartin, would you be able to review this submission to JOSS?

If you haven't reviewed for JOSS before, we carry out our checklist-driven reviews here in GitHub issues following these guidelines. If you have any questions, let me know.

@bencardoen
Copy link

@adamltyson yes, I'd be happy to

@caldarolamartin
Copy link

caldarolamartin commented Sep 20, 2024 via email

@adamltyson
Copy link
Member

Thanks @bencardoen, and no problem at all @caldarolamartin.

@raacampbell has agreed offline to review.

@adamltyson
Copy link
Member

@editorialbot add @bencardoen as reviewer

@editorialbot
Copy link
Collaborator Author

@bencardoen added to the reviewers list!

@adamltyson
Copy link
Member

@editorialbot add @raacampbell as reviewer

@editorialbot
Copy link
Collaborator Author

@raacampbell added to the reviewers list!

@adamltyson
Copy link
Member

@editorialbot start review

@editorialbot
Copy link
Collaborator Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C++ C Matlab pre-review Track: 2 (BCM) Biomedical Engineering, Biosciences, Chemistry, and Materials
Projects
None yet
Development

No branches or pull requests

6 participants