Giter VIP home page Giter VIP logo

Comments (91)

klb2 avatar klb2 commented on August 20, 2024 3

@vissarion:
I have completed my review and after @paddyroddy has added the missing years to the references in the paper, I recommend the paper for acceptance.

from joss-reviews.

klb2 avatar klb2 commented on August 20, 2024 2

I currently have issues installing the required packages pyssht and pys2let on my machine. I have opened a bug report in their projects.

After I am able to resolve these issues, I will continue my review of the submitted sleplet package.

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024 2

Thanks for your hard work @danielskatz, @vissarion, @Saran-nns and @klb2!

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024 1

I've added argument/function level description of the API https://astro-informatics.github.io/sleplet/index.html

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024 1

Have addressed the community guidelines

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024 1

Years added!

from joss-reviews.

Saran-nns avatar Saran-nns commented on August 20, 2024 1

@paddyroddy I started my review and created an issue in the repo with few suggestions and requests

from joss-reviews.

klb2 avatar klb2 commented on August 20, 2024 1

Years added!

Thank you @paddyroddy

That completes my review @vissarion

from joss-reviews.

Saran-nns avatar Saran-nns commented on August 20, 2024 1

@vissarion @paddyroddy Couple of steps to go. Will be happy to flag accept then

from joss-reviews.

Saran-nns avatar Saran-nns commented on August 20, 2024 1

@paddyroddy thank you, great work @vissarion my checklist is complete and I am happy to recommend accept

from joss-reviews.

vissarion avatar vissarion commented on August 20, 2024 1

Many thanks to both reviewers @Saran-nns and @klb2 !

@paddyroddy could you please create a tagged release and an archive with the code (e.g. in zenodo) and share the link and the release version here? Please use the same title and authors while creating the zenodo archive.

from joss-reviews.

danielskatz avatar danielskatz commented on August 20, 2024 1

@vissarion - please set the archive to the DOI - note that link above from editorialbot isn't one that works

I'll proofread the paper later and continue the process

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024 1

I've made the changes @danielskatz

from joss-reviews.

danielskatz avatar danielskatz commented on August 20, 2024 1

Congratulations to @paddyroddy (Patrick James Roddy) on your work's publication!!

And thanks to @Saran-nns and @klb2 for reviewing, and to @vissarion for editing!
We couldn't do this without your voluntary efforts

