Skip to content

sort out if the high-resolution ellipse code still works #7491

@tacaswell

Description

@tacaswell

I am not sure that the code at https://github.com/matplotlib/matplotlib/blob/master/lib/matplotlib/patches.py#L1693 still does what it should do

Code originally went in via 7a82ea8

        for theta in thetas:
            if inside:
                self._path = Path.arc(last_theta, theta, 8)
                Patch.draw(self, renderer)
                inside = False
            else:
                inside = True
            last_theta = theta

was changed via b7479ef to

        for theta in thetas:
            if inside:
                _path = Path.arc(last_theta, theta, 8)
                Patch.draw(self, renderer)
                inside = False
            else:
                inside = True
            last_theta = theta

and changed via 1938b44 to

        for theta in thetas:
            if inside:
                Path.arc(last_theta, theta, 8)
                Patch.draw(self, renderer)
                inside = False
            else:
                inside = True
            last_theta = theta

attn @mdboom

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions