Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rcparam['axes.grid']=True != axes.grid(True) ? #2109

Closed
Carreau opened this issue Jun 1, 2013 · 1 comment
Closed

rcparam['axes.grid']=True != axes.grid(True) ? #2109

Carreau opened this issue Jun 1, 2013 · 1 comment

Comments

@Carreau
Copy link
Contributor

Carreau commented Jun 1, 2013

Hi,

To follow a question on list, I think there is a inconsistency in the rcParameters axes.grid.

In short, rcparam['axes.grid']=True draw grid on which='both',
whereas ax.grid(True) Draw grid only on which='major'.

This was probably not seen before, because by default graph have NullLocator for 'minor', but this affect default loglog graph for example.

Cf example below and full code on nbviewer:
http://nbviewer.ipython.org/5690246

I tried to narrow down where this happend in the code but havent found (yet), I'm not really familiar with matplotlib codebase. (searching to understand lib/axes.py and lib/axis.pyfor now)

If I find the problem and write a fix I'll do a PR. can I/should I introduce in the same pr a new rcParam axes.grid.which which can take the value 'both|major|minor' ?

Sorry if the vocabulary is not exact, I hope I made myself understandable.

With grid(True)
png-1

with Rcparams
png

@dmcdougall
Copy link
Member

Closed by #2110.

@Carreau Please feel free to reopen if there's more work to be done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants