Re: [matplotlib-devel] partial ring patch

2008-11-14 Thread Paul Kienzle
On Nov 14, 2008, at 12:36 PM, Paul Kienzle wrote: >> >> Any reason not to implement this simply as an additional kwarg to >> Wedge, rather than a new class -- since Ring with a r2 == 0 is >> equivalent to Wedge anyway? Just thinking of having less code to >> maintain...but maybe that's more confu

Re: [matplotlib-devel] partial ring patch

2008-11-14 Thread Michael Droettboom
Paul Kienzle wrote: > > On Nov 14, 2008, at 9:59 AM, Michael Droettboom wrote: > >> Paul Kienzle wrote: >>> Hi, >>> >>> We found we needed to draw a partial ring, but didn't see one in >>> patches.py. >>> >>> Attached is a generalization of Wedge to accept an inner and an >>> outer radius. >>> >>

Re: [matplotlib-devel] partial ring patch

2008-11-14 Thread Paul Kienzle
On Nov 14, 2008, at 9:59 AM, Michael Droettboom wrote: > Paul Kienzle wrote: >> Hi, >> >> We found we needed to draw a partial ring, but didn't see one in >> patches.py. >> >> Attached is a generalization of Wedge to accept an inner and an >> outer radius. >> >> Should I add this to patches?

Re: [matplotlib-devel] partial ring patch

2008-11-14 Thread Michael Droettboom
Paul Kienzle wrote: > Hi, > > We found we needed to draw a partial ring, but didn't see one in > patches.py. > > Attached is a generalization of Wedge to accept an inner and an outer > radius. > > Should I add this to patches? Looks like a useful feature to me. Could be used for doing these sort