Skip to content

MultiIndex notebook repr is incorrect when number of columns is > max_columns #7174

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

Closed
cpcloud opened this issue May 19, 2014 · 17 comments · Fixed by #7232
Closed

MultiIndex notebook repr is incorrect when number of columns is > max_columns #7174

cpcloud opened this issue May 19, 2014 · 17 comments · Fixed by #7232
Labels
Bug MultiIndex Output-Formatting __repr__ of pandas objects, to_string
Milestone

Comments

@cpcloud
Copy link
Member

cpcloud commented May 19, 2014

See this comment: #7146 (comment)

notebook http://goo.gl/qLUUx8

@jreback jreback added this to the 0.14.0 milestone May 19, 2014
@jreback
Copy link
Contributor

jreback commented May 19, 2014

cc @bjonen can you have a look?

@cpcloud
Copy link
Member Author

cpcloud commented May 19, 2014

in the second case i think the ... should be in the first level of the multindex as well as all the others

@jreback
Copy link
Contributor

jreback commented May 19, 2014

if the frame is transposed (and use max_rows) is it same issue?

@cpcloud
Copy link
Member Author

cpcloud commented May 19, 2014

yes, well, sort of. it looks like there might be an off-by-one error some where in the repring code for html .. i'll put a pr together if @bjonen doesn't have time.

@bjonen
Copy link
Contributor

bjonen commented May 19, 2014

@cpcloud I can have a look tonight.

@cpcloud
Copy link
Member Author

cpcloud commented May 19, 2014

ok sounds good

@cpcloud
Copy link
Member Author

cpcloud commented May 19, 2014

@bjonen i've almost got this finished

@bjonen
Copy link
Contributor

bjonen commented May 19, 2014

Ok excellent

@cpcloud
Copy link
Member Author

cpcloud commented May 20, 2014

@bjonen want to take a look at this? i thought i had it, but i don't think i'll be able to get to this in time for 0.14.0 ... here's a notebook with where i left off: http://nbviewer.ipython.org/gist/cpcloud/61910750733823892bd1 ... i think i'm pretty close but there's an issue with the central ... an a multiindex ... i think you might have a better feel for this code since you just changed it :)

you can cherry pick my changes from https://github.com/cpcloud/pandas/tree/fix-mi-format-repr-7174 if you want

@cpcloud cpcloud removed their assignment May 20, 2014
@bjonen
Copy link
Contributor

bjonen commented May 21, 2014

@cpcloud No problem, I'll have a look.

@jreback
Copy link
Contributor

jreback commented May 22, 2014

@bjonen fix for this?

@bjonen
Copy link
Contributor

bjonen commented May 22, 2014

Coming today. Columns are done, rows to be done.

@jreback
Copy link
Contributor

jreback commented May 22, 2014

gr8 thanks

@jreback
Copy link
Contributor

jreback commented May 23, 2014

@bjonen ?

@bjonen
Copy link
Contributor

bjonen commented May 23, 2014

Sorry didnt find the time yesterday. I'll submit asap.

@jreback
Copy link
Contributor

jreback commented May 24, 2014

@bjonen fix for this?

@jreback
Copy link
Contributor

jreback commented May 25, 2014

thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug MultiIndex Output-Formatting __repr__ of pandas objects, to_string
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants