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

Performance issues for large graphs #30

Open
d12 opened this issue May 28, 2019 · 3 comments
Open

Performance issues for large graphs #30

d12 opened this issue May 28, 2019 · 3 comments

Comments

@d12
Copy link

d12 commented May 28, 2019

Profiling an ActiveRecord query generates a flamegraph that is borderline unusable. Zooming in to the graph such that labels show slows down the page to 300ms per frame, 3FPS. The majority of time is spent in d3.js restyling and rendering.

Can zoom be implemented in a way that doesn't require expensive re-calculations whenever we want to move around the graph?

Screen Shot 2019-05-28 at 10 55 35 AM

Screen Shot 2019-05-28 at 10 55 51 AM

@SamSaffron
Copy link
Owner

I would love some improvements made here, for sure, care to give it a shot. also @thisduck is planning to merge in his implementation as an optional rendering engine which works way better with very large graphs.

@d12
Copy link
Author

d12 commented May 29, 2019

I'm not super well versed in d3, I'm not sure I could do much better than this 😅 . Any reason for d3 over plain HTML/CSS?

@aroben
Copy link
Collaborator

aroben commented May 29, 2019

Some related work: tmm1/stackprof#74 tmm1/stackprof#100

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants