-
Notifications
You must be signed in to change notification settings - Fork 1.6k
refresh on shannon-fano algorithm and added tests #68
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
Conversation
Codecov Report
@@ Coverage Diff @@
## master #68 +/- ##
==========================================
+ Coverage 31.49% 38.07% +6.58%
==========================================
Files 33 34 +1
Lines 1594 1770 +176
Branches 247 272 +25
==========================================
+ Hits 502 674 +172
- Misses 1085 1088 +3
- Partials 7 8 +1
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will approve and merge now and check tests later
* Refresh shannon-fano algorithm and add tests * Reduce shannon-fano complexity
No description provided.