Merge pull request #700 from jdegenstein/BBox_optimal

joints.py -> use new optimal keyword for bounding_box with False
gumyr · Sep 17, 2024 · b2cb62f · b2cb62f
2 parents 235260a + 9d59791
commit b2cb62f
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/build123d/joints.py b/src/build123d/joints.py
@@ -69,7 +69,7 @@ def location(self) -> Location:
     @property
     def symbol(self) -> Compound:
         """A CAD symbol (XYZ indicator) as bound to part"""
-        size = self.parent.bounding_box().diagonal / 12
+        size = self.parent.bounding_box(optimal=False).diagonal / 12
         return Compound.make_triad(axes_scale=size).locate(self.location)
 
     def __init__(
@@ -228,7 +228,7 @@ def location(self) -> Location:
     @property
     def symbol(self) -> Compound:
         """A CAD symbol representing the axis of rotation as bound to part"""
-        radius = self.parent.bounding_box().diagonal / 30
+        radius = self.parent.bounding_box(optimal=False).diagonal / 30
 
         return Compound(
             [
@@ -652,7 +652,7 @@ def location(self) -> Location:
     @property
     def symbol(self) -> Compound:
         """A CAD symbol representing joint as bound to part"""
-        radius = self.parent.bounding_box().diagonal / 30
+        radius = self.parent.bounding_box(optimal=False).diagonal / 30
         circle_x = Edge.make_circle(radius, self.angle_reference)
         circle_y = Edge.make_circle(radius, self.angle_reference.rotated((90, 0, 0)))
         circle_z = Edge.make_circle(radius, self.angle_reference.rotated((0, 90, 0)))