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

Disable transition animation #43

Open
bigbug opened this issue Jan 17, 2022 · 1 comment
Open

Disable transition animation #43

bigbug opened this issue Jan 17, 2022 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@bigbug
Copy link
Collaborator

bigbug commented Jan 17, 2022

For large files the transition animation seems to lead to a higher computation time.
During experiments it was seen, that an initial rendering of the file took 15 seconds. Subsequent renders of the same file (with small changes on only one node) then took about 45 seconds. It was seen that removing

graphviz
  .fade(true)
  .transition(transition)
  .tweenPaths(true) // default
  .tweenShapes(true) // default

from the code lead to significant better render times.
The removal of this code could possibly reduce the render times. Maybe already lowering transition is sufficient. A mechanism could be implemented strip away the transition if the underlying DOT render of the previous DOT source took over 15?? seconds.

@bigbug bigbug added the enhancement New feature or request label Jan 17, 2022
@bigbug bigbug self-assigned this Jan 17, 2022
@tintinweb
Copy link
Owner

what about making this configurable or auto-sensing and disabling it if the input is larger than a threshold? 🤔

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

No branches or pull requests

2 participants