-
Notifications
You must be signed in to change notification settings - Fork 22
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
CreatePolygonTool upgrade #161
Conversation
sine[i] = Math.sin((i+0.5)*2.0*Math.PI/sides); | ||
sine[i] = 1e-10*Math.round(sine[i]*1e10); | ||
sine[i] = Math.sin((i+0.5)*2.0*Math.PI/sides); | ||
sine[i] = 1e-10*(Math.round(sine[i]*1e10)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only thing I've done so far: you've got a redundant pair of parentheses here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line 83
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops. I tried somethig different here, this happened at returning the original.
vertex[i].x = sine[i]*xscale; | ||
vertex[i].y = -cosine[i]*yscale; | ||
} | ||
((Mesh)(objInfo.object)).setVertexPositions(vertex); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And another here. objectInfo.object
does not need to be wrapped.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks.
I've come down on the side of consistency. I think we'll be better off with this than without. |
11d2a3a
to
ec9ea02
Compare
Fixed. |
Upgraded to work with the same logic as the primitives and Spline Mesh. Resolves #156.
Let me know if you find anything funny, the modifications were prettey heavy.