-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
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
Comments
I set mine to 10 by default but that may be too small. |
up2u |
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... |
please do. if it doesn't make it, no problem. |
@cpcloud 0.13 or push? |
0.13 |
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. |
is this the same as |
no this is different this for repring large sequences (in general, MultiIjndex, Index, lists, dicts, etc) max rows is for a frame summary |
what's the problem with None? then its computed right? |
no ... it's unlimited if it's None, so a huge index will take forever to repr |
ahh...I see...why don't we just put a sensible value as th edefault, maybe 100? |
I don't have this set as a matter of fact...and the unlimited is annoying |
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 |
gr8 |
@cpcloud this and max_edge_items? |
@cpcloud push? |
@cpcloud ? |
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
The text was updated successfully, but these errors were encountered: