tick_params direction 'inout'

Hi all,

I don't have time to make a patch at the moment, but I thought I'd point it out for anyone to give it a go...

tick_params(axis='both', **kwargs)
     Change the appearance of ticks and tick labels.

     Keyword arguments:

...

     *direction* : ['in' | 'out']
         Puts ticks inside or outside the axes.

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. If it were me, I'd allow 'both' to work as well.

M

Hi all,

I don't have time to make a patch at the moment, but I thought I'd point
it out for anyone to give it a go...

tick_params(axis='both', **kwargs)
     Change the appearance of ticks and tick labels.

     Keyword arguments:

...

     *direction* : ['in' | 'out']
         Puts ticks inside or outside the axes.

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:

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

···

On Wed, Nov 14, 2012 at 7:37 PM, Mike Kaufman <mckauf@...149...> wrote:

--
Paul Ivanov
314 address only used for lists, off-list direct email at:
http://pirsquared.org | GPG/PGP key id: 0x0F3E28F7

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

···

On 11/15/12 2:54 PM, Paul Ivanov wrote:

On Wed, Nov 14, 2012 at 7:37 PM, Mike Kaufman <mckauf@...149...

    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:
DOC: 'inout' option for tick_params direction by ivanov · Pull Request #1503 · matplotlib/matplotlib · GitHub

    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

I agree with the fact that implementing “both” to mean the same as “inout” should go into master only.

Ben Root

···

On Fri, Nov 16, 2012 at 7:44 AM, Mike Kaufman <mckauf@…149…> wrote:

On 11/15/12 2:54 PM, Paul Ivanov wrote:

On Wed, Nov 14, 2012 at 7:37 PM, Mike Kaufman <mckauf@…149…

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