We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
_ecg_findpeaks_manikandan()
Describe the bug In the function, if the window goes outside the signal length, we try (badly) to clip the window, causing an IndexError See here
IndexError
To Reproduce Example steps to reproduce the behaviour. For instance:
signal = np.arange(5) search_window_half = 10 i = 0 highs = np.arange(i + 1, i + search_window_half + 1) highs = np.delete( highs, np.arange(np.where(highs == len(signal))[0], len(highs) + 1) # there shouldn't be a +1 here )
We could remove the +1, but why not simply change highs to:
+1
highs
highs = np.arange(i + 1, min(i + search_window_half + 1, len(signal))
and remove the if block ?
Expected behaviour Not throw an error
The text was updated successfully, but these errors were encountered:
No branches or pull requests
Describe the bug
In the function, if the window goes outside the signal length, we try (badly) to clip the window, causing an
IndexError
See here
To Reproduce
Example steps to reproduce the behaviour. For instance:
We could remove the
+1
, but why not simply changehighs
to:and remove the if block ?
Expected behaviour
Not throw an error
The text was updated successfully, but these errors were encountered: