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

[Tweak] IcoSphereMesh should have a limit on Subdivision #1206

Closed
Aerizeon opened this issue Oct 26, 2020 · 1 comment
Closed

[Tweak] IcoSphereMesh should have a limit on Subdivision #1206

Aerizeon opened this issue Oct 26, 2020 · 1 comment
Labels
Tweak Minor tweak

Comments

@Aerizeon
Copy link

The IcoSphereMesh currently does not appear to have a limit on the SubDivisions input.
This can lead to unanticipated results when edited by new users, as the number of triangles increases at a rate of 20×4n, where n is the number of subdivisions.
For example:
Subdivision 1: 80 Tris
Subdivision 2: 320 Tris
[...]
Subdivision 8: 1,310,720 Tris
Subdivision 9: 5,242,880 Tris
Subdivision 10: 20,971,520 Tris

Through testing with Ukilop and ShadowPanther, we think that 8 is probably the largest useful value, with 9 being extremely intensive to generate and render.

@Frooxius
Copy link
Collaborator

Yeah 8 sounds good, I've added the clamp for now in 2020.10.26.1103. Thanks!

@shiftyscales shiftyscales added the Tweak Minor tweak label Jan 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tweak Minor tweak
Projects
None yet
Development

No branches or pull requests

3 participants