Repository navigation
perf: parallel-producer spike + bank the serial collision win - #13
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Hoist the outer entity's AABB bounds out of the inner loop (the previous version recomputed them for every otherEntity) and inline the rectangle intersection test, which removes the two temp AABB object literals (four allocations) built on every pair check. Drops the now-unused intersects import. Selected collision pairs and emitted events are unchanged — this is the serial/precomputed transformation whose pair-for-pair equivalence to the original is asserted by the perf-proofs parallel-producer parity harness. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Evidence-based evaluation of Worker threads + shared read-only snapshots for producer-only systems, using shmup collision as the probe. Adds three serial kernels (current-object replica, precomputed, packed SoA), a persistent SAB worker pool, the end-to-end benchmark matrix, and a correctness gate (cross-kernel parity + hand-derived ground truth) that aborts rather than reporting timings for incorrect kernels. Finding: the single-threaded packed cleanup is ~10x; SAB workers add a further 2.5-5.5x only at large-N low-density, and lose on dense collisions because emit-all-candidates forfeits the serial greedy early-break. Full writeup and recommendation in src/worker/README.md; original brief in src/worker/SPIKE.md. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Vite 8 runs on rolldown and marks build.rollupOptions deprecated in favor of build.rolldownOptions (identical shape). Clears the type-aware oxlint error that was failing the repo-wide `pnpm lint` CI step. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
✅ Deploy Preview for objecs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Completes the parallel-producer spike (brief in
packages/perf-proofs/src/worker/SPIKE.md): can objecs gain CPU performance from Worker threads + shared read-only snapshots for producer-only systems, using shmup collision as the probe — without any public API change.It also banks the single-threaded win the spike uncovered, and fixes an unrelated pre-existing lint failure so CI is green.
Commits
chore:bump node 26.11.1 / pnpm 12.12.1 (mise.toml)perf(examples):hoist AABB bounds out of the inner loop and inline the intersection test in the shmup collision system — removes 4 temp-object allocations per pair check. Selected pairs and emitted events are unchanged (asserted by the parity harness below).test(perf-proofs):the full spike — three serial kernels, a persistent SAB worker pool, the end-to-end benchmark matrix, and a correctness gate that aborts rather than reporting timings for incorrect kernels.fix(examples):rollupOptions→rolldownOptions(Vite 8 / rolldown); clears the type-aware oxlint error that was failingpnpm lint.Findings (AMD Ryzen 9 7950X3D, Node 26)
packedcleanup is ~10× over the shipped collision kernel, every scenario and size. Mostly from killing per-pair temp allocations + hoisting bounds. No threads, no API change. (Commit 2 applies the readable ~2.5× slice of this to the demo.)break.Scene.update()doesn't have today.Recommendation: qualified go, narrow scope. Bank the serial win (done here); real parallel payoff is for large-N low-density read-only producers (AI sensing / visibility / procgen), not collision — a spatial broadphase beats both for collision. Full writeup + answers to the brief's 10 questions in
packages/perf-proofs/src/worker/README.md.Verification
pnpm lint✅ ·pnpm --filter objecs build✅ ·pnpm --filter objecs check-exports✅ · objecs tests 25 pass ✅ · perf-proofs typecheck ✅ · examplestsc✅ · worker parity gate green (84 fixture/worker combinations).No public objecs API change. 🤖 Generated with Claude Code