Skip to content

Commit

Permalink
joints.py -> use new optimal keyword for bounding_box with False
Browse files Browse the repository at this point in the history
  • Loading branch information
jdegenstein authored Sep 17, 2024
1 parent 235260a commit 9d59791
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/build123d/joints.py
Original file line number Diff line number Diff line change
Expand Up @@ -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__(
Expand Down Expand Up @@ -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(
[
Expand Down Expand Up @@ -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)))
Expand Down

0 comments on commit 9d59791

Please sign in to comment.