On Fri, Nov 16, 2012 at 7:44 AM, Mike Kaufman <mck...@gmail.com> wrote:
> On 11/15/12 2:54 PM, Paul Ivanov wrote:
> > On Wed, Nov 14, 2012 at 7:37 PM, Mike Kaufman <mck...@gmail.com
>
> >
> > I just found that *direction* accepts 'inout' as well, which
> > does indeed place the tick on both sides of the spine. So the
> > documentation should be updated to reflect this.
> >
> >
> > Thanks for the report, Mike, here's a PR for the patch:
> > https://github.com/matplotlib/matplotlib/pull/1503
> >
> > If it were me, I'd allow 'both' to work as well.
> >
> >
> > I'm amenable to that, just not sure if that counts as a new feature and
> > should go into master, or a bugfix and go into v1.2.x.
> >
> > I can add this functionality to #1503 if that makes sense to go to v1.2.x
>
> Thanks Paul,
>
> One is a doc fix, the other requires changing code (inside axis.py?). So
> best practice probably argues that 'both', if acceptable, should really
> only go into master.
>
> That said, I think the 'both' change only requires a change to a pair of
> conditionals...
>
> M
>
>
I agree with the fact that implementing "both" to mean the same as "inout"
should go into master only.
Ben Root
------------------------------------------------------------------------------
Monitor your physical, virtual and cloud infrastructure from a single
web console. Get in-depth insight into apps, servers, databases, vmware,
SAP, cloud infrastructure, etc. Download 30-day Free Trial.
Pricing starts from $795 for 25 servers or applications!
http://p.sf.net/sfu/zoho_dev2dev_nov
_______________________________________________
Matplotlib-devel mailing list
Matplotlib-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
