Fix Entropy Test #697
Merged
Fix Entropy Test #697
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
chia7712
reviewed
Sep 11, 2022
|
我忘記在上面說明了，上面的運行結果就是測試10000次的結果 |
chia7712
approved these changes
Sep 11, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#677
修正bug
原来的测试是根据数据在范围内的变化验证entropy是否在正常工作。
在图中范围内撒一百个点，例左边上下范围比右边大所以点落的就会更分散，点分散代表事件的随机程度更大。因此左边会比右边entropy更大。
但是在极少数的情况中，哪怕范围比较大，落的点可能恰好会比较集中，这时候机会变成右边比左边entropy。
想要杜绝的话就直接设置固定值，这样也能看作事件发生概率的比较（一直发生同一件事）