Skip to content

Performance Benchmarking [SYNTH-58] - #1511

Open
azaleacolburn wants to merge 6 commits into
devfrom
colbura/58/perf-profiling
Open

Performance Benchmarking [SYNTH-58]#1511
azaleacolburn wants to merge 6 commits into
devfrom
colbura/58/perf-profiling

Conversation

@azaleacolburn

@azaleacolburn azaleacolburn commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Task

SYNTH-58

Symptom

No real way to verify performance improvements or check for regressions.

Solution

Use vitest bench to run benchmarks for the:

  • MirabufParser
  • PhysicsSystem
  • SceneRendere

I would be happy to add benchmarks for other systems if need be.

In order to compare performance across commits, simply:

git checkout [COMMIT_ONE]
bun run bench:baseline
git checkout [COMMIT_TWO]
bun run bench:check

Verification

bun run bench runs and outputs reasonable times
bun run bench:baseline writes chromium performance to src/bench/baseline.json
bun run bench:check notes nearly identical performance to the cached baseline without any changes


Before merging, ensure the following criteria are met:

  • All acceptance criteria outlined in the ticket are met.
  • Necessary test cases have been added and updated.
  • A feature toggle or safe disable path has been added (if applicable).
  • User-facing polish:
    • Ask: "Is this ready-looking?"
  • Cross-linking between Jira and GitHub:
    • PR links to the relevant Jira issue.
    • Jira ticket has a comment referencing this PR.

@azaleacolburn
azaleacolburn force-pushed the colbura/58/perf-profiling branch from 6876e84 to 426ec67 Compare August 17, 2026 19:59
Comment thread fission/src/bench/baseline.json Outdated
Comment thread fission/src/bench/baseline.json Outdated
@azaleacolburn
azaleacolburn force-pushed the colbura/58/perf-profiling branch from 205fd66 to 437c612 Compare August 18, 2026 15:18
@azaleacolburn
azaleacolburn marked this pull request as ready for review August 18, 2026 21:46
@azaleacolburn
azaleacolburn requested review from a team as code owners August 18, 2026 21:46
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