Skip to content
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

[REVIEW]: fathon: A Python package for a fast computation of detrendend fluctuation analysis and related algorithms #1828

Closed
whedon opened this issue Oct 22, 2019 · 81 comments
Assignees
Labels

Comments

@whedon
Copy link
Collaborator

@whedon whedon commented Oct 22, 2019

Submitting author: @stfbnc (Stefano Bianchi)
Repository: https://github.com/stfbnc/fathon.git
Version: v0.1.2
Editor: @mbobra
Reviewer: @ali-ramadhan, @ashwinvis
Archive: 10.5281/zenodo.3601637

Status

status

Status badge code:

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

Reviewers and authors:

Please avoid lengthy details of difficulties in the review thread. Instead, please create a new issue in the target repository and link to those issues (especially acceptance-blockers) by leaving comments in the review thread below. (For completists: if the target issue tracker is also on GitHub, linking the review thread in the issue or vice versa will create corresponding breadcrumb trails in the link target.)

Reviewer instructions & questions

@ali-ramadhan & @ashwinvis, please carry out your review in this issue by updating the checklist below. If you cannot edit the checklist please:

  1. Make sure you're logged in to your GitHub account
  2. Be sure to accept the invite at this URL: https://github.com/openjournals/joss-reviews/invitations

The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @mbobra know.

Please try and complete your review in the next two weeks

Review checklist for @ali-ramadhan

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 repository url?
  • 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 (@stfbnc) made major contributions to the software? Does the full list of paper authors seem appropriate and complete?

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: Do the authors clearly state what problems the software is designed to solve and who the target audience is?
  • 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?

Review checklist for @ashwinvis

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 repository url?
  • 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 (@stfbnc) made major contributions to the software? Does the full list of paper authors seem appropriate and complete?

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: Do the authors clearly state what problems the software is designed to solve and who the target audience is?
  • 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?
@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Oct 22, 2019

Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @ali-ramadhan, @ashwinvis it looks like you're currently assigned to review this paper 🎉.

⭐️ Important ⭐️

If you haven't already, you should seriously consider unsubscribing from GitHub notifications for this (https://github.com/openjournals/joss-reviews) repository. As a reviewer, you're probably currently watching this repository which means for GitHub's default behaviour you will receive notifications (emails) for all reviews 😿

To fix this do the following two things:

  1. Set yourself as 'Not watching' https://github.com/openjournals/joss-reviews:

watching

  1. You may also like to change your default settings for this watching repositories in your GitHub profile here: https://github.com/settings/notifications

notifications

For a list of things I can do to help you, just type:

@whedon commands

For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:

@whedon generate pdf
@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Oct 22, 2019

Attempting PDF compilation. Reticulating splines etc...
@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Oct 22, 2019

@mbobra

This comment has been minimized.

Copy link
Member

@mbobra mbobra commented Oct 22, 2019

👋@ali-ramadhan @ashwinvis Thank you again for volunteering to review. The review instructions are listed above. Please let me know if you have any questions!

@ali-ramadhan

This comment has been minimized.

Copy link

@ali-ramadhan ali-ramadhan commented Nov 6, 2019

I like the idea of having a simple and intuitive open-source Python package offering numerous DFA algorithms and the examples look great, but I think fathon needs some work to make it more accessible to potential users.

My major comments are:

  1. fathon is difficult to install. I was actually not able to install it. It could greatly benefit from an automated package management solution.
  2. Currently documentation is provided via a single pdf file. I think generating documentation using a package like Sphinx would provide potential users with searchable documentation that gets updated alongside the code and integrates in the great docstrings that are already in the source code.
  3. Right now the burden is on the user to run manual tests to verify that fathon works, but I think this approach cannot scale as improvements are made and other people decide to contribute. I think an automated testing solution would greatly improve the situation here.

Maybe @mbobra can confirm whether these three points are necessary for publication in JOSS. I do think they would improve fathon and make the package much more accessible to potential users.

A more detailed review follows below.

I plan to come back and finish this review once I can install fathon.

Functionality

Installation

  • Option 1 (git clone stfbnc/fathon.git) does not work. It should be git clone https://github.com/stfbnc/fathon.git.
  • The second option is to "Go to the repository and download the code" however I couldn't figure out where to download the code. Usually on GitHub tagged releases are downloadable as zip files or tarballs, but fathon has no tagged releases. Unless the README is referring to the green "Clone or download" button which would be good to mention.

I wonder if providing a Python package installable via pip or conda would make fathon more accessible and easier to install. I'm not super familiar with either but because of the C libraries, presumably conda is the better option?

GSL seems to be a pre-requisite which is available through most Linux package managers. Installing fathon on Windows might prove difficult and it looks like you might need to install all of Cygwin just to get GSL to install fathon. I might mention this as most Windows users have no idea what Cygwin is.

I was not able to install fathon myself. I opened an issue about this: stfbnc/fathon#1

I believe the installation instructions are unfriendly. fathon seems much harder to install for me than the vast majority of Python packages, many of them which depend on C libraries too.

Besides having to manually install or compile GSL, you also need to know where include and library files are placed on your operating system, and you need to know the "standard location of Python packages" for your particular OS and Python environment combination.

I think this is a huge barrier to usability and most Python users won't be able to install fathon.

I opened an issue to ask whether providing a pip or conda package is an option: stfbnc/fathon#2

Functionality

I was not able to install fathon but when I do I will try to confirm this.

I'm not sure if whether four tests on Gaussian white noise are sufficient to claim that fathon fully works. That seems to be the easiest test case. I would like to explore testing other cases.

Performance

No performance claims I could find except for "It is mostly written in Cython and C in order to speed up computations" which seems reasonable.

Documentation

A statement of need

Some motivation is provided in the JOSS paper and documentation.

Installation instructions

There is a clearly-stated list of depdencies (GSL) although Cython might be missing (see stfbnc/fathon#1). I strongly believe that fathon will benefit from having installation handled via an automated package managemnent solution such as pip or conda (see stfbnc/fathon#2).

Examples

The examples directory contains some nice-looking examples that I'm looking forward to running and playing around with. There are also some illustrative examples in the fathon_docs.pdf.

Functionality documentation

I could not find any documentation for the API. I also think having the documentation in PDF form is detrimental to being simple and user friendly as it becomes difficult to link to them and it seems that nobody can contribute to improving the docs as the LaTeX document is not part of the repository.

I believe with documentation packages like Sphinx, documenting the API becomes easy, and it's trivial to deploy the docs to a website like ReadTheDocs.

Looking through the code, there are really good docstrings so it's a shame that they're currently hidden from the user!

Automated tests

There are no automated tests, and no continuous integration.

There is a Jupyter notebook and four Python scripts provided where the burden is on the user to run the tests.

I think it would be easy to convert the four scripts to automated tests that run on a CI platform like Travis CI. I haven't set up Travis with Python but it was painless with other languages, so I assume it should be easy to set up.

Community guidelines

I could not find any community guidelines or a contributor's guide. There should be "clear guidelines for third parties wishing to 1) Contribute to the software 2) Report issues or problems with the software 3) Seek support". This can all be included in the README I believe.

I find it kind of weird that the README states "For bugs or improvements, write to fathon.package@gmail.com" when the package is on GitHub. I think the README should strongly encourage the submission of GitHub issues so that bugs and improvements can be visible to all users and the discussion can be more transparent. This also allows other people to contribute as it's not dependent on one person checking an email inbox and having the time to reply. Keeping track of issues on GitHub also leaves behind a rich history of package development that allows other contributors to take over if the original package developer no longer has the time to develop the package.

Software paper

Summary

I think the first paragraph is not accessible to a "diverse, non-specialist audience". DFA itself isn't actually described. Terms such as "fluctuation function", "Hurst exponent", and "anti-persistent" would not be understood by non-specialists I think.

I also think the mass citation of ~10 different works without context on the second sentence is not helpful to readers, especially non-specialists. I would suggest picking 2-4 works and placing them in context so that a non-specialist reader can understand how DFA was used in that particular work and why it was useful.

Statements such as "found various applications in other fields, like geophysics or economy" seem vague and not very helpful. It can be argued that the entire field of mathematics and statistics has found found various applications in other fields, like geophysics or economy.

The rest of the paper looks good to me.

State of the field

A quick Google search yielded three results for open-source Python packages that provide DFA algorithms:

It doesn't have to enter the README or JOSS paper but I'm curious how fathon is different. Do the others not provide the four algorithms provided by fathon? Are these packages worth mentioning?

A MATLAB package is mentioned in the JOSS paper which I assume is more comprehensive than these Python packages?

Quality of writing

In the first paragraph, I think you mean "economics" instead of "economy".

Otherwise I think it's fine.

References

These seem fine to me, but as mentioned above I think there may be too many.

I think "co_2" and "mauna loa" should be captialized to "CO_2" and "Mauna Loa" for the Varotsos et al. (2007) reference.

Also missing some captialization in the Janosi et al. (1999) reference. Should be "Also I think Statistical analysis of 5 s index data of the Budapest Stock Exchange".

"matlab" should be "Matlab" in the Ihlen (2012) reference.

@mbobra

This comment has been minimized.

Copy link
Member

@mbobra mbobra commented Nov 6, 2019

👋 @ashwinvis Let me know if you have any questions or need any help completing your review!

@ashwinvis

This comment has been minimized.

Copy link

@ashwinvis ashwinvis commented Nov 6, 2019

@mbobra Working on it :)

