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

Panel rendering is pretty slow #416

Open
cirex-web opened this issue Jan 12, 2024 · 4 comments
Open

Panel rendering is pretty slow #416

cirex-web opened this issue Jan 12, 2024 · 4 comments
Labels
enhancement New feature or request

Comments

@cirex-web
Copy link
Contributor

Expected Behavior

Rendering a single page should take ~16ms, but the panel page takes 20+ ms. This is quite bad when compared to the map page which has a render time of .5ms. (Performance should probably not be this bad when all we're displaying is a grid with some text)
image
image

Steps to Reproduce

.

Device Information

.

@cirex-web cirex-web added the bug Something isn't working label Jan 12, 2024
@GhostOf0days GhostOf0days added enhancement New feature or request and removed bug Something isn't working labels Jan 12, 2024
@GhostOf0days GhostOf0days changed the title [Bug] Panel rendering kind of slow Panel rendering is pretty slow Jan 12, 2024
@GhostOf0days
Copy link
Member

GhostOf0days commented Jan 13, 2024

We may be able to see some performance increases in general if we switch our Virtual DOM to Million.js (see #96) and switch our runtime to Bun (this would be a more massive undertaking).

@cirex-web
Copy link
Contributor Author

or perhaps just use plain JSX rather than a UI framework?

@GhostOf0days
Copy link
Member

GhostOf0days commented Jan 13, 2024

Yeah, but, either way, I eventually wanted to switch out the Virtual DOM and runtime to Million.js and Bun for a long time. Million.js just wasn't stable back then although it is now. Bun is stable, but, it requires WSL for Windows since they're still working on a stable build for Windows (currently, unstable Windows build passes around 33% of Bun's internal tests).

@GhostOf0days
Copy link
Member

So kind of could help kill two birds with one stone (i don't remember if that's the saying lol).

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