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

Have the ability to parse large outputs #34

Open
Jorriss opened this issue Apr 30, 2015 · 2 comments · Fixed by MisterZeus/StatisticsParser#1 · May be fixed by #51
Open

Have the ability to parse large outputs #34

Jorriss opened this issue Apr 30, 2015 · 2 comments · Fixed by MisterZeus/StatisticsParser#1 · May be fixed by #51

Comments

@Jorriss
Copy link
Owner

Jorriss commented Apr 30, 2015

Large outputs (5000 rows) are crashing the browser.

Found by Clive Strong and Dennis Miller

@jzabroski
Copy link

Can you attach an example? I can investigate. Thanks.

Shah-Nisarg added a commit to Shah-Nisarg/StatisticsParser that referenced this issue Feb 10, 2020
The `<div>` element is not closed correctly right now. This causes the browser to do a bit of error-correction. The error-correction process itself is a bit buggy, and in this case it creates a hierarchy of tables instead of putting tables side-by-side. That causes performance issues when large amount of data is loaded.

This change should fix: Jorriss#34
@mjkent
Copy link

mjkent commented Dec 29, 2023

Here is what happens after enough tables are put on the page - Chrome and Firefox give up on trying to fix the missing divs.
image

An example file:
TooManyStatistics.txt

@mjkent mjkent mentioned this issue Dec 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants