X-Git-Url: https://gerrit.simantics.org/r/gitweb?a=blobdiff_plain;f=org.simantics.plant3d%2Fsrc%2Forg%2Fsimantics%2Fplant3d%2Fscenegraph%2Fcontrolpoint%2FPipeControlPoint.java;h=9e9beb9c37711e63012c006b4221839de8cfb819;hb=68314736ee2b8da184a5e01e88b719b4fd64ab49;hp=2905d26c4493058b1483705c8a2df8e68a9ba209;hpb=9e624e8e768f81e8c5edd665ac0bce6717013d5d;p=simantics%2F3d.git

diff --git a/org.simantics.plant3d/src/org/simantics/plant3d/scenegraph/controlpoint/PipeControlPoint.java b/org.simantics.plant3d/src/org/simantics/plant3d/scenegraph/controlpoint/PipeControlPoint.java
index 2905d26c..9e9beb9c 100644
--- a/org.simantics.plant3d/src/org/simantics/plant3d/scenegraph/controlpoint/PipeControlPoint.java
+++ b/org.simantics.plant3d/src/org/simantics/plant3d/scenegraph/controlpoint/PipeControlPoint.java
@@ -15,6 +15,7 @@ import javax.vecmath.Vector3d;
 import org.simantics.g3d.math.MathTools;
 import org.simantics.g3d.property.annotations.GetPropertyValue;
 import org.simantics.g3d.scenegraph.G3DNode;
+import org.simantics.g3d.scenegraph.base.INode;
 import org.simantics.plant3d.scenegraph.IP3DNode;
 import org.simantics.plant3d.scenegraph.Nozzle;
 import org.simantics.plant3d.scenegraph.P3DRootNode;
@@ -581,15 +582,29 @@ public class PipeControlPoint extends G3DNode implements IP3DNode {
 		if (dir == null || dir.lengthSquared() < MathTools.NEAR_ZERO)
 			return MathTools.getIdentityQuat();
 
-		Vector3d up = new Vector3d(0.0, 1.0, 0.0);
-		double a = up.angle(getPathLegEndpointVector());
+		final P3DRootNode root = getRoot();
+		Vector3d up = root != null ? new Vector3d(root.getUpVector()) : new Vector3d(0.0, 1.0, 0.0);
+		final Vector3d legDir = getPathLegEndpointVector();
+		double a = up.angle(legDir);
 		if (a < 0.1 || (Math.PI - a) < 0.1) {
-			up.set(1.0, 0.0, 0.0);
+			// Rotate components
+			up.set(up.getY(), up.getZ(), up.getX());
 		}
+		
+		// Project up vector into a normal of the leg direction before applying to 'dir'
+		MathTools.mad(up, legDir, -legDir.dot(up)/legDir.lengthSquared());
+		up.normalize();
 
 		return getControlPointOrientationQuat(dir, up, angle);
 	}
 
+	public P3DRootNode getRoot() {
+		INode n = getParent();
+		while (n != null && !(n instanceof P3DRootNode))
+			n = n.getParent();
+		return (P3DRootNode) n;
+	}
+
 	public static Quat4d getControlPointOrientationQuat(Vector3d dir, Vector3d up,  double angle) {
 		if (dir == null || dir.lengthSquared() < MathTools.NEAR_ZERO)
 			return MathTools.getIdentityQuat();
@@ -600,7 +615,8 @@ public class PipeControlPoint extends G3DNode implements IP3DNode {
 
 
 		Vector3d right = new Vector3d();
-
+		
+		up = new Vector3d(up);
 		right.cross(dir, up);
 		up.cross(right, dir);
 		right.normalize();