Fixed call to path.Path.contains_point from pnpoly. #2020

Merged
merged 1 commit into from May 17, 2013

3 participants

@aseagram

contains_point accepts a list as its first argument. It was being called with contains_point(x,y) rather than contains_point([x,y]).

@pelson
Matplotlib Developers member

👍 for the change. Though nxutils was deprecated in v1.2.0 and may very well be deleted for v1.3.x. @mdboom is there a reason to keep it around?

@mdboom
Matplotlib Developers member

We (knock on wood) have all of the last bugs in the replacement ironed out now, so I don't think there's a good reason to keep this around. However, this is still a helpful patch and I'll manually merge it onto 1.2.x.

@mdboom mdboom merged commit 51107ea into matplotlib:master May 17, 2013

1 check passed

Details default The Travis CI build passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment