Skip to content

feat(scaling): show block gas limit and benchmark runs - #218

Open
wlawt wants to merge 8 commits into
mainfrom
toshi/update-scaling-tab-ui
Open

wlawt wants to merge 8 commits into
mainfrom
toshi/update-scaling-tab-ui

Conversation

@wlawt

@wlawt wlawt commented Oct 28, 2026 •

Copy link
Copy Markdown

Motivation

Show configured block gas capacity and independent benchmark evidence on Scaling. Implement visual option #6: latest results, then all targets.

Changes

  • Show the block gas limit using the 200ms block interval.
  • Highlight the latest eligible Glamsterdam mix and Main mix jobs independently by creation time, with measured Mgas/s, target, successful TPS, and job links.
  • Retain complete side-by-side workload/target inventories below the cards; separate actual Mgas/s and successful TPS columns. Stack cards and inventories on mobile, with horizontally scrollable tables.
  • Preserve source setup, loading, error, empty and partial-error states. No job matching, extrapolation, or calculated performance gaps.

Tests

Test Name Description
npm test 72 files, 559 tests passed; covers recency over target, mix-only highlights, reverted TPS, inventory retention, and empty states.
npm run typecheck Passed.
Targeted ESLint All three changed TSX files passed.
Staff review + React checklist No findings; no hooks/effects/timers or clickable div/span introduced.
Browser preview Inspected desktop and mobile /scaling using synthetic snapshot and benchmark data.

Screenshots

Synthetic preview data, not live benchmark evidence.

Desktop

Scaling option 6 desktop

Mobile

Scaling option 6 mobile

Co-authored-by: Toshi <toshi-noreply@coinbase.com>
@cb-heimdall

Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@vercel

vercel Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
omni-ui Ready Ready Preview Oct 29, 2026 5:27pm UTC

Request Review

Co-authored-by: Toshi <toshi-noreply@coinbase.com>
Co-authored-by: Toshi <toshi-noreply@coinbase.com>
@wlawt

wlawt commented Oct 28, 2026

Copy link
Copy Markdown
Author

Triaged the non-blocking E2E failure on b9d744b. It is unrelated to this PR: the test web server reports missing R2 configuration for mainnet/sepolia/zeronet, and the remaining failures are in existing Vibenet explorer E2E tests (no live block rows, plus a flaky bounding-box assertion). This PR changes no workflow, E2E, snapshot, or Vibenet files; unit tests, typecheck, lint, and public build pass. I did not alter unrelated code or rerun again because the failure is environmental and reproduced on the prior attempt too.

Co-authored-by: Toshi <toshi-noreply@coinbase.com>
Co-authored-by: Toshi <toshi-noreply@coinbase.com>
Co-authored-by: Toshi <toshi-noreply@coinbase.com>
@wlawt wlawt changed the title feat(scaling): show block gas limit and compare Glamsterdam runs feat(scaling): show block gas limit and benchmark runs Oct 28, 2026
Co-authored-by: Toshi <toshi-noreply@coinbase.com>
mw2000
mw2000 previously approved these changes Oct 29, 2026
Co-authored-by: Toshi <toshi-noreply@coinbase.com>

This branch was successfully deployed

1 active deployment
Preview — b6081083 Deployed Oct 29, 2026 by vercel[bot]
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.

3 participants