fix(geom-closest-point): flip sign of plane W component, extract dist…

…ToPlane()
thi-ng · May 22, 2019 · 74dbcb0 · 74dbcb0
1 parent 3f085a3
commit 74dbcb0
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/packages/geom-closest-point/src/index.ts b/packages/geom-closest-point/src/index.ts
@@ -198,15 +198,15 @@ export const closestPointArray = (p: ReadonlyVec, pts: Vec[]) => {
     return closest;
 };
 
+export const distToPlane = (p: ReadonlyVec, n: ReadonlyVec, w: number) =>
+    dot(n, p) - w;
+
 export const closestPointPlane = (
     p: ReadonlyVec,
     normal: ReadonlyVec,
     w: number,
     out: Vec = []
-) => {
-    out = normalize(out, normal, dot(p, normal) + w);
-    return sub(out, p, out);
-};
+) => sub(out, p, normalize(out, normal, distToPlane(p, normal, w)));
 
 export const closestPointCircle = (
     p: ReadonlyVec,