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

Reply via email to