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

Fast rendering Panning and Zooming SVG based callgraph #75

Open
wants to merge 76 commits into
base: master
Choose a base branch
from

Conversation

Tuurlijk
Copy link

Hi Paul,

Thank you very much for providing this improved xhprof gui!

I have been making a few small changes to xhprof. I noticed that generating a gif callgraph is faster and / or less memory hungry than png. But some graphs still failed to complete. SVG graphs do complete every time, and fast too. I created a small interface around the svg so that a user can easily go back to the previous page too.

Kind regards,

Michiel

screen shot 2013-11-27 at 13 04 29

@Tuurlijk
Copy link
Author

The latest two commits make the call graph clickable. You can jump to the func now by clicking a node. Clicking he main() shows the full graph again.

@preinheimer
Copy link
Owner

Very interesting, I'll try to take a look this week, sorry for the delay.

Michiel Roos added 8 commits December 13, 2013 23:36
…ndsSeparator and decimalSeparator settings in config.php

[CLEANUP] Code cleanup
[FEATURE] Improved back link in call graph
[FEATURE] Enable filtering by domain column by clicking on domain
[FEATURE] CallGraph: Add memory info to nodes
[FEATURE] CallGraph: Add toolTip to nodes
[FEATURE] CallGraph: Add domain and url as graph name
Michiel Roos added 14 commits December 31, 2013 11:33
[FEATURE] Add tooltip on pie charts that explains what happens when clicking a label
[TASK] Cleanup pie chart code
[FEATURE] make function names overflow on the left (Chrome does not get this)
[BUGFIX] Prevent division by 0 in diff view callgraph
[TASK] Style 'View Callgraph' links as a button
[BUGFIX] Disable link hover style in callgraph
[FEATURE] Enable dataTables 'search in table'
[FEATURE] Enable dataTables pagin feature
[CLEANUP] Remove all sorting code from xhprof.php
[FEATURE] Add graph that shows inclusive wall time
[BUGFIX] Use SCRIPT_NAME instead of DOCUMENT_URI
@Tuurlijk
Copy link
Author

Ok, not all dot versions support 'multiline labels'. So the dot version is now checked before attempting to render multiline labels.

@aik099
Copy link
Collaborator

aik099 commented Dec 27, 2016

@Tuurlijk can you please rebase and squash commits in this PR?

This also seem to resolve #104, but in current state (lots of commits) it's pretty hard to figure out what's going on.

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

Successfully merging this pull request may close these issues.

None yet

5 participants