@mbobra

This comment has been minimized.

Copy link
Member

@mbobra mbobra commented Nov 6, 2019

My major comments are:

  1. fathon is difficult to install. I was actually not able to install it. It could greatly benefit from an automated package management solution.

  2. Currently documentation is provided via a single pdf file. I think generating documentation using a package like Sphinx would provide potential users with searchable documentation that gets updated alongside the code and integrates in the great docstrings that are already in the source code.

  3. Right now the burden is on the user to run manual tests to verify that fathon works, but I think this approach cannot scale as improvements are made and other people decide to contribute. I think an automated testing solution would greatly improve the situation here.

Maybe @mbobra can confirm whether these three points are necessary for publication in JOSS. I do think they would improve fathon and make the package much more accessible to potential users.

Thanks for your detailed review, @ali-ramadhan. Here are my replies:

  1. According to the JOSS review guidelines, instructions for installing the software should be clear. An automated package management solution is ideal, but not required. However, the authors must provide clear installation instructions that work as well as a clear list of dependencies.

  2. According to the JOSS review guidelines, the documentation must to include four items: a statement of need, installation instructions, example usage, and API documentation. The first three are pretty clear; the fourth is a bit subjective (and up to the reviewer to decide), but the core API must be documented in a reasonable way.

  3. According to JOSS' review guidelines, the reviewer must be able to check to see if the software actually works. An automated test suite is ideal, but documented manual steps are okay too. But there must be some way for the reviewer to objectively assess whether the software works.

Does this help, @ali-ramadhan?

@ali-ramadhan

This comment has been minimized.

Copy link

@ali-ramadhan ali-ramadhan commented Nov 7, 2019

Thank you @mbobra, that's helpful! Sorry it's my first review but I should have familiarized myself with the review guidelines more closely.

  1. According to the JOSS review guidelines, instructions for installing the software should be clear. An automated package management solution is ideal, but not required. However, the authors must provide clear installation instructions that work as well as a clear list of dependencies.

I would be happy with clear installation instructions that can be followed by people without having to spend too much time figuring out where GSL include/library files are located and where Python packages are usually installed.

  1. According to the JOSS review guidelines, the documentation must to include four items: a statement of need, installation instructions, example usage, and API documentation. The first three are pretty clear; the fourth is a bit subjective (and up to the reviewer to decide), but the core API must be documented in a reasonable way.

I think fathon satisfies the first three and does technically satisfy the fourth as functions in the source code are documented with good docstrings. I feel like it doesn't count as API documentation though unless the docstrings are compiled into documentation, but maybe having docstrings in the source code meets the JOSS requirements.

  1. According to JOSS' review guidelines, the reviewer must be able to check to see if the software actually works. An automated test suite is ideal, but documented manual steps are okay too. But there must be some way for the reviewer to objectively assess whether the software works.

The manual steps in the README sound good then.

@mbobra

This comment has been minimized.

Copy link
Member

@mbobra mbobra commented Nov 7, 2019

@ali-ramadhan Sounds good!
Re: The 1st point - Thanks for opening an issue about the installation instructions.
Re: The 2nd point - Feel free to open an issue about the core API documentation.
Re: The 3rd point - It seems like this is no longer an issue.

@stfbnc Let me know if you have any questions or need any help resolving these issues.

@stfbnc

This comment has been minimized.

Copy link

@stfbnc stfbnc commented Nov 8, 2019

@ali-ramadhan Thanks for the comments, I'm now working on resolving the issues and on the documentation.

@mbobra Can I modify the paper as ali-ramadhan suggested, or should I wait for the second reviewer's comments?

@mbobra

This comment has been minimized.

Copy link
Member

@mbobra mbobra commented Nov 8, 2019

@stfbnc Yes, feel free to modify the paper right away! Every time you modify, you can make a comment in this thread with the command @whedon generate pdf to ask the JOSS bot Whedon to compile a PDF (so that we can all see the latest draft). This is an iterative process, so there is absolutely no issue with modifying and re-generating the paper as many times as you like.

@stfbnc

This comment has been minimized.

Copy link

@stfbnc stfbnc commented Nov 10, 2019

@whedon generate pdf

@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Nov 10, 2019

Attempting PDF compilation. Reticulating splines etc...
@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Nov 10, 2019

@stfbnc

This comment has been minimized.

Copy link

@stfbnc stfbnc commented Nov 10, 2019

@whedon generate pdf

@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Nov 10, 2019

Attempting PDF compilation. Reticulating splines etc...
@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Nov 10, 2019

@stfbnc

This comment has been minimized.

Copy link

@stfbnc stfbnc commented Nov 10, 2019

@ali-ramadhan I have corrected the paper as you suggested, there is now a theoretical introduction on the algorithms covered by fathon.

For the State of the field section:

  • the three packages you mentioned only cover the DFA algorithm (and with not all the options provided by fathon), while they do not cover MFDFA, DCCA, and HT.
  • the Matlab package is instead more comprehensive, and (besides not being written in Python) does not cover all the features offered by fathon. It does not cover DCCA, is less flexible, and some function's parameters are not considered, like the possibility to calculate the fluctuation function only forward or both forward and backward.
@mbobra

This comment has been minimized.

Copy link
Member

@mbobra mbobra commented Nov 18, 2019

👋 @ashwinvis I'm trying to move this along for the authors. Let me know if you want any help in completing your review! I'm genuinely happy to help.

@stfbnc

This comment has been minimized.

Copy link

@stfbnc stfbnc commented Nov 21, 2019

@ali-ramadhan @ashwinvis Sorry for the late reply. I have updated the repository. Requirements are now specified, along with the new installation guidelines. I have also added:

  • automated tests on Travis CI
  • readthedocs documentation
  • contributing guidelines
@stfbnc

This comment has been minimized.

Copy link

@stfbnc stfbnc commented Nov 21, 2019

Code coverage is by now 0%, but only because the package is written in cython and not python. I'm still trying to figure out how to make codecov consider also .pyx or .so files.

@ashwinvis

This comment has been minimized.

Copy link

@ashwinvis ashwinvis commented Nov 21, 2019

@mbobra Thanks for the nudge. I was coordinating with the author to improve the installation and packaging situation stfbnc/fathon#1 (comment) so that we have a version that can be reviewed.

@stfbnc Code coverage is not necessary for the paper, but if you are willing to go that extra mile, it can be fixed by following this document. You need to enable CYTHON_TRACE=1 and Cython.Coverage plugin when you run tests / CI. Do not enable linetrace by default, as it can degrade performance.