(note that the DOI is not yet resolving, so I'll leave this open until it does)

from joss-reviews.

danielskatz avatar danielskatz commented on August 20, 2024 1

The DOI now resolves!

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

Hello humans, 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

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024
Software report:

github.com/AlDanial/cloc v 1.88  T=0.18 s (918.4 files/s, 65529.7 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
Python                         140           1613           1363           7221
Markdown                         3            134              0            571
TOML                             8             10              0            265
YAML                             5             21              0            158
TeX                              1              0              0            120
Unity-Prefab                     4              0              0             12
-------------------------------------------------------------------------------
SUM:                           161           1778           1363           8347
-------------------------------------------------------------------------------


gitinspector failed to run statistical information for the repository

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

Wordcount for paper.md is 968

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

OK DOIs

- 10.1002/j.1538-7305.1961.tb03977.x is OK
- 10.1002/j.1538-7305.1962.tb03279.x is OK
- 10.1051/0004-6361/201220729 is OK
- 10.1145/3134472.3134497 is OK
- 10.1109/TSP.2011.2166394 is OK
- 10.5281/zenodo.7268074 is OK
- 10.1109/LSP.2021.3050961 is OK
- 10.1109/TSP.2022.3233309 is OK
- 10.5281/zenodo.4085210 is OK
- 10.1002/j.1538-7305.1961.tb03976.x is OK
- 10.1111/j.1365-2966.2008.13448.x is OK
- 10.1029/2018GC007529 is OK

MISSING DOIs

- None

INVALID DOIs

- None

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

πŸ‘‰πŸ“„ Download article proof πŸ“„ View article proof on GitHub πŸ“„ πŸ‘ˆ

from joss-reviews.

klb2 avatar klb2 commented on August 20, 2024

Review checklist for @klb2

Conflict of interest

  • I confirm that I have read the JOSS conflict of interest (COI) policy and that: I have no COIs with reviewing this work or that any perceived COIs have been waived by JOSS for the purpose of this review.

Code of Conduct

General checks

  • Repository: Is the source code for this software available at the https://github.com/astro-informatics/sleplet?
  • License: Does the repository contain a plain-text LICENSE file with the contents of an OSI approved software license?
  • Contribution and authorship: Has the submitting author (@paddyroddy) made major contributions to the software? Does the full list of paper authors seem appropriate and complete?
  • Substantial scholarly effort: Does this submission meet the scope eligibility described in the JOSS guidelines
  • Data sharing: If the paper contains original data, data are accessible to the reviewers. If the paper contains no original data, please check this item.
  • Reproducibility: If the paper contains original results, results are entirely reproducible by reviewers. If the paper contains no original results, please check this item.
  • Human and animal research: If the paper contains original data research on humans subjects or animals, does it comply with JOSS's human participants research policy and/or animal research policy? If the paper contains no such data, please check this item.

Functionality

  • Installation: Does installation proceed as outlined in the documentation?
  • Functionality: Have the functional claims of the software been confirmed?
  • Performance: If there are any performance claims of the software, have they been confirmed? (If there are no claims, please check off this item.)

Documentation

  • A statement of need: Do the authors clearly state what problems the software is designed to solve and who the target audience is?
  • Installation instructions: Is there a clearly-stated list of dependencies? Ideally these should be handled with an automated package management solution.
  • Example usage: Do the authors include examples of how to use the software (ideally to solve real-world analysis problems).
  • Functionality documentation: Is the core functionality of the software documented to a satisfactory level (e.g., API method documentation)?
  • Automated tests: Are there automated tests or manual steps described so that the functionality of the software can be verified?
  • Community guidelines: Are there clear guidelines for third parties wishing to 1) Contribute to the software 2) Report issues or problems with the software 3) Seek support

Software paper

  • Summary: Has a clear description of the high-level functionality and purpose of the software for a diverse, non-specialist audience been provided?
  • A statement of need: Does the paper have a section titled 'Statement of need' that clearly states what problems the software is designed to solve, who the target audience is, and its relation to other work?
  • State of the field: Do the authors describe how this software compares to other commonly-used packages?
  • Quality of writing: Is the paper well written (i.e., it does not require editing for structure, language, or writing quality)?
  • References: Is the list of references complete, and is everything cited appropriately that should be cited (e.g., papers, datasets, software)? Do references in the text use the proper citation syntax?

from joss-reviews.

Saran-nns avatar Saran-nns commented on August 20, 2024

Review checklist for @Saran-nns

Conflict of interest

  • I confirm that I have read the JOSS conflict of interest (COI) policy and that: I have no COIs with reviewing this work or that any perceived COIs have been waived by JOSS for the purpose of this review.

Code of Conduct

General checks

  • Repository: Is the source code for this software available at the https://github.com/astro-informatics/sleplet?
  • License: Does the repository contain a plain-text LICENSE file with the contents of an OSI approved software license?
  • Contribution and authorship: Has the submitting author (@paddyroddy) made major contributions to the software? Does the full list of paper authors seem appropriate and complete?
  • Substantial scholarly effort: Does this submission meet the scope eligibility described in the JOSS guidelines
  • Data sharing: If the paper contains original data, data are accessible to the reviewers. If the paper contains no original data, please check this item.
  • Reproducibility: If the paper contains original results, results are entirely reproducible by reviewers. If the paper contains no original results, please check this item.
  • Human and animal research: If the paper contains original data research on humans subjects or animals, does it comply with JOSS's human participants research policy and/or animal research policy? If the paper contains no such data, please check this item.

Functionality

  • Installation: Does installation proceed as outlined in the documentation?
  • Functionality: Have the functional claims of the software been confirmed?
  • Performance: If there are any performance claims of the software, have they been confirmed? (If there are no claims, please check off this item.)

Documentation

  • A statement of need: Do the authors clearly state what problems the software is designed to solve and who the target audience is?
  • Installation instructions: Is there a clearly-stated list of dependencies? Ideally these should be handled with an automated package management solution.
  • Example usage: Do the authors include examples of how to use the software (ideally to solve real-world analysis problems).
  • Functionality documentation: Is the core functionality of the software documented to a satisfactory level (e.g., API method documentation)?
  • Automated tests: Are there automated tests or manual steps described so that the functionality of the software can be verified?
  • Community guidelines: Are there clear guidelines for third parties wishing to 1) Contribute to the software 2) Report issues or problems with the software 3) Seek support

Software paper

  • Summary: Has a clear description of the high-level functionality and purpose of the software for a diverse, non-specialist audience been provided?
  • A statement of need: Does the paper have a section titled 'Statement of need' that clearly states what problems the software is designed to solve, who the target audience is, and its relation to other work?
  • State of the field: Do the authors describe how this software compares to other commonly-used packages?
  • Quality of writing: Is the paper well written (i.e., it does not require editing for structure, language, or writing quality)?
  • References: Is the list of references complete, and is everything cited appropriately that should be cited (e.g., papers, datasets, software)? Do references in the text use the proper citation syntax?

from joss-reviews.

klb2 avatar klb2 commented on August 20, 2024

The developers of the pyssht and pys2let packages fixed the issue and I was now able to install the sleplet package.

I will continue my review this week.

from joss-reviews.

klb2 avatar klb2 commented on August 20, 2024

From the JOSS guidelines (https://joss.readthedocs.io/en/latest/submitting.html#what-should-my-paper-contain):

As this short list shows, JOSS papers are only expected to contain a limited set of metadata (see example below), a Statement of need, Summary, Acknowledgements, and References sections. You can look at an example accepted paper. Given this format, a β€œfull length” paper is not permitted, and software documentation such as API (Application Programming Interface) functionality should not be in the paper and instead should be outlined in the software documentation.

Based on this, I think that the "Demonstration" section should be completely removed from the paper and the "Conclusions" section should be merged into the "Summary" and "Statement of Needs" sections.

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024

From the JOSS guidelines (https://joss.readthedocs.io/en/latest/submitting.html#what-should-my-paper-contain):

As this short list shows, JOSS papers are only expected to contain a limited set of metadata (see example below), a Statement of need, Summary, Acknowledgements, and References sections. You can look at an example accepted paper. Given this format, a β€œfull length” paper is not permitted, and software documentation such as API (Application Programming Interface) functionality should not be in the paper and instead should be outlined in the software documentation.

Based on this, I think that the "Demonstration" section should be completely removed from the paper and the "Conclusions" section should be merged into the "Summary" and "Statement of Needs" sections.

Have done

from joss-reviews.

klb2 avatar klb2 commented on August 20, 2024

@editorialbot generate pdf

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

πŸ‘‰πŸ“„ Download article proof πŸ“„ View article proof on GitHub πŸ“„ πŸ‘ˆ

from joss-reviews.

klb2 avatar klb2 commented on August 20, 2024

Is there any documentation besides the examples in the README? I could not find an API documentation that documents all of the individual functions of the package.

from joss-reviews.

klb2 avatar klb2 commented on August 20, 2024

In the paper, it is stated that

To the author’s knowledge, there is no public software which allows one to compute Slepian wavelets (or similar approach) on the sphere or general manifolds/meshes.
SHTools (Wieczorek & Meschede, 2018) and slepian_alpha (Simons et al., n.d.) are examples of codes which allow one to compute Slepian functions on the sphere. In conjunction with SSHT (McEwen & Wiaux, 2011), S2LET (Leistedt, B. et al., 2013) may be used to develop scale-discretised wavelets on the whole sphere.

and later

Whilst Slepian wavelets may be trivially computed from a set of Slepian functions, the computation of the spherical Slepian functions themselves are computationally complex

Based on this, it is unclear for me what the advantages of the sleplet package are over the existing ones like SHTools and slepian_alpha. It is said in the paper that these two packages allow computing the Slepian functions and that it is trivial to compute the Slepian wavelets from the Slepian functions.

from joss-reviews.

vissarion avatar vissarion commented on August 20, 2024

Good point @klb2. @paddyroddy could you please clarify?

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024

On closer inspection ShTools only computes the Slepian functions of a polar cap https://shtools.github.io/SHTOOLS/pysphericalcapcoef.html which is much easier to compute. Whereas SLEPLET handles any arbitrary mask. Further, it works for mesh data rather than just the sphere.

slepian_alpha is a huge set of MATLAB code with no documentation or anything. I believe it's largely used to create the various plots on this paper, https://arxiv.org/pdf/math/0408424.pdf. Again, it is defined only for the sphere.

In an ideal world, SLEPLET would have relied on another library to generate the Slepian functions and the build the wavelets from them. However, this proved non-trivial to use and, as such, it was necessary to compute them myself. My wording should have been clearer.

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024

Is there any documentation besides the examples in the README? I could not find an API documentation that documents all of the individual functions of the package.

I hadn't realised full API documentation was required. I can look into this.

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024

Is there any documentation besides the examples in the README? I could not find an API documentation that documents all of the individual functions of the package.

I hadn't realised full API documentation was required. I can look into this.

I've made some: https://astro-informatics.github.io/sleplet/

from joss-reviews.

klb2 avatar klb2 commented on August 20, 2024

On closer inspection ShTools only computes the Slepian functions of a polar cap https://shtools.github.io/SHTOOLS/pysphericalcapcoef.html which is much easier to compute. Whereas SLEPLET handles any arbitrary mask. Further, it works for mesh data rather than just the sphere.

slepian_alpha is a huge set of MATLAB code with no documentation or anything. I believe it's largely used to create the various plots on this paper, https://arxiv.org/pdf/math/0408424.pdf. Again, it is defined only for the sphere.

In an ideal world, SLEPLET would have relied on another library to generate the Slepian functions and the build the wavelets from them. However, this proved non-trivial to use and, as such, it was necessary to compute them myself. My wording should have been clearer.

Thank you for clarifying. I guess it would be good, if some of these details could also be added to the paper in order to make the differences to the sleplet package clear.

from joss-reviews.

klb2 avatar klb2 commented on August 20, 2024

With the latest version of the software, I was able to run all provided examples without any errors. The unit tests also all pass on my machine.
Thank you for adding pooch for the data management/downloads @paddyroddy.

from joss-reviews.

klb2 avatar klb2 commented on August 20, 2024

From my side as a reviewer, only the following points are open:

  • Community guidelines (CONTRIBUTING.md and issue templates)
  • Argument/function documentation. I appreciate the API documentation, however, it does not contain any explanations of the individual functions and their arguments.

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024

On closer inspection ShTools only computes the Slepian functions of a polar cap https://shtools.github.io/SHTOOLS/pysphericalcapcoef.html which is much easier to compute. Whereas SLEPLET handles any arbitrary mask. Further, it works for mesh data rather than just the sphere.
slepian_alpha is a huge set of MATLAB code with no documentation or anything. I believe it's largely used to create the various plots on this paper, https://arxiv.org/pdf/math/0408424.pdf. Again, it is defined only for the sphere.
In an ideal world, SLEPLET would have relied on another library to generate the Slepian functions and the build the wavelets from them. However, this proved non-trivial to use and, as such, it was necessary to compute them myself. My wording should have been clearer.

Thank you for clarifying. I guess it would be good, if some of these details could also be added to the paper in order to make the differences to the sleplet package clear.

I've added some comments to explain this.

from joss-reviews.

klb2 avatar klb2 commented on August 20, 2024

@editorialbot generate pdf

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

πŸ‘‰πŸ“„ Download article proof πŸ“„ View article proof on GitHub πŸ“„ πŸ‘ˆ

from joss-reviews.

klb2 avatar klb2 commented on August 20, 2024

On closer inspection ShTools only computes the Slepian functions of a polar cap https://shtools.github.io/SHTOOLS/pysphericalcapcoef.html which is much easier to compute. Whereas SLEPLET handles any arbitrary mask. Further, it works for mesh data rather than just the sphere.
slepian_alpha is a huge set of MATLAB code with no documentation or anything. I believe it's largely used to create the various plots on this paper, https://arxiv.org/pdf/math/0408424.pdf. Again, it is defined only for the sphere.
In an ideal world, SLEPLET would have relied on another library to generate the Slepian functions and the build the wavelets from them. However, this proved non-trivial to use and, as such, it was necessary to compute them myself. My wording should have been clearer.

Thank you for clarifying. I guess it would be good, if some of these details could also be added to the paper in order to make the differences to the sleplet package clear.

I've added some comments to explain this.

Thank you @paddyroddy for adding this part to the paper. It all looks good for me now.
The only minor comment: The dates/years for two references are missing in the paper, for the SLEPLET package on Zenodo (2023) and for the slepian_alpha package on Zenodo (2020).

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024

@editorialbot generate preprint

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

πŸ“„ Preprint file created: Find it here in the Artifacts list πŸ“„

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024

@editorialbot generate preprint

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024

Thank you, @klb2 and @Saran-nns for your guidance.

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

πŸ“„ Preprint file created: Find it here in the Artifacts list πŸ“„

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024

Sorry just have just capitalised the title

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024

@editorialbot generate preprint

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024

@vissarion here is the archive link https://zenodo.org/record/7802428, current version v1.3.4

from joss-reviews.

vissarion avatar vissarion commented on August 20, 2024

Thanks! Please use the full title in the archive i.e. "SLEPLET: Slepian scale-discretised wavelets in Python" which is the same as the paper.

from joss-reviews.

vissarion avatar vissarion commented on August 20, 2024

@editorialbot generate pdf

from joss-reviews.

vissarion avatar vissarion commented on August 20, 2024

@editorialbot check references

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

Couldn't check the bibtex because branch name is incorrect: joss-paper

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

⚠️ An error happened when generating the pdf.

from joss-reviews.

vissarion avatar vissarion commented on August 20, 2024

@editorialbot generate pdf

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

⚠️ An error happened when generating the pdf.

from joss-reviews.

vissarion avatar vissarion commented on August 20, 2024

@editorialbot set main as branch

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

Done! branch is now main

from joss-reviews.

vissarion avatar vissarion commented on August 20, 2024

@editorialbot generate pdf

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

πŸ‘‰πŸ“„ Download article proof πŸ“„ View article proof on GitHub πŸ“„ πŸ‘ˆ

from joss-reviews.

vissarion avatar vissarion commented on August 20, 2024

@editorialbot check references

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

OK DOIs

- 10.1109/TSP.2016.2646668 is OK
- 10.1038/s41586-020-2649-2 is OK
- 10.1002/j.1538-7305.1961.tb03977.x is OK
- 10.1002/j.1538-7305.1962.tb03279.x is OK
- 10.1051/0004-6361/201220729 is OK
- 10.1145/3134472.3134497 is OK
- 10.1109/TSP.2011.2166394 is OK
- 10.1109/LSP.2021.3050961 is OK
- 10.1109/TSP.2022.3233309 is OK
- 10.5281/zenodo.7268074 is OK
- 10.1137/S0036144504445765 is OK
- 10.5281/zenodo.4085210 is OK
- 10.1002/j.1538-7305.1961.tb03976.x is OK
- 10.1111/j.1365-2966.2008.13448.x is OK
- 10.1029/2018GC007529 is OK

MISSING DOIs

- None

INVALID DOIs

- None

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024

Thanks! Please use the full title in the archive i.e. "SLEPLET: Slepian scale-discretised wavelets in Python" which is the same as the paper.

Sorry do you mean in the paper.bib in the archive or do you mean the archive name should be that?

from joss-reviews.

vissarion avatar vissarion commented on August 20, 2024

Thanks! Please use the full title in the archive i.e. "SLEPLET: Slepian scale-discretised wavelets in Python" which is the same as the paper.

Sorry do you mean in the paper.bib in the archive or do you mean the archive name should be that?

I mean the zenodo archive name, thanks.

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024

Apologies, I was on holiday and hadn't understood what you meant previously. I've made a release v1.3.5 with the name in the archive https://zenodo.org/record/7825341.

from joss-reviews.

Saran-nns avatar Saran-nns commented on August 20, 2024

@paddyroddy I think @vissarion asks for the zenodo archive title to be same as the title of the paper SLEPLET: Slepian Scale-Discretised Wavelets in Python. But the archive title is still SLEPLET.

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024

Ah, thanks @Saran-nns, I get it now. All updated!

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024

Realised I forgot to update the Zenodo citation in the paper. Have fixed in v1.3.6 https://zenodo.org/record/7835860.

from joss-reviews.

vissarion avatar vissarion commented on August 20, 2024

@editorialbot set https://zenodo.org/record/7835860 as archive

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

Done! Archive is now [ https://zenodo.org/record/7835860](https://doi.org/ https://zenodo.org/record/7835860)

from joss-reviews.

vissarion avatar vissarion commented on August 20, 2024

@editorialbot set v1.3.6 as version

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

Done! version is now v1.3.6

from joss-reviews.

vissarion avatar vissarion commented on August 20, 2024

@editorialbot recommend-accept

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024
Attempting dry run of processing paper acceptance...

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

OK DOIs

- 10.1109/TSP.2016.2646668 is OK
- 10.1038/s41586-020-2649-2 is OK
- 10.1002/j.1538-7305.1961.tb03977.x is OK
- 10.1002/j.1538-7305.1962.tb03279.x is OK
- 10.1051/0004-6361/201220729 is OK
- 10.1145/3134472.3134497 is OK
- 10.1109/TSP.2011.2166394 is OK
- 10.1109/LSP.2021.3050961 is OK
- 10.1109/TSP.2022.3233309 is OK
- 10.5281/zenodo.7268074 is OK
- 10.1137/S0036144504445765 is OK
- 10.5281/zenodo.4085210 is OK
- 10.1002/j.1538-7305.1961.tb03976.x is OK
- 10.1111/j.1365-2966.2008.13448.x is OK
- 10.1029/2018GC007529 is OK

MISSING DOIs

- Errored finding suggestions for "Slepian Wavelets for the Analysis of Incomplete Da...", please try later

INVALID DOIs

- None

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

πŸ‘‹ @openjournals/csism-eics, this paper is ready to be accepted and published.

Check final proof πŸ‘‰πŸ“„ Download article

If the paper PDF and the deposit XML files look good in openjournals/joss-papers#4151, then you can now move forward with accepting the submission by compiling again with the command @editorialbot accept

from joss-reviews.

danielskatz avatar danielskatz commented on August 20, 2024

@editorialbot set 10.5281/zenodo.7835860 as archive

@vissarion - nvm, I'll just do this while I'm proofreading...

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

Done! Archive is now 10.5281/zenodo.7835860

from joss-reviews.

danielskatz avatar danielskatz commented on August 20, 2024

@paddyroddy - I'm suggesting some changes in astro-informatics/sleplet#142 Please merge this, or let me know what you disagree with.

In addition, I have a suggestion: In the paper, lines 29-31 seem redundant with the previous part of that section. Can these lines be removed (29) and/or merged (30-31)?

Once you've decided on and made changes, please let me know, and we can continue the acceptance and publication process.

from joss-reviews.

danielskatz avatar danielskatz commented on August 20, 2024

@editorialbot recommend-accept

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024
Attempting dry run of processing paper acceptance...

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

πŸ‘‹ @openjournals/csism-eics, this paper is ready to be accepted and published.

Check final proof πŸ‘‰πŸ“„ Download article

If the paper PDF and the deposit XML files look good in openjournals/joss-papers#4155, then you can now move forward with accepting the submission by compiling again with the command @editorialbot accept

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

OK DOIs

- 10.1109/TSP.2016.2646668 is OK
- 10.1038/s41586-020-2649-2 is OK
- 10.1002/j.1538-7305.1961.tb03977.x is OK
- 10.1002/j.1538-7305.1962.tb03279.x is OK
- 10.1051/0004-6361/201220729 is OK
- 10.1145/3134472.3134497 is OK
- 10.1109/TSP.2011.2166394 is OK
- 10.1109/LSP.2021.3050961 is OK
- 10.1109/TSP.2022.3233309 is OK
- 10.5281/zenodo.7268074 is OK
- 10.1137/S0036144504445765 is OK
- 10.5281/zenodo.4085210 is OK
- 10.1002/j.1538-7305.1961.tb03976.x is OK
- 10.1111/j.1365-2966.2008.13448.x is OK
- 10.1029/2018GC007529 is OK

MISSING DOIs

- Errored finding suggestions for "Slepian Scale-Discretised Wavelets on Manifolds", please try later

INVALID DOIs

- None

from joss-reviews.

danielskatz avatar danielskatz commented on August 20, 2024

@editorialbot accept

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024
Doing it live! Attempting automated processing of paper acceptance...

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

Ensure proper citation by uploading a plain text CITATION.cff file to the default branch of your repository.

If using GitHub, a Cite this repository menu will appear in the About section, containing both APA and BibTeX formats. When exported to Zotero using a browser plugin, Zotero will automatically create an entry using the information contained in the .cff file.

You can copy the contents for your CITATION.cff file here:

CITATION.cff

cff-version: "1.2.0"
authors:
- family-names: Roddy
  given-names: Patrick J.
  orcid: "https://orcid.org/0000-0002-6271-1700"
doi: 10.5281/zenodo.7835860
message: If you use this software, please cite our article in the
  Journal of Open Source Software.
preferred-citation:
  authors:
  - family-names: Roddy
    given-names: Patrick J.
    orcid: "https://orcid.org/0000-0002-6271-1700"
  date-published: 2023-04-20
  doi: 10.21105/joss.05221
  issn: 2475-9066
  issue: 84
  journal: Journal of Open Source Software
  publisher:
    name: Open Journals
  start: 5221
  title: "SLEPLET: Slepian Scale-Discretised Wavelets in Python"
  type: article
  url: "https://joss.theoj.org/papers/10.21105/joss.05221"
  volume: 8
title: "SLEPLET: Slepian Scale-Discretised Wavelets in Python"

If the repository is not hosted on GitHub, a .cff file can still be uploaded to set your preferred citation. Users will be able to manually copy and paste the citation.

Find more information on .cff files here and here.

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

🐦🐦🐦 πŸ‘‰ Tweet for this paper πŸ‘ˆ 🐦🐦🐦

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

🐘🐘🐘 πŸ‘‰ Toot for this paper πŸ‘ˆ 🐘🐘🐘

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

🚨🚨🚨 THIS IS NOT A DRILL, YOU HAVE JUST ACCEPTED A PAPER INTO JOSS! 🚨🚨🚨

Here's what you must now do:

  1. Check final PDF and Crossref metadata that was deposited πŸ‘‰ openjournals/joss-papers#4156
  2. Wait a couple of minutes, then verify that the paper DOI resolves https://doi.org/10.21105/joss.05221
  3. If everything looks good, then close this review issue.
  4. Party like you just published a paper! πŸŽ‰πŸŒˆπŸ¦„πŸ’ƒπŸ‘»πŸ€˜

Any issues? Notify your editorial technical team...

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

πŸŽ‰πŸŽ‰πŸŽ‰ Congratulations on your paper acceptance! πŸŽ‰πŸŽ‰πŸŽ‰

If you would like to include a link to your paper from your README use the following code snippets:

Markdown:
[![DOI](https://joss.theoj.org/papers/10.21105/joss.05221/status.svg)](https://doi.org/10.21105/joss.05221)

HTML:
<a style="border-width:0" href="https://doi.org/10.21105/joss.05221">
  <img src="https://joss.theoj.org/papers/10.21105/joss.05221/status.svg" alt="DOI badge" >
</a>

reStructuredText:
.. image:: https://joss.theoj.org/papers/10.21105/joss.05221/status.svg
   :target: https://doi.org/10.21105/joss.05221

This is how it will look in your documentation:

DOI

We need your help!

The Journal of Open Source Software is a community-run journal and relies upon volunteer effort. If you'd like to support us please consider doing either one (or both) of the the following:

from joss-reviews.

paddyroddy avatar paddyroddy commented on August 20, 2024

@editorialbot generate preprint

from joss-reviews.

editorialbot avatar editorialbot commented on August 20, 2024

πŸ“„ Preprint file created: Find it here in the Artifacts list πŸ“„

from joss-reviews.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    πŸ–– Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. πŸ“ŠπŸ“ˆπŸŽ‰

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❀️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.