On Wednesday, June 09, 2010 08:00:13 pm Eric Firing wrote:
> On 06/08/2010 11:07 PM, Matthias Michler wrote:
> > On Wednesday, June 09, 2010 11:00:31 am Eric Firing wrote:
> >> On 06/08/2010 10:48 PM, Matthias Michler wrote:
> >>> On Friday, April 23, 2010 11:08:50 am Matthias Michler wrote:
> >>>> Hello list, Hello developers,
> >>>>
> >>>> I'd like to summarize my discussion with Gökhan ("Turning off minor
> >>>> grids on log scaled plot") in the last days and propose two patches.
> >>>>
> >>>> The first patch adds the keyword argument 'which' from the axis.grid
> >>>> to the method 'grid' of the Axes
> >>>> (axes_grid_for_major_and_minor_ticks.patch). This allows to change the
> >>>> drawing of grid lines for x- and y-axis at the same time.
> >>>>
> >>>> Furthemore Gökhan proposed to toggle *all* (namely major and minor
> >>>> tick) grid lines after pressing the key 'g'. Therefore the call
> >>>> event.inaxes.grid(), which toggles only the (default) major tick grid
> >>>> lines, is replaced by event.inaxes.grid(which='majorminor') (see
> >>>> toggling_all_tick_grid_lines.patch).
> >>>> This yields the expected behavior if e.g. all (major and minor) tick
> >>>> grid lines are shown, because than toggling the grid means to remove
> >>>> all grid lines. But to be honest I'm not sure the latter is the
> >>>> intended behavior in all cases. For instance in the case of shown
> >>>> major tick lines toggling all means turning off major tick lines and
> >>>> turning on minor tick lines by pressing 'g'. This sounds a little bit
> >>>> crazy to me, although that's what toggling is about.
> >>
> >> I think that behavior would indeed drive the user crazy, so I would not
> >> want to commit a patch that does that.
> >>
> >> Eric
> >
> > Hi Eric,
> >
> > thanks a lot for your comment. What do you think about the first part
> > namely adding the kwarg 'which' from the axis method to the Axes method
> > grid?
>
> I went ahead and did it in 8402--check to see that this is what you had
> in mind. The which kwarg can be 'both', 'minor', or 'major'.
Hi Eric,
thanks a lot for this commit. This is exactly what I wanted.
There is only one thing about the new possibilities 'both', 'minor', or
'major'. The first possibility replaces 'majorminor', 'minormajor',
'some_string_including_major_and_minor', ... and I'm afraid that this could
break someone's code.
Kind regards,
Matthias
------------------------------------------------------------------------------
ThinkGeek and WIRED's GeekDad team up for the Ultimate
GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the
lucky parental unit. See the prize list and enter to win:
http://p.sf.net/sfu/thinkgeek-promo
_______________________________________________
Matplotlib-users mailing list
Matplotlib-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matplotlib-users
