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

new algorithm for id generation using parametrisation #6

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

new algorithm for id generation using parametrisation #6

wants to merge 4 commits into from

Conversation

gyounes
Copy link
Contributor

@gyounes gyounes commented Dec 13, 2016

the j (number of jumps) parameter allows a more efficient behaviour.

  • increasing j to a certain point (relatively to the number of ids generated) gives the optimal behaviour
  • j should be at least 2 (maybe can be fixed, i will check)

@CBaquero
Copy link
Owner

This is a big modification, so maybe we should have more than one algorithm in the code. I will check with you how this one works first.

@gyounes
Copy link
Contributor Author

gyounes commented Dec 14, 2016

I can be there when you want to check it if you prefer. I dont know if having two or more ... is better and how but there is definitely room for improvements.

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

Successfully merging this pull request may close these issues.

2 participants