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

[PRE REVIEW]: PyNumDiff: A Python package for numerical differentiation of noisy time-series data #3922

Closed
whedon opened this issue Nov 18, 2021 · 37 comments

Comments

@whedon
Copy link

whedon commented Nov 18, 2021

Submitting author: @florisvb (Floris van Breugel)
Repository: https://github.com/florisvb/PyNumDiff
Version: v0.0.3
Editor: @christinahedges
Reviewers: @pmli, @billtubbs
Managing EiC: Arfon Smith

⚠️ JOSS reduced service mode ⚠️

Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post.

Status

status

Status badge code:

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

Author instructions

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

@florisvb if you have any suggestions for potential reviewers then please mention them here in this thread (without tagging them with an @). In addition, this list of people have already agreed to review for JOSS and may be suitable for this submission (please start at the bottom of the list).

Editor instructions

The JOSS submission bot @whedon is here to help you find and assign reviewers and start the main review. To find out what @whedon can do for you type:

@whedon commands
@whedon
Copy link
Author

whedon commented Nov 18, 2021

Hello human, I'm @whedon, a robot that can help you with some common editorial tasks.

⚠️ JOSS reduced service mode ⚠️

Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post.

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
Copy link
Author

whedon commented Nov 18, 2021

Wordcount for paper.md is 741

@whedon
Copy link
Author

whedon commented Nov 18, 2021

PDF failed to compile for issue #3922 with the following error:

 /app/vendor/ruby-2.6.6/lib/ruby/2.6.0/psych.rb:456:in `parse': (c3fa3df322ad5171f67cce41/paper.md): did not find expected key while parsing a block mapping at line 2 column 1 (Psych::SyntaxError)
	from /app/vendor/ruby-2.6.6/lib/ruby/2.6.0/psych.rb:456:in `parse_stream'
	from /app/vendor/ruby-2.6.6/lib/ruby/2.6.0/psych.rb:390:in `parse'
	from /app/vendor/ruby-2.6.6/lib/ruby/2.6.0/psych.rb:277:in `load'
	from /app/vendor/ruby-2.6.6/lib/ruby/2.6.0/psych.rb:578:in `block in load_file'
	from /app/vendor/ruby-2.6.6/lib/ruby/2.6.0/psych.rb:577:in `open'
	from /app/vendor/ruby-2.6.6/lib/ruby/2.6.0/psych.rb:577:in `load_file'
	from /app/vendor/bundle/ruby/2.6.0/bundler/gems/whedon-c5c16aedb3d6/lib/whedon.rb:127:in `load_yaml'
	from /app/vendor/bundle/ruby/2.6.0/bundler/gems/whedon-c5c16aedb3d6/lib/whedon.rb:87:in `initialize'
	from /app/vendor/bundle/ruby/2.6.0/bundler/gems/whedon-c5c16aedb3d6/lib/whedon/processor.rb:38:in `new'
	from /app/vendor/bundle/ruby/2.6.0/bundler/gems/whedon-c5c16aedb3d6/lib/whedon/processor.rb:38:in `set_paper'
	from /app/vendor/bundle/ruby/2.6.0/bundler/gems/whedon-c5c16aedb3d6/bin/whedon:58:in `prepare'
	from /app/vendor/bundle/ruby/2.6.0/gems/thor-0.20.3/lib/thor/command.rb:27:in `run'
	from /app/vendor/bundle/ruby/2.6.0/gems/thor-0.20.3/lib/thor/invocation.rb:126:in `invoke_command'
	from /app/vendor/bundle/ruby/2.6.0/gems/thor-0.20.3/lib/thor.rb:387:in `dispatch'
	from /app/vendor/bundle/ruby/2.6.0/gems/thor-0.20.3/lib/thor/base.rb:466:in `start'
	from /app/vendor/bundle/ruby/2.6.0/bundler/gems/whedon-c5c16aedb3d6/bin/whedon:131:in `<top (required)>'
	from /app/vendor/bundle/ruby/2.6.0/bin/whedon:23:in `load'
	from /app/vendor/bundle/ruby/2.6.0/bin/whedon:23:in `<main>'

@whedon
Copy link
Author

whedon commented Nov 18, 2021

Software report (experimental):

github.com/AlDanial/cloc v 1.88  T=0.13 s (591.6 files/s, 74938.7 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
Python                          40           1216           1904           2320
Jupyter Notebook                 3              0           2989            413
Markdown                         2             51              0            182
reStructuredText                27             94             98            147
TeX                              1             14              0            131
DOS Batch                        1              8              1             26
YAML                             1              3              0             10
make                             1              4              7              9
-------------------------------------------------------------------------------
SUM:                            76           1390           4999           3238
-------------------------------------------------------------------------------


Statistical information for the repository 'b1774cd1a49cf4a3510dac50' was
gathered on 2021/11/18.
The following historical commit information, by author, was found:

Author                     Commits    Insertions      Deletions    % of changes
Floris van Breugel              48          7985            786           10.99
floris                           4             8             21            0.04
luckystarufo                    18         34640          36398           88.98

Below are the number of rows from each author that have survived and are still
intact in the current revision:

Author                     Rows      Stability          Age       % in comments
Floris van Breugel          554            6.9          6.3                4.69
Yuying                     4886          100.0          1.6               15.78

@arfon
Copy link
Member

arfon commented Nov 18, 2021

@florisvb – thanks for your submission to JOSS. It looks like the paper can't be compiled because the indentation in the YAML header for this row is off:

- name: Department of Biology, University of Washington
   index: 3

Could you take a look at fixing this please?

@florisvb
Copy link

florisvb commented Nov 18, 2021 via email

@arfon
Copy link
Member

arfon commented Nov 19, 2021

@whedon invite @christinahedges as editor

👋 @christinahedges – would you be able to take on this submission for us?

@whedon
Copy link
Author

whedon commented Nov 19, 2021

@christinahedges has been invited to edit this submission.

@christinahedges
Copy link

@whedon assign me as editor

@whedon
Copy link
Author

whedon commented Nov 19, 2021

OK, the editor is @christinahedges

@christinahedges
Copy link

@whedon generate pdf

@whedon
Copy link
Author

whedon commented Nov 19, 2021

👉📄 Download article proof 📄 View article proof on GitHub 📄 👈

@christinahedges
Copy link

@whedon check references

@whedon
Copy link
Author

whedon commented Nov 19, 2021

Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

OK DOIs

- None

MISSING DOIs

- 10.1109/access.2020.3034077 may be a valid DOI for title: Numerical differentiation of noisy data: A unifying multi-objective optimization framework
- 10.1515/9783110226157.263 may be a valid DOI for title: An introduction to total variation for image analysis
- 10.1073/pnas.1517384113 may be a valid DOI for title: Discovering governing equations from data by sparse identification of nonlinear dynamical systems
- 10.1016/j.cpc.2007.03.009 may be a valid DOI for title: Numerical differentiation of experimental data: local versus global methods
- 10.1109/msp.2011.941097 may be a valid DOI for title: What is a Savitzky-Golay filter?[lecture notes]
- 10.1021/ac60214a047 may be a valid DOI for title: Smoothing and differentiation of data by simplified least squares procedures.
- 10.1109/9780470544334.ch9 may be a valid DOI for title: A new approach to linear filtering and prediction problems
- 10.1016/j.automatica.2017.08.011 may be a valid DOI for title: Generalized Kalman smoothing: Modeling and algorithms
- 10.1016/0167-2789(92)90242-f may be a valid DOI for title: Nonlinear total variation based noise removal algorithms

INVALID DOIs

- None

@christinahedges
Copy link

Hello @florisvb, welcome to JOSS! Thanks for your submission. Taking a look at your paper submission I can see a few fixable issues that you may want to work on while I find reviewers:

  1. The references are missing DOIs (see whedon's comments above)
  2. In your submission we usually encourage a state of the field section or paragraph which references common software that is use to solve similar problems in the field. You should add some references to the tools you mention in your submission.

I will get to work finding some reviewers.

@florisvb
Copy link

florisvb commented Nov 23, 2021 via email

@christinahedges
Copy link

@whedon check references

@whedon
Copy link
Author

whedon commented Nov 24, 2021

Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

OK DOIs

- 10.1109/access.2020.3034077 is OK
- 10.1515/9783110226157.263 is OK
- 10.1073/pnas.1517384113 is OK
- 10.1016/j.cpc.2007.03.009 is OK
- 10.1109/msp.2011.941097 is OK
- 10.1021/ac60214a047 is OK
- 10.1109/9780470544334.ch9 is OK
- 10.1017/S000192400001232X is OK
- 10.1016/j.automatica.2017.08.011 is OK
- 10.1016/0167-2789(92)90242-f is OK
- 10.5402/2011/164564 is OK
- 10.1038/s41586-020-2649-2 is OK

MISSING DOIs

- None

INVALID DOIs

- https://doi.org/10.21105/joss.00530 is INVALID because of 'https://doi.org/' prefix

@christinahedges
Copy link

@florisvb, thank you for fixing your DOIs (it looks like there's one invalid DOI that is easily fixed.)

I see from your submission that you have published a paper on this topic in another journal. There may be some concern from JOSS that this is not a unique submission. Could you please justify that this submission to JOSS is unique, and separate from the previously published article? I will then discuss with the rest of the JOSS team.

@florisvb
Copy link

florisvb commented Nov 24, 2021 via email

@christinahedges
Copy link

Hi @florisvb, I spoke with the editorial team, and we agree this is acceptable. I am looking for reviewers for this work, please would you let me know your suggestions for anyone in your field who is not conflicted with this work, and able to give an impartial review.

@florisvb
Copy link

florisvb commented Nov 30, 2021 via email

@christinahedges
Copy link

@whedon assign @pmli as reviewer

@whedon
Copy link
Author

whedon commented Dec 20, 2021

OK, @pmli is now a reviewer

@christinahedges
Copy link

Hello @florisvb and @pmli, thank you for your patience in getting this review started. Unfortunately, perhaps because of the holidays, I've struggled finding a reviewer for this submission despite quite a lot of digging. @florisvb if you have any further suggestions of unconflicted reviewers please do let me know otherwise I think we will have to be patient. I will also see if I can rope in some other JOSS editors so we can get this kicked off.

Thanks again for your patience!

@florisvb
Copy link

florisvb commented Jan 18, 2022 via email

@christinahedges
Copy link

If there are people in your field you feel have used the tool in their work, but are not conflicted with you, that would be ideal. Anyone in data science and software would still be totally fine, so long as it's clear they are an expert reviewer for some aspect of the tool. I will continue to ping people, thanks for your help!

@christinahedges
Copy link

@whedon add @biletubes as reviewer

@whedon
Copy link
Author

whedon commented Jan 20, 2022

OK, @biletubes is now a reviewer

@christinahedges
Copy link

@whedon assign @pmli as reviewer

@whedon whedon assigned christinahedges and pmli and unassigned pmli and christinahedges Jan 20, 2022
@whedon
Copy link
Author

whedon commented Jan 20, 2022

OK, @pmli is now a reviewer

@christinahedges
Copy link

@whedon add @billtubbs as reviewer

@whedon
Copy link
Author

whedon commented Jan 20, 2022

OK, @billtubbs is now a reviewer

@christinahedges
Copy link

Hi @florisvb I was able to find our second reviewer who has experience in running this tool. Thanks so much @pmli and @billtubbs for joining us! I'll be kicking off the review now, which will start a new issue with a new number. You'll get a notification for the review issue, please join me over there and there will be further instructions!

@christinahedges
Copy link

@whedon start review

@whedon
Copy link
Author

whedon commented Jan 20, 2022

OK, I've started the review over in #4078.

@whedon whedon closed this as completed Jan 20, 2022
@billtubbs
Copy link

Hi @christinahedges . Just realized I asked a question to a bot (@whedon). I have a problem in the #4708 review: See this comment.

Should this checklist be clickable by me?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants