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

Tavor needs a rand.Uint64n method or the move to bigint #80

Open
zimmski opened this issue Feb 1, 2015 · 0 comments
Open

Tavor needs a rand.Uint64n method or the move to bigint #80

zimmski opened this issue Feb 1, 2015 · 0 comments

Comments

@zimmski
Copy link
Owner

zimmski commented Feb 1, 2015

The amount of permutations can get big very fast. Currently we use uint64 where we can but there is no rand.Uint64n only a rand.Int63n which often fails because of the need of a int64 conversation which converts very high numbers to negative numbers failing the call to every rand method because they want positive numbers.

Another solution would be to use bigints which would make everything easier since we can then use arbitrary large numbers.

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

No branches or pull requests

1 participant