search

jolars / eulerr

Area-Proportional Euler and Venn Diagrams with Ellipses
https://jolars.github.io/eulerr/
GNU General Public License v3.0
129 stars 18 forks source link

Limit the number of groups #85

Open privefl opened 2 years ago

privefl commented 2 years ago

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 commented 2 years ago

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