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

Plots don't render on Chrome with a large data set (>600MB) #5869

Open
3 of 7 tasks
shefalijoshi opened this issue Oct 13, 2022 · 3 comments
Open
3 of 7 tasks

Plots don't render on Chrome with a large data set (>600MB) #5869

shefalijoshi opened this issue Oct 13, 2022 · 3 comments
Assignees
Labels
needs:info Needs follow up or investigation performance impacts or improves performance severity:medium source:user request type:bug

Comments

@shefalijoshi
Copy link
Contributor

shefalijoshi commented Oct 13, 2022

Summary

When trying to plot an extremely large dataset (with no mixmaxing), the data transferred over the network overwhelms the browser. You will inevitably reach a point where the browser runs out of memory and errors. There is different behavior in different browsers.

Chrome: Data array in memory grows and grows and grows, until Chrome has had enough, and will drop the data from memory silently.

Firefox: Data array in memory grows and grows and grows, until FF has had enough, and will throw a console error.

Expected vs Current Behavior

At the very least the user should understand that the plot is not going to render for a data set that large.

Steps to Reproduce

Environment

  • Open MCT Version:
  • Deployment Type:
  • OS:
  • Browser:

Impact Check List

  • Data loss or misrepresented data?
  • Regression? Did this used to work or has it always been broken?
  • Is there a workaround available?
  • Does this impact a critical component?
  • Is this just a visual bug with no functional impact?
  • Does this block the execution of e2e tests?
  • Does this have an impact on Performance?

Additional Information

@unlikelyzero unlikelyzero added this to the Target:2.1.2 milestone Oct 17, 2022
@ozyx ozyx added the needs:info Needs follow up or investigation label Oct 18, 2022
@unlikelyzero unlikelyzero modified the milestones: Target:2.1.2, 2.1.3 Oct 27, 2022
@unlikelyzero unlikelyzero added source:user request performance impacts or improves performance labels Oct 27, 2022
@unlikelyzero unlikelyzero self-assigned this Oct 27, 2022
@davetsay
Copy link
Contributor

davetsay commented Oct 27, 2022

@unlikelyzero , added a summary of what we were seeing

@davetsay
Copy link
Contributor

davetsay commented Nov 1, 2022

performance.measureUserAgentSpecificMemory

@JonanOribe
Copy link
Contributor

Maybe one first step here it's to show an alert pop-up to the user about what is happening. Then, second step looks for a way to deal with those load volumes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs:info Needs follow up or investigation performance impacts or improves performance severity:medium source:user request type:bug
Projects
None yet
Development

No branches or pull requests

5 participants