-
-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
chore(TS): types and tweaks for intersection file #8603
Conversation
Build Stats
|
I thought I did this already... |
const b2xb1x = b2.x - b1.x, | ||
a1yb1y = a1.y - b1.y, | ||
b2yb1y = b2.y - b1.y, | ||
a1xb1x = a1.x - b1.x, | ||
a2ya1y = a2.y - a1.y, | ||
a2xa1x = a2.x - a1.x, | ||
uaT = b2xb1x * a1yb1y - b2yb1y * a1xb1x, | ||
ubT = a2xa1x * a1yb1y - a2ya1y * a1xb1x, | ||
uB = b2yb1y * a2xa1x - b2xb1x * a2ya1y; |
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.
I am guessing all of this can be reduced by using dotProduct
crossProduct
but it is a headache
Don't you prefect subtracting points and working with vectors?
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.
I can do that as a fun thing when the rest is done, and only if i save some bytes, otherwise i see it as non useful
@ShaMan123 anything holding this PR? |
Go ahead |
Motivation
Finalize the migration for this file
Changes
changed Intersection in static methods with thistook back this,