X Tutup
Skip to content

ticks based on number of subplots#776

Closed
ivanov wants to merge 2 commits intomatplotlib:masterfrom
ivanov:subplots-with-tickadjustments
Closed

ticks based on number of subplots#776
ivanov wants to merge 2 commits intomatplotlib:masterfrom
ivanov:subplots-with-tickadjustments

Conversation

@ivanov
Copy link
Copy Markdown
Member

@ivanov ivanov commented Mar 20, 2012

here's an implementation of adjusting the tick locator depending on how many subplots there are. I implemented this at PyData a few weeks back, but forgot to submit the PR.

to see the goodies of this PR, use this in the ipython notebook

%pylab inline
#rcParams['figure.figsize']=2,2
plt.subplots()
plt.plot(24*np.random.rand(10), np.random.rand(10))
for x in range(7):
    fig, axes = plt.subplots(1,x+2, sharey=True)
    for ax in axes.flat: ax.plot(24*np.random.rand(10), np.random.rand(10))

plt.subplots()
plt.plot(np.random.rand(10), 24*np.random.rand(10))

for x in range(7):
    fig, axes = plt.subplots(x+2,1, sharex=True)
    for ax in axes.flat: ax.plot(np.random.rand(10), 24*np.random.rand(10))
#fig.tight_layout()

(I'll rebase this if this won't merge cleanly)

jdh2358
jdh2358 reviewed Mar 20, 2012
View reviewed changes
lib/matplotlib/pyplot.py Outdated
Copy link
Copy Markdown
Collaborator

@jdh2358 jdh2358 Mar 20, 2012

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like you left a print in here

@jdh2358
Copy link
Copy Markdown
Collaborator

jdh2358 commented Mar 20, 2012

Hey Paul,

This is a very useful PR, because the too many ticks problem is a common annoyance. I think "subplots" is the perfect place to add enhancements like these because a) it knows the whole geometry and b) it is still somewhat lightly used and so less likely to change/break people's existing code.

I would like to do more: in particular, I think we should expose a tight_layout=True kwarg to subplots so that tight layout is on by default (but configurable), Can you add this to the PR?

ivanov added 2 commits March 20, 2012 15:10
@ivanov
WIP on ticks based on number of subplots
3b9c2e0
@ivanov
finished ticks based on number of subplots
f03bdbb 
We don't need to adjust every locator when either x- or y-axis is shared
across all of the subplots.
@ivanov
Copy link
Copy Markdown
Member Author

ivanov commented Mar 21, 2012

Hi John,

I removed the print statements, and adjusted the logic for sharex and sharey, since locators get reused in that case.

as for tight_layout, I think with the current implementation, it only make sense to call tight_layout after whatever plotting will happen, since it's a one-time action which only takes into account the current state of the relevant artists. I hope @leejjoon or anyone else corrects me if I'm wrong.

@leejjoon
Copy link
Copy Markdown
Contributor

leejjoon commented Mar 21, 2012

Yes, tight_layout is a one-time action. Although we may implement some automatic use, see #774.

@jdh2358
Copy link
Copy Markdown
Collaborator

jdh2358 commented Mar 25, 2012

Hey Paul,

just wanted to point out that we are in the thick handling the v1.1.x release right now so are in a holding pattern on master PRs, but I am definitely interested in this so make sure to ping us if we fall quite on this PR after we get 1.1.1 out.

JDH

@ivanov ivanov mentioned this pull request Mar 27, 2012
Closed
@pelson
Copy link
Copy Markdown
Member

pelson commented Nov 3, 2012

@ivanov : I think we can start looking at this again now that (for the most part) v1.2.0 is out of the way. Would you be willing to rebase this, and add appropriate tests and we can get this PR back on the agenda.

Cheers,

@ivanov ivanov closed this Dec 8, 2012
@pelson
Copy link
Copy Markdown
Member

pelson commented Dec 10, 2012

@ivanov is that a no, or a "I'm on it"? 😄

@ivanov
Copy link
Copy Markdown
Member Author

ivanov commented Dec 11, 2012

both :)

I took a stab and there were too many conflicts immediately, and I didn't
have the mental bandwidth to juggle them at the time, let me take another
stab...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ivanov @jdh2358 @leejjoon @pelson
X Tutup