openjournals / joss-reviews

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

[PRE REVIEW]: QhX: A Python package for periodicity detection in red noise #7516

Open editorialbot opened 1 day ago

editorialbot commented 1 day ago

Submitting author: !--author-handle-->@lionandjelka<!--end-author-handle-- (Andjelka Kovacevic) Repository: https://github.com/lionandjelka/QhX1 Branch with paper.md (empty if default branch): paper-submission Version: v0.2.0 Editor: Pending Reviewers: Pending Managing EiC: Warrick Ball

Status

status

Status badge code:

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

Author instructions

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

@lionandjelka 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 1 day 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 1 day ago
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

✅ OK DOIs

- 10.1051/0004-6361/201833304 is OK
- 10.1177/00037028241241308 is OK
- 10.48550/arXiv.2310.16896 is OK
- 10.48550/arXiv.1811.06542 is OK
- 10.3390/universe9060287 is OK
- 10.3847/1538-4365/ac88ce is OK
- 10.1093/mnras/stv1230 is OK
- 10.1086/309985 is OK
- 10.1093/mnras/sty3466 is OK
- 10.3847/1538-4357/ace31a is OK
- 10.1051/0004-6361/202450886 is OK

🟡 SKIP DOIs

- No DOI given, and none found for title: Time-frequency Analysis
- No DOI given, and none found for title: LSST: From Science Drivers to Reference Design and...
- No DOI given, and none found for title: Two-Dimensional Correlation Analysis of Periodicit...
- No DOI given, and none found for title: Fidgit: AGN DataChallenge

❌ MISSING DOIs

- 10.1086/170594 may be a valid DOI for title: 3C 345: Is the Variability of Quasars Nonlinear?
- 10.1093/mnras/sty1316 may be a valid DOI for title: Multifractality Signatures in Quasars Time Series ...
- 10.1038/159591a0 may be a valid DOI for title: Acoustical Quanta and the Theory of Hearing
- 10.3847/1538-3881/acb596 may be a valid DOI for title: Detecting Long-period Variability in the SDSS Stri...
- 10.1515/astro-2020-0007 may be a valid DOI for title: Two-Dimensional Correlation Analysis of Periodicit...
- 10.1093/mnras/staa737 may be a valid DOI for title: Confirmed Short Periodic Variability of Subparsec ...
- 10.3847/1538-4357/aaf731 may be a valid DOI for title: The Optical Variability of Supermassive Black Hole...
- 10.1093/mnras/stx3137 may be a valid DOI for title: Oscillatory Patterns in the Light curves of Five L...
- 10.1086/118137 may be a valid DOI for title: Wavelets for Period Analysis of Unevenly Sampled T...
- 10.1007/978-1-4612-2544-7_2 may be a valid DOI for title: Wavelets, Spectrum Analysis and 1/f Processes
- 10.1016/b978-0-12-811220-5.00002-2 may be a valid DOI for title: Advances in Two-Dimensional Correlation Spectrosco...

❌ INVALID DOIs

- None
editorialbot commented 1 day ago

Software report:

github.com/AlDanial/cloc v 1.90  T=4.01 s (51.9 files/s, 326946.2 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
CSV                             15              1              0        1105043
HTML                            68           3388            272          41331
SVG                              4              0              0           5360
Jupyter Notebook                22              0         142481           2392
Python                          35            774           1760           1832
CSS                              6            223             57           1015
JavaScript                      12            131            221            880
reStructuredText                35            489            528            554
TeX                              1             46              0            315
Markdown                         2             61              0            146
YAML                             3             11              0             59
TOML                             1              7              0             54
make                             2             16             18             43
DOS Batch                        1              8              1             26
JSON                             1              0              0             23
-------------------------------------------------------------------------------
SUM:                           208           5155         145338        1159073
-------------------------------------------------------------------------------

Commit count by author:

    65  lionandjelka
    34  OS Kernel
editorialbot commented 1 day ago

Paper file info:

📄 Wordcount for paper.md is 1421

✅ The paper includes a Statement of need section

editorialbot commented 1 day ago

License info:

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

editorialbot commented 1 day ago

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

editorialbot commented 1 day ago

Five most similar historical JOSS papers:

pySYD: Automated measurements of global asteroseismic parameters Submitting author: @ashleychontos Handling editor: @mbobra (Active) Reviewers: @danhey, @benjaminpope Similarity score: 0.7134

ClassiPyGRB: Machine Learning-Based Classification and Visualization of Gamma Ray Bursts using t-SNE Submitting author: @KenethGarcia Handling editor: @dfm (Active) Reviewers: @wkerzendorf, @dfm Similarity score: 0.7132

pygwb: a Python-based library for gravitational-wave background searches Submitting author: @a-renzini Handling editor: @plaplant (Active) Reviewers: @Sbozzolo, @cmbiwer Similarity score: 0.7094

Mayawaves: Python Library for Interacting with the Einstein Toolkit and the MAYA Catalog Submitting author: @deborahferguson Handling editor: @eloisabentivegna (Retired) Reviewers: @cjoana, @Sbozzolo Similarity score: 0.6998

CWInPy: A Python package for inference with continuous gravitational-wave signals from pulsars Submitting author: @mattpitkin Handling editor: @dfm (Active) Reviewers: @GregoryAshton, @ColmTalbot Similarity score: 0.6958

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

lionandjelka commented 17 hours ago

Thank you. Here are my suggestions for potential reviewers: astrogilda afeinstein20 sgeorge91 rmorgan10 jessie-dotson bliiir

warrickball commented 3 hours ago

Hi @lionandjelka, and thanks for your submission to JOSS. All our editors are currently busy so I'm going to place this on our current waiting list. In any case, the project you've submitted has a variety of issues that I've listed below that need to be addressed before we open this up for an editor to pick up, so I'm going to tag it as "paused" until the following points are addressed.

  1. The repo description says it's a "test repo of Qhx". Is that still true? I see you have several different Qhx repos, so it isn't clear this is the one you intended to submit.

  2. There are a lot of extra files in the repo, including at least

    • some files named 0, 1, 2 and 3,
    • a bunch of CSV files,
    • some Jupyter Notebooks (which might be part of the docs),
    • probably some profiling output in prof and
    • some build artefacts (e.g. Qhx.egg-info/ and build/).

    I don't think any of these should be in the repo and are only likely to confuse any potential reviewers.

  3. The Qhx documentation describes two versions of the same code. A final step in the JOSS publication process will be to tag a new release version, which is difficult to reconcile with having two versions. Regardless, just for ease-of-use, it makes more sense to have a single package with appropriate options—be that different functions or relevant arguments—to select between two different modes of execution.

  4. This is less important at the outset but the paper is also a bit long. The JOSS documentation says:

    The paper should be between 250-1000 words. Authors submitting papers significantly longer than 1000 words may be asked to reduce the length of their paper.

    The JOSS paper should be little more than a brief summary of the software, how it fills a current niche and how it would be (or has been) used in research. At the very least, your section describing the contents of the documentation page ("Documentation and Tutorials") can be removed.

Feel free to ask me any questions and keep me posted on your progress through these items.