New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixed hatch clipping. #2003
Fixed hatch clipping. #2003
Conversation
Looks good. I'll merge once Travis has had a chance to do its thing. |
Don't overwrite unit_circle vertices in clipping test
points = [(0.5,0.5), (1.5,0.5)] | ||
|
||
assert np.all(path.contains_points(points) == [True, False]) | ||
|
||
|
||
@image_comparison(baseline_images=["clip_path_clipping"], remove_text=True) | ||
def test_clipping(): | ||
exterior = mpath.Path.unit_rectangle() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is against v1.2.x where #2010 is against master. I think it needs to be the other way around...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed. So we can merge this, merge #2010, merge 1.2.x over to master, and then make my suggested change as part of that. (It doesn't really matter, but that feels like the path of least resistance...)
@mdboom - I don't think there is anything left for me to do in this PR. Is it good for merge? |
Replaces #2001