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.

Sign up for GitHub

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

Jump to bottom

Bug in _DispEn2D.py #8

Closed
YingFan-Chen opened this issue Oct 29, 2023 · 2 comments
Closed

Bug in _DispEn2D.py #8

YingFan-Chen opened this issue Oct 29, 2023 · 2 comments

Comments

@YingFan-Chen
Copy link

image

I think 'Mat' should be replaced by 'Zi' in line 127 of above screenshot, cause 'Zi' matrix is the one after mapping in dispersion entropy.
@MattWillFlood
Copy link
Owner

Dear @YingFan-Chen

Thank you so much for spotting this bug!
This is very helpful :)

This correction will be included in the next package release which is due very soon!

@MattWillFlood MattWillFlood reopened this Oct 29, 2023
@MattWillFlood MattWillFlood mentioned this issue Mar 24, 2024
Closed
@MattWillFlood
Copy link
Owner

This issue has been fixed in the latest release (v1.0) :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@MattWillFlood @YingFan-Chen