doc: Directly call CQ-editor in environment

README.md

@@ -36,7 +36,7 @@ Test the environment with `python3 -c "import nhf"`
 To visualize an object, create a file `visualize.py`, and run `cq-editor`:
 
 ``` sh
-python3 -m cq_editor visualize.py
+CQ-editor visualize.py
 ```
 
 ### Folder Structure

nhf/parts/joints.py

@@ -317,7 +317,7 @@ class TorsionJoint:
         if not rider_slot_begin:
             rider_slot_begin = self.rider_slot_begin
         def slot(loc):
-            wire = Cq.Wire.makePolygon(self._slot_polygon(flip=False))
+            wire = Cq.Wire.makePolygon(self._slot_polygon(flip=False)).close()
             face = Cq.Face.makeFromWires(wire)
             return face.located(loc)
         wall_profile = (

nhf/utils.py

@@ -8,30 +8,6 @@ from cadquery.occ_impl.solver import ConstraintSpec
 from nhf import Role
 from nhf.materials import KEY_ITEM, KEY_MATERIAL
 
-# Bug fixes
-def _subloc(self, name: str) -> Tuple[Cq.Location, str]:
-    """
-       Calculate relative location of an object in a subassembly.
-
-       Returns the relative positions as well as the name of the top assembly.
-       """
-
-    rv = Cq.Location()
-    obj = self.objects[name]
-    name_out = name
-
-    if obj not in self.children and obj is not self:
-        locs = []
-        while not obj.parent is self:
-            locs.append(obj.loc)
-            obj = cast(Cq.Assembly, obj.parent)
-            name_out = obj.name
-
-        rv = functools.reduce(lambda l1, l2: l2 * l1, locs)
-
-    return (rv, name_out)
-Cq.Assembly._subloc = _subloc
-
 ### Vector arithmetic
 
 def location_sub(self: Cq.Location, rhs: Cq.Location) -> Cq.Vector: