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

Improve loading performance #48

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

MTRNord
Copy link
Contributor

@MTRNord MTRNord commented Dec 24, 2024

First of all a disclaimer that all tests and profiles had been done with Chromium "Version 131.0.6778.204 (Offizieller Build) Fedora Project (64-Bit)" and not firefox.

The before profile has been done using https://matrix-org.github.io/tardis/ and the after using a debug build.

Before the changes the redraw method on my pc took ~22802.8ms. After the changes I am getting ~9583.5ms. This is rougly an improvement of 58%

There only was a visual comparison on correctness however. I strongly think there have been no changes in behaviour of the code however I sugest having a good read :)

The profiles and the file used for testing can be found here https://cloud.midnightthoughts.space/s/yWfniCMFZ7xA99w (size limit of github exceeded. Its my nextcloud)

Copy link
Member

@kegsay kegsay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything but graph.ts LGTM - deferring to @ara4n on the semantic changes to the graph render.

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.

2 participants