-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
be80898 breaks display of large dataframes in IPython notebooks #5588
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
cc @takluyver can u post %prun str(df) python 2.7 ? |
Yes, 2.7.3 (Enthought Canopy, all updates) Here you go:
|
Interestingly, the call to df.info() works fine, it's only the automatic switching from a display of the df object to the df.info() call that doesn't seem to work. |
this is exceeding slow
takes 5s, with only 10k rows |
@takluyver also let's add a vbench in |
I swear, every time someone touches the display code.... I don't think the HTML code itself has had a perf regression, it's been slow for a long time OTOH, with pprint_thing, we hardly get any unicode bugs anymore so... thanks @michaelaye for reporting this so quickly. |
merged #5589 |
yep...thanks @y-p ! |
hm, now I get an overly large table. It's coming fast, but it's not, as before, adapting to current window width? |
do you not see scrollers (on both axes)? can you post a link/pic? |
From what I saw of the code while refactoring it, no, there was never any attempt to detect the width of the browser window, and I don't think there's any good way to do so. Previously, your DataFrame would have switched to info view because of the number of rows. Now it's displaying a truncated view, but the default for In the notebook, clicking on the "Out [n]" prompt should put the output into a smaller div with scrollbars, which can make it easier to work with. |
I see. I was just irritated by the different behavior and actually kinda like the solution. And I can influence it with max_columns, so all fine here. Value of default |
gr8...thanks for the report @michaelaye maybe @takluyver (or @michaelaye ) if you want....to update the new doc section on this change to emphasize that a user may want to change |
reopening: https://github.com/TomAugspurger |
merged #5649 |
In 044ee06, I could just do, with no apparent delay:
In be80898, the notebook gets stuck and CPU is boiling and nothing is returned after 5 mins.
related #5550
The text was updated successfully, but these errors were encountered: