Skip to content

I21 performance#22

Merged
QSparks merged 9 commits into
i19-scenario-menufrom
i21-performance
May 19, 2026
Merged

I21 performance#22
QSparks merged 9 commits into
i19-scenario-menufrom
i21-performance

Conversation

@QSparks
Copy link
Copy Markdown
Collaborator

@QSparks QSparks commented May 11, 2026

resolves #21

@QSparks QSparks requested a review from Nospamas May 11, 2026 15:25
@QSparks QSparks self-assigned this May 11, 2026
Copy link
Copy Markdown
Contributor

@Nospamas Nospamas left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Comment thread docker/app/server.js
@@ -0,0 +1,96 @@
const http = require("node:http");
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Its nice to see HTTP server gone, but I do wonder if our static file servers would be best served by something lighter weight like nginx.

@QSparks QSparks merged commit b315e2c into i19-scenario-menu May 19, 2026
1 check passed
QSparks added a commit that referenced this pull request May 19, 2026
* Add rails and labels for model and scenario selection

* I21 performance (#22)

* Defer FWA index loading

* Add preconnect and favicon

* Reduce log noise

* Use window location origin for timeseries base URL

* Convert logo to webp, set size and fetch priority

* Lazy load DataSelectionTable, PointPlotter and HelpGuide

* Replace http-server with node static server

* Increase keepBuffer, disable updateWhenZooming

* Update HelpGuide
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