On Sun, Jan 10, 2010 at 1:55 PM, Alvin Penner <penner@...1856...> wrote:

This is a request for comment on a proposed extension to generate Voronoi Diagrams as a pattern. This is a follow-up to LP Bug 171901. https://bugs.launchpad.net/inkscape/+bug/171901

1. I could easily crash it with 10px size:

Traceback (most recent call last):

File "C:\inkscape\inkscape\share\extensions\generate_voronoi.py", line 144, in <module>

e.affect()

File "c:\inkscape\inkscape\share\extensions\inkex.py", line 208, in affect

self.effect()

File "C:\inkscape\inkscape\share\extensions\generate_voronoi.py", line 106, in effect

voronoi.voronoi(sl, c)

File "C:\inkscape\inkscape\share\extensions\voronoi.py", line 200, in voronoi

edge = Edge.bisect(bot,newsite)

File "C:\inkscape\inkscape\share\extensions\voronoi.py", line 397, in bisect

newedge.a = dx/dy

ZeroDivisionError: float division

Can we make it more stable?

2. What's the use of making it a pattern? If you enlarge the path to reveal more of the pattern, the edges of the pattern tile won't match. This would make sense only if they matched, otherwise I think it should create subpaths of the current path, or a new path doing the subdivision.

3. It would make a lot more visual sense if it would coordinate with the nodes of the original path, by placing the nodes of the grid in these nodes. This way you won't even need the Size control because the size will depend on the density of nodes of the original path, and can be different in different parts of the fill. That will be much more useful e.g. to generate something as lifelike as scales on a turtle's legs.