Skip to content

Commit 76e6a57

Browse files
Remove inner radius for :class:.Sector (#3922)
A Sector is a special case of `AnnualarSector` where the inner radius is zero. As such, it does not make sense to also have a parameter for the inner radius. * Simplify Sector Class by Removing Redundant Parameters * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
1 parent 95966a7 commit 76e6a57

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

manim/mobject/geometry/arc.py

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -897,17 +897,15 @@ class Sector(AnnularSector):
897897
898898
class ExampleSector(Scene):
899899
def construct(self):
900-
sector = Sector(outer_radius=2, inner_radius=1)
901-
sector2 = Sector(outer_radius=2.5, inner_radius=0.8).move_to([-3, 0, 0])
900+
sector = Sector(radius=2)
901+
sector2 = Sector(radius=2.5, angle=60*DEGREES).move_to([-3, 0, 0])
902902
sector.set_color(RED)
903903
sector2.set_color(PINK)
904904
self.add(sector, sector2)
905905
"""
906906

907-
def __init__(
908-
self, outer_radius: float = 1, inner_radius: float = 0, **kwargs
909-
) -> None:
910-
super().__init__(inner_radius=inner_radius, outer_radius=outer_radius, **kwargs)
907+
def __init__(self, radius: float = 1, **kwargs) -> None:
908+
super().__init__(inner_radius=0, outer_radius=radius, **kwargs)
911909

912910

913911
class Annulus(Circle):

0 commit comments

Comments
 (0)