-
-
Notifications
You must be signed in to change notification settings - Fork 422
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
0.2.8 #926
0.2.8 #926
Conversation
setup.py
Outdated
@@ -30,7 +28,7 @@ def find_version(): | |||
requirements = ["numpy", "pandas", "scipy", "scikit-learn>=1.0.0", "matplotlib"] | |||
|
|||
# Optional Dependencies (only needed / downloaded for testing purposes, for instance to test against some other packages) | |||
setup_requirements = ["pytest-runner", "numpy"] | |||
setup_requirements = ["pytest-runner", "numpy", "requests"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is that alright @hoechenberger ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello, no, we need it in requirements
(only) :)
Is my current understanding
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, thanks!
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #926 +/- ##
==========================================
+ Coverage 54.86% 56.02% +1.15%
==========================================
Files 303 304 +1
Lines 14255 14367 +112
==========================================
+ Hits 7821 8049 +228
+ Misses 6434 6318 -116 ☔ View full report in Codecov by Sentry. |
[Feature] Add Fast Visibility Graph Based ECG R-peak Detector
Remove tests from pyproject
Signed-off-by: Stavros Avramidis <[email protected]>
[FIX] rr xcorr arange steps don't match
[Fix] build docs warning
sparsEDA integration
[Fix] adding sampling rate parameter to eda_intervalrelated method
…cate_channel_names [Fix] read_acqknowledge now also imports duplicated channel names
[Fix] refactor ecg_findpeaks
I think it's a clearer demonstration of a possible usecase
[Feature] stim_events_find to retrieve multi-channel stimulus events
Code Climate has analyzed commit 4cd9ec4 and detected 5 issues on this pull request. Here's the issue category breakdown:
View more on Code Climate. |
Roadmap
RSP
rsp_segment()
(similar toppg_segment()
)rsp_plot()
(same asppg_plot()
)PPG
method
argument to compute only one RSA signal. Do something with warning that will be thrown when usingbio_process()
on signals < 32 sec where it will complain that the signal is too short to compute RSA with one of these methods.Misc