Skip to content

Assign a default value for display max_seq_items #3391

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
ghost opened this issue Apr 18, 2013 · 19 comments · Fixed by #3974 or #5753
Closed

Assign a default value for display max_seq_items #3391

ghost opened this issue Apr 18, 2013 · 19 comments · Fixed by #3974 or #5753
Labels
Output-Formatting __repr__ of pandas objects, to_string
Milestone

Comments

@ghost
Copy link

ghost commented Apr 18, 2013

Perhaps I've opened a dupe issue in the past, can't find it.
pd.options.display.max_seq_items should not be None by default.
It's easy to hang the terminal by repr()ing with some mal-chosen data.
#3426

@cpcloud
Copy link
Member

cpcloud commented Jun 19, 2013

I set mine to 10 by default but that may be too small.

@ghost
Copy link
Author

ghost commented Jul 15, 2013

@jreback , @cpcloud , 0.12?

@jreback
Copy link
Contributor

jreback commented Jul 15, 2013

up2u

@cpcloud
Copy link
Member

cpcloud commented Jul 15, 2013

i could take care of this in the pr i have for repring large sequences but i don't think i'm going to be able to finish it by 0.12...

@ghost
Copy link
Author

ghost commented Jul 15, 2013

please do. if it doesn't make it, no problem.

@jreback
Copy link
Contributor

jreback commented Sep 28, 2013

@cpcloud 0.13 or push?

@cpcloud
Copy link
Member

cpcloud commented Sep 28, 2013

0.13

@jtratner
Copy link
Contributor

I kinda wish we could add defaults to config options (i.e., if I'm not defined, use this other config option instead), then we could have this default to max_rows. Does add some complexit.

@jreback
Copy link
Contributor

jreback commented Sep 28, 2013

is this the same as max_rows? (or is this applicable for something else?)

@cpcloud
Copy link
Member

cpcloud commented Sep 28, 2013

no this is different

this for repring large sequences (in general, MultiIjndex, Index, lists, dicts, etc)

max rows is for a frame summary

@jreback
Copy link
Contributor

jreback commented Sep 28, 2013

what's the problem with None? then its computed right?

@cpcloud
Copy link
Member

cpcloud commented Sep 28, 2013

no ... it's unlimited if it's None, so a huge index will take forever to repr

@jreback
Copy link
Contributor

jreback commented Sep 28, 2013

ahh...I see...why don't we just put a sensible value as th edefault, maybe 100?

@jreback
Copy link
Contributor

jreback commented Sep 28, 2013

I don't have this set as a matter of fact...and the unlimited is annoying

@cpcloud
Copy link
Member

cpcloud commented Sep 28, 2013

yep that's what i've done (pr is up , but i haven't touched it in a while)... there's a tiny issue with using this with max_edge_items .. i'll get it ready for 0.13

@jreback
Copy link
Contributor

jreback commented Sep 28, 2013

gr8

@jreback
Copy link
Contributor

jreback commented Oct 4, 2013

@cpcloud this and max_edge_items?

@jreback
Copy link
Contributor

jreback commented Oct 11, 2013

@cpcloud push?

@jreback
Copy link
Contributor

jreback commented Oct 14, 2013

@cpcloud ?

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Output-Formatting __repr__ of pandas objects, to_string
Projects
None yet
3 participants