PEP8 compliance on the delaunay module #1518

Merged
merged 1 commit into from Nov 20, 2012

Projects

None yet

3 participants

@NelleV
Collaborator
NelleV commented Nov 19, 2012

Here is a pep8 compliance PR on the delaunay module.

Cheers,
N

@pelson
Member
pelson commented Nov 19, 2012

👍 - I would be happy to merge this in a couple of days.

@ianthomas23 ianthomas23 and 1 other commented on an outdated diff Nov 19, 2012
lib/matplotlib/delaunay/interpolate.py
triangulation, one would only have to alter those triangles and Voronoi
- polygons. The new Voronooi diagram would have a polygon around the inserted
- point. This polygon would "steal" area from the original Voronoi polygons.
- For each node i in the natural neighbors set, we compute the area stolen
- from its original Voronoi polygon, stolen[i]. We define the natural
- neighbors coordinates
+ polygons. The new Voronooi diagram would have a polygon around the
@ianthomas23
ianthomas23 Nov 19, 2012 Matplotlib Developers member

Typo: Voronooi should be Voronoi.

@NelleV
NelleV Nov 19, 2012 collaborator

Good catch!

I fixed it in my last push.

@ianthomas23 ianthomas23 merged commit b004914 into matplotlib:master Nov 20, 2012

1 check passed

Details default The Travis build passed
@pelson
Member
pelson commented Nov 20, 2012

Thanks for merging @ianthomas23 .

Good stuff again @NelleV .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment