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

Allow some variation in differential entropy analytical tests #214

Closed
kahaaga opened this issue Dec 22, 2022 · 0 comments
Closed

Allow some variation in differential entropy analytical tests #214

kahaaga opened this issue Dec 22, 2022 · 0 comments
Assignees
Labels
tests Related with tests
Milestone

Comments

@kahaaga
Copy link
Member

kahaaga commented Dec 22, 2022

Test for differential entropy estimators occasionally fail on CI. Because we're using random data from certain probability distributions to estimate the true entropy, the estimates may not be entirely accurate. Currently, it is just tested whether the estimated entropy is equal to the analytical entropy (rounded to two decimal points). However, we should allow some more slack (e.g. 5% or something), so tests don't randomly fail.

This happened for example in #213 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests Related with tests
Projects
None yet
Development

No branches or pull requests

1 participant