openjournals / joss-reviews

Reviews for the Journal of Open Source Software
Creative Commons Zero v1.0 Universal
696 stars 36 forks source link

[PRE REVIEW]: Shapelets: A Python package implementing shapelet functions and their applications #5911

Closed editorialbot closed 8 months ago

editorialbot commented 9 months ago

Submitting author: !--author-handle-->@nasserma<!--end-author-handle-- (Nasser Mohieddin Abukhdeir) Repository: https://github.com/uw-comphys/shapelets Branch with paper.md (empty if default branch): publications Version: v0.1 Editor: !--editor-->@AoifeHughes<!--end-editor-- Reviewers: @tbmiller-astro, @Anshuman5 Managing EiC: Kyle Niemeyer

Status

status

Status badge code:

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

Author instructions

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

@nasserma 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 commented 9 months ago

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 commented 9 months ago
Software report:

github.com/AlDanial/cloc v 1.88  T=0.02 s (715.0 files/s, 73343.9 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
Markdown                         3            136              0            230
Python                           6            138            350            173
TeX                              1             16              0            166
YAML                             1              1              0             18
TOML                             1              0              0              3
-------------------------------------------------------------------------------
SUM:                            12            291            350            590
-------------------------------------------------------------------------------

gitinspector failed to run statistical information for the repository
editorialbot commented 9 months ago

Wordcount for paper.md is 2373

editorialbot commented 9 months ago
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

OK DOIs

- 10.1038/s41592-019-0686-2 is OK

MISSING DOIs

- 10.1111/j.1365-2966.2005.09453.x may be a valid DOI for title: Polar shapelets
- 10.1088/1361-6528/aaf353 may be a valid DOI for title: A generalized shapelet-based method for analysis of nanostructured surface imaging
- 10.1103/physreve.91.033307 may be a valid DOI for title: Theory and application of shapelets to the analysis of surface self-assembly imaging
- 10.1093/mnras/stz787 may be a valid DOI for title: Exponential shapelets: basis functions for data analysis of isolated features
- 10.1088/2053-1591/3/8/082001 may be a valid DOI for title: Computational characterization of ordered nanostructured surfaces
- 10.1364/opex.12.005760 may be a valid DOI for title: Optical coherence tomography of skin for measurement of epidermal thickness by shapelet-based image analysis
- 10.1007/s10827-008-0107-5 may be a valid DOI for title: Contextual modulation of V1 receptive fields depends on their spatial symmetry
- 10.1152/jn.00498.2005 may be a valid DOI for title: Responses of V1 neurons to two-dimensional hermite functions
- 10.1088/0004-637x/813/2/102 may be a valid DOI for title: Gravitational lens modeling with basis sets
- 10.1093/mnras/stu2611 may be a valid DOI for title: Generative pulsar timing analysis
- 10.1007/978-1-4612-2812-7_1 may be a valid DOI for title: Bond-Orientational Order

INVALID DOIs

- None
kyleniemeyer commented 9 months ago

@editorialbot set main as branch

editorialbot commented 9 months ago

Done! branch is now main

kyleniemeyer commented 9 months ago

@editorialbot check repository

editorialbot commented 9 months ago
Software report:

github.com/AlDanial/cloc v 1.88  T=0.03 s (797.7 files/s, 120257.5 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
Python                          15            485           1094            940
reStructuredText                 4            255             82            343
Markdown                         3            136              0            230
TeX                              1             16              0            166
YAML                             1              1              0             18
TOML                             1              0              0              3
-------------------------------------------------------------------------------
SUM:                            25            893           1176           1700
-------------------------------------------------------------------------------

gitinspector failed to run statistical information for the repository
editorialbot commented 9 months ago

Wordcount for paper.md is 2373

kyleniemeyer commented 9 months ago

@editorialbot set publications as branch

editorialbot commented 9 months ago

Done! branch is now publications

kyleniemeyer commented 9 months ago

@editorialbot invite @jgostick as editor

Hi @jgostick, could you edit this submission related to image analysis? Although it has applications in astronomy, it also seems relevant to materials science as well.

editorialbot commented 9 months ago

Invitation to edit this submission sent!

jgostick commented 9 months ago

The author of this package is my colleague and collaborator. It'll pretty awkward in the lunch room when I have to reject his submission!

kyleniemeyer commented 9 months ago

@jgostick ah, yes that's a clear conflict! I'll find someone else.

nasserma commented 9 months ago

The author of this package is my colleague and collaborator. It'll pretty awkward in the lunch room when I have to reject his submission!

Just for the record, lunch room conversations with @jgostick have always been awkward.

nasserma commented 9 months ago

Happy (Canadian) Thanksgiving! I just wanted to check-in on the status of assigning an editor and also point out a somewhat special situation regarding the development of the shapelets package.

The core package was developed internally and included as supplemental information for the research publication that introduced one of the application methods (self_assembly). This code was developed by R. Suderman (@rsuderman) and later modified by T. Akdeniz as part of another research publication; both are co-authors of this submission.

Myself, @mptino, and @abaacus have rewritten, enhanced, and added to the core package in order to improve its usability for researchers interested in the use of shapelet functions for their own purposes, but also included several past applications (astronomy and self_assembly). Thus the three of us have made all of the commits to the relatively new public repo, the past contributions of Rob and Thomas are in various internal repos and are not reflected in the public repo. However, all of the authors have made significant contributions to both the development of methods and the code itself.

kyleniemeyer commented 9 months ago

@nasserma understood, thanks for the explanation.

kyleniemeyer commented 9 months ago

@editorialbot invite @AoifeHughes as editor

Hi @AoifeHughes, could you edit this submission? It may fall outside your primary areas of expertise but involves image analysis. Thanks!

editorialbot commented 9 months ago

Invitation to edit this submission sent!

AoifeHughes commented 9 months ago

Hi @kyleniemeyer, sure I can edit

AoifeHughes commented 9 months ago

@editorialbot generate pdf

kyleniemeyer commented 9 months ago

@editorialbot assign @AoifeHughes as editor

editorialbot commented 9 months ago

Assigned! @AoifeHughes is now the editor

AoifeHughes commented 9 months ago

hmm, why didn't that work before?

AoifeHughes commented 9 months ago

@editorialbot generate pdf

xuanxu commented 9 months ago

The pdf compilation is failing because the metadata of the paper.md file includes incorrect values for some authors' orcid. Updating those XXXX-XXXX-XXXX-XXXX values or removing them should fix it.

danielskatz commented 9 months ago

@xuanxu - we used to get error messages in this case - can we get them back? (It will save you time too 🙂)

nasserma commented 9 months ago

I'll correct the paper.md file and add the missing DOIs to the references later today, sorry for the inconvenience.

xuanxu commented 9 months ago

@xuanxu - we used to get error messages in this case - can we get them back? (It will save you time too 🙂)

Yes, I'll take a look into it. I'm not sure why the messages are not automatically posted by editorialbot

nasserma commented 9 months ago

@AoifeHughes The invalid ORCIDs and missing DOIs have been corrected.

@xuanxu Also, note that the workflow we added to the repo per the JOSS instructions has always resulted in a PDF being created, so we were not aware that the invalid ORCIDs were a problem until now.

AoifeHughes commented 9 months ago

@editorialbot generate pdf

editorialbot commented 9 months ago

:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:

editorialbot commented 9 months ago

Five most similar historical JOSS papers:

Scarplet: A Python package for topographic template matching and diffusion dating Submitting author: @rmsare Handling editor: @kthyng (Active) Reviewers: @fclubb, @mcflugen Similarity score: 0.8342

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

SkyPy: A package for modelling the Universe Submitting author: @rrjbca Handling editor: @arfon (Active) Reviewers: @cescalara, @rmorgan10 Similarity score: 0.8192

beamshapes: a Python package to generate directivity patterns for various sound source models Submitting author: @thejasvibr Handling editor: @faroit (Active) Reviewers: @nils-werner, @hagenw Similarity score: 0.8121

SLEPLET: Slepian Scale-Discretised Wavelets in Python Submitting author: @paddyroddy Handling editor: @vissarion (Active) Reviewers: @Saran-nns, @klb2 Similarity score: 0.8100

⚠️ 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 before considering asking the reviewers of these papers to review again for JOSS.

AoifeHughes commented 8 months ago

Hi @nasserma , I'm just putting out some requests to reviewers and haven't gotten responses yet, please feel free to make some suggestions of potential reviewers from your own knowledge or by using https://reviewers.joss.theoj.org/ to help us proceed faster

nasserma commented 8 months ago

Hello @AoifeHughes , thanks for putting out requests to reviewers, here are a few from different areas (astrophysics, materials science, computational science) that have relevant background from the list of registered reviewers:

@tbmiller-astro , @rmorgan10 ; @vsangelidakis , @Anshuman5 ; @ziatdinovmax , @ml-evs

vsangelidakis commented 8 months ago

Hello, Unfortunately I cannot review this work due to limited available time. Good luck with your submission!

mptino commented 8 months ago

Hi @AoifeHughes - any luck with finding reviewers?

AoifeHughes commented 8 months ago

@editorialbot add @tbmiller-astro as reviewer

editorialbot commented 8 months ago

@tbmiller-astro added to the reviewers list!

AoifeHughes commented 8 months ago

@mptino Im trying, I sent out a dozen requests last week and we have one so far, if you have any other suggestions that would be helpful, otherwise I'll go through some more later myself.

nasserma commented 8 months ago

@tbmiller-astro , @rmorgan10 ; @vsangelidakis , @Anshuman5 ; @ziatdinovmax , @ml-evs

@AoifeHughes Thanks for your efforts. I just want to confirm that these suggested reviewers were all contacted before I look for more. Also, should we be limiting ourselves to the JOSS reviewers list for suggested reviewers, or also look outside of this list?

AoifeHughes commented 8 months ago

Yes, and those qualified to review but not on our list are acceptable

Anshuman5 commented 8 months ago

Hello, I would be able to review this work. Let me know the deadline for submitting the review.

AoifeHughes commented 8 months ago

@editorialbot add @Anshuman5 as reviewer

editorialbot commented 8 months ago

@Anshuman5 added to the reviewers list!

AoifeHughes commented 8 months ago

@editorialbot start review

editorialbot commented 8 months ago

OK, I've started the review over in https://github.com/openjournals/joss-reviews/issues/6058.