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

ENH: Add _repr_html_ to SimpleTable #1976

Merged
merged 2 commits into from Sep 20, 2014

Conversation

Projects
None yet
2 participants
@jseabold
Copy link
Member

commented Sep 20, 2014

see #1555

jseabold added a commit that referenced this pull request Sep 20, 2014

Merge pull request #1976 from jseabold/repr-html-simpletable
ENH: Add _repr_html_ to SimpleTable

@jseabold jseabold merged commit ffcfee9 into statsmodels:master Sep 20, 2014

1 of 2 checks passed

continuous-integration/travis-ci The Travis CI build is in progress
Details
continuous-integration/appveyor AppVeyor build succeeded
Details

@jseabold jseabold deleted the jseabold:repr-html-simpletable branch Sep 20, 2014

@josef-pkt

This comment has been minimized.

Copy link
Member

commented Sep 20, 2014

pep-8 cleanup means we have taken over SimpleTable, if we haven't done so already some time ago.
We won't be able to merge changes from Alan's version anymore.

@jseabold

This comment has been minimized.

Copy link
Member Author

commented Sep 20, 2014

It was really hard to read. We can revert if it's a huge issue.

@josef-pkt

This comment has been minimized.

Copy link
Member

commented Sep 21, 2014

No I think it's fine. I think Alan mentioned once that we take over SimpleTable and he would be talking with you, IIRC.

We have been changing parts in several places, but, I guess, Alan is still the only one who really knows how SimpleTable fits together.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.