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

Enrico facet plot #853

Closed
wants to merge 86 commits into from
Closed

Conversation

josef-pkt
Copy link
Member

rebased version of Enrico's facet plot PR #692

Warning: the rebase needs to be redone and this branch will be abandoned

experimental branch (until I understand what's going on, let's see what Travis says)
(I'm not fully setup anymore to test quickly different python versions)

rebase looked good, no differences in facet plot files to original branch

some strange duplicate commit messages for mosaic plot
https://github.com/josef-pkt/statsmodels/commits/enrico_facet-plot?page=2
There was a mistake in the automatic rebase, all those changes apply to sandbox/mosaic_plots.py and its tests.
Those don't exist anymore in master.

I'm getting 15 errors and 2 failures on python 2.6, largely patsy formula parsing errors.

  • Most tests are disabled with @nottest,
  • tests include show that pops up plots
  • test import pylab instead of pyplot (uses extra pylab functions)

…as series and dict with only one level of nesting
…el for more robusteness, added an optional rotation parameter
…el for more robusteness, added an optional rotation parameter
…as series and dict with only one level of nesting
EnricoGiampieri and others added 23 commits May 25, 2013 15:38
@jseabold
Copy link
Member

jseabold commented Apr 3, 2014

Closing this in favor of #1388.

@jseabold jseabold closed this Apr 3, 2014
@bashtage bashtage added reference rejected Used for PRs with changes that are not acceptable labels Jul 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
reference rejected Used for PRs with changes that are not acceptable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants