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

Limit the number of groups #85

Open
privefl opened this issue Oct 6, 2021 · 1 comment
Open

Limit the number of groups #85

privefl opened this issue Oct 6, 2021 · 1 comment

Comments

@privefl
Copy link
Contributor

privefl commented Oct 6, 2021

It was a bit foolish of me, but I quickly tried running eulerr::euler() with 45 groups.
This resulted in a crash. I guess the number of combinations scales very fast with the number of groups.

Maybe it would be a good idea to set a max number of groups (maybe as a parameter), and errors if the user inadvertently provides more.

@jolars
Copy link
Owner

jolars commented Oct 6, 2021

Hm, you're right, some kind of check would probably be reasonable.

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

2 participants