Michael Droettboom wrote:
> Thanks for updating the docstring.  I actually saw this as a usability 
> bug and have come up with a patch such that polar() (et al) will 
> *replace* the current axes with a polar plot if it isn't already polar.  
> This is (from the user's perspective) similar to how, for example, a 
> histogram plot would work -- that is, you don't have to tell subplot 
> you're about to plot a histogram.
> 
> But Ryan's new docstring is not obsolete with respect to my proposed 
> change.  Both techniques will work, and in fact subplot(polar=True); 
> polar(...) will be slightly faster since it avoids creating a linear 
> axes which is subsequently thrown away.
> 
> Any argument against committing my change?

None here.  I'm +1.

> The polar theta tick formatter could be changed to call "round", I 
> guess.  Alternatively, it looks like '%0.0f' also does the right thing.  
> I think that's generally what people would want for polar plots.  This 
> change would only affect polar theta ticks, so we don't need to worry 
> about a change in behavior in standard Euclidean plots.
> 
> Ryan's workaround (to get around this numerically external to 
> matplotlib) is a good suggestion as well, but I think changing the 
> formatter may be less surprising...

I like the idea of using %0.0f.  If I don't hear any objections, I'll go 
ahead and make the change.  It definitely will make things less 
confusing for our users.

Ryan

-- 
Ryan May
Graduate Research Assistant
School of Meteorology
University of Oklahoma

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Matplotlib-users mailing list
Matplotlib-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matplotlib-users

Reply via email to