@mbobra

This comment has been minimized.

Copy link
Member

@mbobra mbobra commented Dec 9, 2019

@stfbnc Let me know your timeline/plans for this paper. You should feel absolutely free to take as much time as you need, so there's no rush or anything. I only want to know the timeline because if it is long (>1 month or so) we can add a "paused" label to this review to indicate to the Editors in Chief that they don't need to watch over this thread in the interim.

@mbobra

This comment has been minimized.

Copy link
Member

@mbobra mbobra commented Dec 9, 2019

/ooo December 16 until January 6

@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Jan 8, 2020

Reference check summary:

OK DOIs

- None

MISSING DOIs

- https://doi.org/10.1016/j.physa.2010.10.022 may be missing for title: DCCA cross-correlation coefficient: quantifying level of cross-correlation.
- https://doi.org/10.1103/physrevlett.100.084102 may be missing for title: Detrended cross-correlation analysis: a new method for analyzing two nonstationary time series.
- https://doi.org/10.3389/fphys.2012.00141 may be missing for title: Introduction to multifractal detrended fluctuation analysis in Matlab.
- https://doi.org/10.1103/physreve.60.1390 may be missing for title: Statistical properties of the volatility of price fluctuations.
- https://doi.org/10.1103/physrevlett.90.108501 may be missing for title: Scaling of atmosphere and ocean temperature correlations in observations and climate models.
- https://doi.org/10.1103/physreve.47.3730 may be missing for title: Finite-size effects on long-range correlations: Implications for analyzing DNA sequences

INVALID DOIs

- None
@stfbnc

This comment has been minimized.

Copy link

@stfbnc stfbnc commented Jan 8, 2020

@whedon generate pdf

@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Jan 8, 2020

@stfbnc

This comment has been minimized.

Copy link

@stfbnc stfbnc commented Jan 8, 2020

@whedon check references

@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Jan 8, 2020

Reference check summary:

OK DOIs

- http://dx.doi.org/10.1016/j.physa.2010.10.022 is OK
- http://dx.doi.org/10.1103/PhysRevLett.100.084102 is OK
- http://dx.doi.org/10.3389/fphys.2012.00141 is OK
- http://dx.doi.org/10.1016/S0378-4371(02)01383-3 is OK
- http://dx.doi.org/10.1103/PhysRevE.69.056107 is OK
- http://dx.doi.org/10.1103/PhysRevE.60.1390 is OK
- http://dx.doi.org/10.1103/PhysRevLett.90.108501 is OK
- https://doi.org/10.5194/acp-7-629-2007 is OK
- http://dx.doi.org/10.1103/PhysRevE.47.3730 is OK

MISSING DOIs

- None

INVALID DOIs

- None
@stfbnc

This comment has been minimized.

Copy link

@stfbnc stfbnc commented Jan 8, 2020

@mbobra DOIs added!

@mbobra

This comment has been minimized.

Copy link
Member

@mbobra mbobra commented Jan 8, 2020

@whedon set 10.5281/zenodo.3601637 as archive

@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Jan 8, 2020

OK. 10.5281/zenodo.3601637 is the archive.

@mbobra

This comment has been minimized.

Copy link
Member

@mbobra mbobra commented Jan 8, 2020

@whedon set v0.1.2 as version

@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Jan 8, 2020

OK. v0.1.2 is the version.

@mbobra

This comment has been minimized.

Copy link
Member

@mbobra mbobra commented Jan 8, 2020

@openjournals/joss-eics This paper is accepted and ready for final processing! Nice work, @stfbnc 🎉

@danielskatz

This comment has been minimized.

Copy link

@danielskatz danielskatz commented Jan 8, 2020

Thanks @mbobra - just as a minor correction, let's call it "ready-to-accept" until the final processing is done :)

@danielskatz

This comment has been minimized.

Copy link

@danielskatz danielskatz commented Jan 8, 2020

@whedon accept

@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Jan 8, 2020

Attempting dry run of processing paper acceptance...
@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Jan 8, 2020

Reference check summary:

OK DOIs

- http://dx.doi.org/10.1016/j.physa.2010.10.022 is OK
- http://dx.doi.org/10.1103/PhysRevLett.100.084102 is OK
- http://dx.doi.org/10.3389/fphys.2012.00141 is OK
- http://dx.doi.org/10.1016/S0378-4371(02)01383-3 is OK
- http://dx.doi.org/10.1103/PhysRevE.69.056107 is OK
- http://dx.doi.org/10.1103/PhysRevE.60.1390 is OK
- http://dx.doi.org/10.1103/PhysRevLett.90.108501 is OK
- https://doi.org/10.5194/acp-7-629-2007 is OK
- http://dx.doi.org/10.1103/PhysRevE.47.3730 is OK

MISSING DOIs

- None

INVALID DOIs

- None
@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Jan 8, 2020

Check final proof 👉 openjournals/joss-papers#1209

If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#1209, then you can now move forward with accepting the submission by compiling again with the flag deposit=true e.g.

@whedon accept deposit=true
@danielskatz

This comment has been minimized.

Copy link

@danielskatz danielskatz commented Jan 8, 2020

👋 @stfbnc - please merge stfbnc/fathon#6 or let me know what parts you disagree with

@stfbnc

This comment has been minimized.

Copy link

@stfbnc stfbnc commented Jan 8, 2020

@danielskatz Thanks for the corrections, I've merged your pull request.

@danielskatz

This comment has been minimized.

Copy link

@danielskatz danielskatz commented Jan 9, 2020

@whedon accept

@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Jan 9, 2020

Attempting dry run of processing paper acceptance...
@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Jan 9, 2020

Reference check summary:

OK DOIs

- http://dx.doi.org/10.1016/j.physa.2010.10.022 is OK
- http://dx.doi.org/10.1103/PhysRevLett.100.084102 is OK
- http://dx.doi.org/10.3389/fphys.2012.00141 is OK
- http://dx.doi.org/10.1016/S0378-4371(02)01383-3 is OK
- http://dx.doi.org/10.1103/PhysRevE.69.056107 is OK
- http://dx.doi.org/10.1103/PhysRevE.60.1390 is OK
- http://dx.doi.org/10.1103/PhysRevLett.90.108501 is OK
- https://doi.org/10.5194/acp-7-629-2007 is OK
- http://dx.doi.org/10.1103/PhysRevE.47.3730 is OK

MISSING DOIs

- None

INVALID DOIs

- None
@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Jan 9, 2020

Check final proof 👉 openjournals/joss-papers#1210

If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#1210, then you can now move forward with accepting the submission by compiling again with the flag deposit=true e.g.

@whedon accept deposit=true
@danielskatz

This comment has been minimized.

Copy link

@danielskatz danielskatz commented Jan 9, 2020

@whedon accept deposit=true

@whedon whedon added the accepted label Jan 9, 2020
@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Jan 9, 2020

Doing it live! Attempting automated processing of paper acceptance...
@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Jan 9, 2020

🐦🐦🐦 👉 Tweet for this paper 👈 🐦🐦🐦

@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Jan 9, 2020

🚨🚨🚨 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#1211
  2. Wait a couple of minutes to verify that the paper DOI resolves https://doi.org/10.21105/joss.01828
  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...

@danielskatz

This comment has been minimized.

Copy link

@danielskatz danielskatz commented Jan 9, 2020

Thanks to @ali-ramadhan & @ashwinvis for reviewing, and @mbobra for editing!
And congratulations to @stfbnc!!

@danielskatz danielskatz closed this Jan 9, 2020
@whedon

This comment has been minimized.

Copy link
Collaborator Author

@whedon whedon commented Jan 9, 2020

🎉🎉🎉 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.01828/status.svg)](https://doi.org/10.21105/joss.01828)

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

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

This is how it will look in your documentation:

DOI

We need your help!

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:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants
You can’t perform that action at this time.