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

NJOEntropy logic #3

Closed
peterbartos opened this issue Jan 19, 2017 · 1 comment
Closed

NJOEntropy logic #3

peterbartos opened this issue Jan 19, 2017 · 1 comment

Comments

@peterbartos
Copy link

It looks that your NJOEntropy logic is different than the original logic from the mattt/Navajo project.

You are increasing the sizeOfCharacterSet by 26 by every lower-case character, meanwhile the original project only once, on first lower-case character.

Is this change on purpose?
@jasonnam
Copy link
Owner

Hey Peter! Thank you for pointing out that issue. This is not intended and very serious issue. I just pushed update and sorry for your inconveniences.
This is why we work together! Thanks again.

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
@jasonnam @peterbartos