Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.
Sign up[REVIEW]: HRV: a Pythonic package for Heart Rate Variability Analysis #1867
Comments
This comment has been minimized.
This comment has been minimized.
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @hooman650, @PGomes92, @paulvangentcom it looks like you're currently assigned to review this paper 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:
For a list of things I can do to help you, just type:
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
|
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@hooman650, @PGomes92, @paulvangentcom thanks for agreeing to review this work. You have instructions and check-boxes at the top of this issue to guide you through the process. Let me know if you have questions. |
This comment has been minimized.
This comment has been minimized.
@hooman650, @PGomes92, @paulvangentcom thanks for your help in reviewing this work! |
This comment has been minimized.
This comment has been minimized.
I have opened an issue in the toolbox repository. There are many HRV toolboxes available that exactly perform the same computations (time, frequency and nonlinearity analysis). I recommend adding a few subroutines for preprocessing RR intervals and removing artifacts. I have opened an issue and made a few recommendations there. Additionally, contribution guidelines are missing. |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
Thanks for the reminder. I'll wrap up the review tomorrow. |
whedon commentedNov 6, 2019
•
edited by hooman650
Submitting author: @rhenanbartels (Rhenan Bartels)
Repository: https://github.com/rhenanbartels/hrv
Version: 0.2.4
Editor: @Kevin-Mattheus-Moerman
Reviewer: @hooman650, @PGomes92, @paulvangentcom
Archive: Pending
Status
Status badge code:
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
@hooman650 & @PGomes92 & @paulvangentcom, please carry out your review in this issue by updating the checklist below. If you cannot edit the checklist please:
The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @Kevin-Mattheus-Moerman know.
Review checklist for @hooman650
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @PGomes92
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @paulvangentcom
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper