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

scatter_matrix: coordinate misalignment among diagonal and off-diagonal plots #5497

Closed
acorbe opened this issue Nov 12, 2013 · 3 comments
Closed
Labels
Milestone

Comments

@acorbe
Copy link
Contributor

acorbe commented Nov 12, 2013

I make extensive use of scatter_matrix and I find misalignments in the coordinates of the scatter plots (off-diagonal) and the histograms (on-diagonal) which seem undesirable to me.

Consider the picture example
).

In this case, there is a clear misalignment (of about .02) in the x-coordinate between the subplot in position 1,2 and the subplot in position 2,2.
In other words, the peeks in subplots 1,2 and 2,2 should occur at the same x location
(at least, IMHO, this would be more meaningful behavior).

@TomAugspurger
Copy link
Contributor

Can you share the data that generated the plot? (or some fake data that gives a similar picture)

@acorbe
Copy link
Contributor Author

acorbe commented Nov 12, 2013

This gives the problem to me

 a = - np.linspace(.4,.6,5000) + np.random.randn(5000)
 b = np.linspace(.9,1,5000) + .1*np.random.randn(5000)
 db = pd.DataFrame(zip(a,b),columns=list('ab'))
 pd.scatter_matrix(db)   

example2

It seems that the x-coordinate increment is different in subplot 1,1 and subplot 2,1.
The offset (x_11 - x_21) ranges from 0 (at the center of the quadrants) to almost 1 (in this case) at the boundary.
The x-values are hand taken from the status bar.

@jreback
Copy link
Contributor

jreback commented Jan 21, 2014

closed by 2f24ff2

@jreback jreback closed this as completed Jan 21, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants