Skip to content
This repository was archived by the owner on Jan 26, 2026. It is now read-only.

add benchmark to CI/CD#32

Merged
agavra merged 6 commits intomainfrom
bench_cicd
May 29, 2025
Merged

add benchmark to CI/CD#32
agavra merged 6 commits intomainfrom
bench_cicd

Conversation

@agavra
Copy link
Contributor

@agavra agavra commented May 29, 2025

fixes #29 by running benchmarks on CI/CD and reporting the changes on the pull request - since both baseline and new run on the same GHA runner the diff should be somewhat reliable even if the absolute value of the numbers are not.

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. enhancement New feature or request labels May 29, 2025
@agavra agavra force-pushed the bench_cicd branch 2 times, most recently from 8fa7c52 to 7e6a19f Compare May 29, 2025 02:15
@github-actions
Copy link

github-actions bot commented May 29, 2025

📊 Benchmark Comparison Report

This pull request includes Criterion benchmarks comparing performance to the main branch.
This comment will automatically update as the benchmarks are re-run on each commit.

The table below shows relative ratios and timing stats for each benchmark group:

group                main                                   pr
-----                ----                                   --
ops/merge            1.03  1509.2±26.32ns        ? ?/sec    1.00  1463.8±18.04ns        ? ?/sec
ops/project          1.00    566.6±8.51ns        ? ?/sec    1.05    594.0±7.81ns        ? ?/sec
serde/deserialize    1.00    314.7±4.43ns        ? ?/sec    1.03    324.0±4.70ns        ? ?/sec
serde/serialize      1.00    310.8±7.61ns        ? ?/sec    1.00    310.5±5.01ns        ? ?/sec

✅ Benchmarks completed successfully.

🧠 Notes:

  • These benchmarks are not a pass/fail gate and are informative only.
  • Use this as a signal to review performance-sensitive changes.
  • Results may be unreliable due to GHA runner hardware variance.
  • If results indicate a significant performance regression, run the benchmarks locally to confirm.

Reported by the benchmark CI bot

@imprint-serde imprint-serde deleted a comment from github-actions bot May 29, 2025
@agavra agavra merged commit 27c4353 into main May 29, 2025
2 checks passed
@agavra agavra deleted the bench_cicd branch May 29, 2025 04:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Restructure Benchmarks to Run in CI

1 participant