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

Two fixes to avoid matplotlib osx backend issues #546

Closed
wants to merge 2 commits into from

Conversation

mwaskom
Copy link
Owner

@mwaskom mwaskom commented May 9, 2015

This changes some things in FacetGrid and heatmap to avoid two common and very annoying issues caused by the the matplotlib osx backend.

I believe this is the root matplotlib issue: matplotlib/matplotlib#2654

This has come up a few times in seaborn: #545, #408, #231, and elsewhere.

The fix to FacetGrid should, I believe, simply make that work. The fix to heatmap is a compromise where the exception that is raised when trying to determine if the ticklabels overlap gets caught and the labels are not rotated. That means that the plots might not look as good for everyone, but this feels better than the plot simply not working.

I haven't tested this extensively, but I seem to be able to plot using the OSX backend without running into these common issues.

@mwaskom
Copy link
Owner Author

mwaskom commented May 9, 2015

Unfortunately this causes issues with the Grid.add_legend method. Not sure what the best workaround is.

@mwaskom
Copy link
Owner Author

mwaskom commented May 9, 2015

Hm, also there is some weird issue such that the legend does not immediately appear when the gui window pops up, but it does show up if the window is resized.

@mwaskom
Copy link
Owner Author

mwaskom commented Jun 28, 2015

Closing in favor of #611

@mwaskom mwaskom closed this Jun 28, 2015
@mwaskom mwaskom deleted the fix_macosx_issues branch June 28, 2015 21:23
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

Successfully merging this pull request may close these issues.

None yet

1 participant