openjournals / joss-reviews

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

[PRE REVIEW]: A Python package for Remote Sensing Data Analysis #5577

Closed editorialbot closed 1 year ago

editorialbot commented 1 year ago

Submitting author: !--author-handle-->@ytarazona<!--end-author-handle-- (Yonatan Tarazona Coronel) Repository: https://github.com/ytarazona/scikit-eo Branch with paper.md (empty if default branch): Version: v0.2.12 Editor: !--editor-->@pdebuyl<!--end-editor-- Reviewers: Pending Managing EiC: Kristen Thyng

Status

status

Status badge code:

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

Author instructions

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

@ytarazona 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 year 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 year ago
Software report:

github.com/AlDanial/cloc v 1.88  T=0.12 s (461.8 files/s, 53794.4 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
Python                          18           1063            826           1458
Markdown                        25           1051              0           1208
TeX                              1             13              0            173
YAML                             6             16             13            169
Jupyter Notebook                 4              0            485             36
HTML                             1              2              0              9
SVG                              1              0              0              1
-------------------------------------------------------------------------------
SUM:                            56           2145           1324           3054
-------------------------------------------------------------------------------

gitinspector failed to run statistical information for the repository
editorialbot commented 1 year ago
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

OK DOIs

- 10.1080/07038992.2021.1941823 is OK
- 10.3390/rs12183062 is OK
- 10.1080/10095020.2019.1710438 is OK
- 10.1038/nclimate1908 is OK
- 10.1038/s41559-022-01702-5 is OK
- 10.1016/J.RSE.2021.112577 is OK
- 10.5281/ZENODO.3233986 is OK
- 10.1023/A:1010933404324 is OK
- 10.1038/s41558-017-0049-x is OK
- 10.1038/s41586-021-03436-z is OK

MISSING DOIs

- None

INVALID DOIs

- https://doi.org/10.1016/j.rsase.2020.100337 is INVALID because of 'https://doi.org/' prefix
- https://doi.org/10.1016/j.ecolind.2018.07.012 is INVALID because of 'https://doi.org/' prefix
- https://doi.org/10.1016/j.rse.2014.02.015 is INVALID because of 'https://doi.org/' prefix
- https://doi.org/10.1016/j.isprsjprs.2021.04.015 is INVALID because of 'https://doi.org/' prefix
editorialbot commented 1 year ago

Wordcount for paper.md is 2766

editorialbot commented 1 year ago

Failed to discover a Statement of need section in paper

editorialbot commented 1 year ago

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

kthyng commented 1 year ago

@pdebuyl can you edit this submission?

kthyng commented 1 year ago

@editorialbot invite @pdebuyl as editor

editorialbot commented 1 year ago

Invitation to edit this submission sent!

kthyng commented 1 year ago

Hi @ytarazona and thanks for your submission! One quick note is that your paper is quite a bit too low. We try not to be too heavy handed about it, but 10 pages is much longer than we hope for. You can get more info here: https://joss.readthedocs.io/en/latest/submitting.html#what-should-my-paper-contain You could move the examples to the readme, for example, and set up some standard documentation like with ReadTheDocs.

ytarazona commented 1 year ago

Hi @kthyng, sorry for the delayed response. Thank you so much for the suggestion made. You can find the new version of the paper https://github.com/ytarazona/scikit-eo/blob/main/paper/paper.md. The examples were moved to the readme as you mentioned. Please, let me know if any additional changes are needed.

pdebuyl commented 1 year ago

@editorialbot assign me as editor

editorialbot commented 1 year ago

Assigned! @pdebuyl is now the editor

kthyng commented 1 year ago

@ytarazona I'm going to ping the greater editorial board to make sure this submission is in scope for JOSS — this process will take 1 to 2 weeks; thanks for your patience!

kthyng commented 1 year ago

@editorialbot query scope

editorialbot commented 1 year ago

Submission flagged for editorial review.

ytarazona commented 1 year ago

@kthyng, Thank you so much!. Suggested reviewers could be Necip Enes Gengeç or Simon Filhol. Both are in the remote sensing and image processing domain. Sincerely, Yonatan

danielskatz commented 1 year ago

@editorialbot generate pdf

danielskatz commented 1 year ago

@editorialbot check repository

editorialbot commented 1 year ago
Software report:

github.com/AlDanial/cloc v 1.88  T=0.11 s (519.9 files/s, 65987.7 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
Python                          18           1063            826           1458
Markdown                        26           1104              0           1264
YAML                             6             16             14            169
TeX                              1             12              0            164
Jupyter Notebook                 6              0           1283            104
HTML                             1              2              0              9
SVG                              1              0              0              1
-------------------------------------------------------------------------------
SUM:                            59           2197           2123           3169
-------------------------------------------------------------------------------

gitinspector failed to run statistical information for the repository
editorialbot commented 1 year ago

Wordcount for paper.md is 1742

editorialbot commented 1 year ago

Failed to discover a Statement of need section in paper

danielskatz commented 1 year ago

@editorialbot check references

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

OK DOIs

- 10.1080/07038992.2021.1941823 is OK
- 10.1016/j.rsase.2020.100337 is OK
- 10.1016/j.ecolind.2018.07.012 is OK
- 10.1016/j.rse.2014.02.015 is OK
- 10.1016/j.isprsjprs.2021.04.015 is OK
- 10.3390/rs12183062 is OK
- 10.1080/10095020.2019.1710438 is OK
- 10.1038/nclimate1908 is OK
- 10.1038/s41559-022-01702-5 is OK
- 10.1016/J.RSE.2021.112577 is OK
- 10.5281/ZENODO.3233986 is OK
- 10.1038/s41558-017-0049-x is OK
- 10.1038/s41586-021-03436-z is OK

MISSING DOIs

- None

INVALID DOIs

- None
editorialbot commented 1 year ago

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

kthyng commented 1 year ago

@ytarazona Thank you for your submission. Unfortunately the greater JOSS editorial board has determined it not to be in scope at this time because it would require too much back and forth with our reviewers to address. Before considering a resubmission, check our documentation and some published JOSS articles to prepare your package to a more robust, complete, and polished state. Some of the comments include:

kthyng commented 1 year ago

@editorialbot reject

editorialbot commented 1 year ago

Paper rejected.

ytarazona commented 1 year ago

@kthyng, Thank you so much for the review and the suggestions made!. We will be working on it and we are gonna resubmit again soon. Sincerely, Yonatan