Skip to content

fix: run baseline bench comparison on main pushes#22

Merged
RostiMelk merged 3 commits intomainfrom
fix/bench-summary-features
Feb 7, 2026
Merged

fix: run baseline bench comparison on main pushes#22
RostiMelk merged 3 commits intomainfrom
fix/bench-summary-features

Conversation

@RostiMelk
Copy link
Member

@RostiMelk RostiMelk commented Feb 7, 2026

Main branch summaries were missing the comparison table and feature breakdown that PR comments get. Now both events run a baseline benchmark and produce the full report. Removes unused benchmark-report.md generation.

@vercel
Copy link

vercel bot commented Feb 7, 2026

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

Project Deployment Actions Updated (UTC)
react-logo-soup Ready Ready Preview, Comment Feb 7, 2026 4:01pm

Request Review

@github-actions
Copy link

github-actions bot commented Feb 7, 2026

Benchmark Comparison: main vs fix/bench-summary-features

Threshold: 5%+ change, >100us absolute delta, and statistically significant (p<0.05).

Benchmark main fix/bench-summary-features Change p-value Verdict
content detection (1 logo) 35.50 us 33.78 us -4.8% 0.399 unchanged
render pass (20 logos) 862 ns 833 ns -3.3% 0.214 unchanged
mount 20 logos (no detection) 4.85 us 3.04 us -37.3% <0.001 *** unchanged
mount 20 logos (defaults) 842.91 us 811.25 us -3.8% 0.002 ** unchanged

No regressions detected.

Feature cost breakdown

How expensive are individual features? Measured on this run's HEAD commit.

Feature On Off Cost Sig
densityAware: true vs false 33.28 us 31.64 us ~same 0.181
alignBy: visual-center-y vs bounds 789 ns 57 ns 14× slower <0.001 ***
cropToContent: true vs false 2.13 ms 41 ns 2.13 ms <0.001 ***
layout update: full mount vs cached 836.80 us 1.89 us 442× slower <0.001 ***

Full benchmark output in the CI job logs.

@RostiMelk RostiMelk merged commit 1b88111 into main Feb 7, 2026
4 checks passed
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.

1 participant