Thanks. This has been committed to the SVN trunk r5502. Cheers, Mike
Michael Droettboom wrote: > Looks like a nice solution to me. I don't use pie charts much myself, > so I'll give it some time to see if there are any dissenters. If none, > I'll go ahead and commit this to SVN. > > Thanks! > Mike > > Benoit Hirbec wrote: > >> Hello, >> >> Pie charts are very useful, but when wedges labels are too long, the >> label text overlaps the pie wedge, resulting in a poor readability >> (see the attached file 'before_patch.pdf'). >> >> I propose to modify the file 'axes.py' in method 'pie' by replacing: >> >> xt = x + labeldistance*radius*math.cos(thetam) >> yt = y + labeldistance*radius*math.sin(thetam) >> >> t = self.text(xt, yt, label, >> size=rcParams['xtick.labelsize'], >> horizontalalignment='center', >> verticalalignment='center') >> >> texts.append(t) >> >> By: >> >> xt = x + labeldistance*radius*math.cos(thetam) >> yt = y + labeldistance*radius*math.sin(thetam) >> label_alignment = xt > 0 and 'left' or 'right' >> >> t = self.text(xt, yt, label, >> size=rcParams['xtick.labelsize'], >> horizontalalignment=label_alignment, >> verticalalignment='center') >> >> texts.append(t) >> >> Only 2 lines changed ;-) >> >> This way, labels are right aligned when placed at the left of the pie, >> and left aligned when placed on the right of the pie (instead of being >> always centered). See the attached file 'after_patch.pdf' for an >> example. >> I also give an example in the example.py file. >> >> Do you agree to commit that to the trunk? >> >> Thanks for making matplotlib, >> >> Benoit Hirbec >> Garden >> >> >> ------------------------------------------------------------------------ >> >> >> ------------------------------------------------------------------------ >> >> ------------------------------------------------------------------------ >> >> ------------------------------------------------------------------------- >> Check out the new SourceForge.net Marketplace. >> It's the best place to buy or sell services for >> just about anything Open Source. >> http://sourceforge.net/services/buy/index.php >> ------------------------------------------------------------------------ >> >> _______________________________________________ >> Matplotlib-devel mailing list >> Matplotlib-devel@lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/matplotlib-devel >> > > -- Michael Droettboom Science Software Branch Operations and Engineering Division Space Telescope Science Institute Operated by AURA for NASA ------------------------------------------------------------------------- Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://sourceforge.net/services/buy/index.php _______________________________________________ Matplotlib-devel mailing list Matplotlib-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/matplotlib-devel