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

0.2.6 #863

Merged
merged 137 commits into from
Sep 12, 2023
Merged

0.2.6 #863

merged 137 commits into from
Sep 12, 2023

Conversation

DominiqueMakowski
Copy link
Member

@DominiqueMakowski DominiqueMakowski commented Jul 9, 2023

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Jul 9, 2023

Codecov Report

Patch coverage: 67.55% and project coverage change: -0.06% ⚠️

Comparison is base (366583e) 55.23% compared to head (d867214) 55.17%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #863      +/-   ##
==========================================
- Coverage   55.23%   55.17%   -0.06%     
==========================================
  Files         297      302       +5     
  Lines       13918    14140     +222     
==========================================
+ Hits         7687     7802     +115     
- Misses       6231     6338     +107     
Files Changed Coverage Δ
neurokit2/complexity/entropy_shannon.py 83.33% <ø> (ø)
neurokit2/data/read_bitalino.py 15.09% <ø> (ø)
neurokit2/hrv/hrv_nonlinear.py 66.66% <ø> (ø)
neurokit2/ppg/ppg_intervalrelated.py 88.88% <0.00%> (-2.54%) ⬇️
neurokit2/data/read_xdf.py 8.57% <8.57%> (ø)
neurokit2/ecg/ecg_clean.py 75.60% <15.38%> (-11.35%) ⬇️
neurokit2/data/database.py 18.42% <18.42%> (ø)
neurokit2/signal/signal_fixpeaks.py 72.22% <39.62%> (-0.79%) ⬇️
neurokit2/stats/rescale.py 78.57% <50.00%> (-11.43%) ⬇️
neurokit2/bio/bio_process.py 56.25% <66.66%> (-0.17%) ⬇️
... and 30 more

... and 4 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

SimonLafran and others added 20 commits July 14, 2023 14:24
See #867 (comment)

_eventrelated_addinfo in epochs/eventrelated_utils.py has these lines:

```
if "Index" in epoch.columns:
    output["Event_Onset"] = epoch.loc[np.min(np.abs(epoch.index))]["Index"]
```

np.min(np.abs(...) returns the smallest absolute time offset in seconds.
If the smallest absolute value is of a negative time offset, then it
produces a non-existent index, causing an error when indexing in
epoch.loc.

This commit replaces it with iloc[np.argmin(np.abs(...))], indexing by
integer index.
[FIX] KeyError in _eventrelated_addinfo
[Docs] ecg_process: improve documentation and readability
@codeclimate
Copy link

codeclimate bot commented Sep 7, 2023

Code Climate has analyzed commit d867214 and detected 0 issues on this pull request.

View more on Code Climate.

@DominiqueMakowski DominiqueMakowski merged commit ce33299 into master Sep 12, 2023
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants