eulerr icon indicating copy to clipboard operation
eulerr copied to clipboard

Limit the number of groups

Open privefl opened this issue 2 years ago • 1 comments

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.

privefl avatar Oct 06 '21 08:10 privefl

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

jolars avatar Oct 06 '21 09:10 jolars