From 085227e3e643485c4b92621968309c5acd6d9fb8 Mon Sep 17 00:00:00 2001 From: Ian Jackson Date: Sat, 30 Dec 2017 12:13:13 +0000 Subject: [PATCH] Revert "helixish: railway_inplane_basis should not include a scale factor" This reverts commit 5aba1dbd37afe0078ba09dbf6cf142b559d82d2b. OK, so we'll make the railway basis have a scale in the (x,y) plane so that Q is at [1,0,0]. This is used for debugging output, only. Signed-off-by: Ian Jackson --- helixish.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/helixish.py b/helixish.py index e49ed12..1d36fae 100644 --- a/helixish.py +++ b/helixish.py @@ -89,7 +89,7 @@ class HelixishCurve(): railway_cos_theta = cos(railway_theta) railway_cos_phi = cos(railway_phi) - railway_inplane_basis_x = unit_v(np.hstack((q_plane[0:2], [0]))) + railway_inplane_basis_x = np.hstack((q_plane[0:2], [0])) railway_inplane_basis = np.column_stack(( railway_inplane_basis_x, np.cross([0,0,1], railway_inplane_basis_x), -- 2.30.2