Skip to content

Conversation

@cknitt
Copy link
Member

@cknitt cknitt commented Oct 23, 2024

In the future, as requested here, have the bot only comment if there is a >5% difference in benchmark results.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Syntax Benchmarks

Benchmark suite Current: 33f0493 Previous: e1b7fb7 Ratio
Parse RedBlackTree.res - time/run 1.2050737133333334 ms 1.2123143266666667 ms 0.99
Parse RedBlackTree.res - allocs/run 199057 words 199057 words 1
Print RedBlackTree.res - time/run 2.3123508 ms 2.3104395133333333 ms 1.00
Print RedBlackTree.res - allocs/run 236403 words 236403 words 1
Print RedBlackTreeNoComments.res - time/run 2.1444886266666665 ms 2.10057036 ms 1.02
Print RedBlackTreeNoComments.res - allocs/run 249186 words 249186 words 1
Parse Napkinscript.res - time/run 39.443766366666665 ms 39.28006235333333 ms 1.00
Parse Napkinscript.res - allocs/run 8670066 words 8670066 words 1
Print Napkinscript.res - time/run 72.76365672666667 ms 77.00100409999999 ms 0.94
Print Napkinscript.res - allocs/run 9776327 words 9776327 words 1
Parse HeroGraphic.res - time/run 5.128570593333333 ms 5.13472718 ms 1.00
Parse HeroGraphic.res - allocs/run 1219326 words 1219326 words 1
Print HeroGraphic.res - time/run 8.660113246666667 ms 8.775952553333333 ms 0.99
Print HeroGraphic.res - allocs/run 1396466 words 1396466 words 1

This comment was automatically generated by workflow using github-action-benchmark.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'Syntax Benchmarks'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.01.

Benchmark suite Current: 33f0493 Previous: e1b7fb7 Ratio
Print RedBlackTreeNoComments.res - time/run 2.1444886266666665 ms 2.10057036 ms 1.02

This comment was automatically generated by workflow using github-action-benchmark.

@cknitt cknitt force-pushed the benchmark-threshold branch from 33f0493 to ef118ca Compare October 23, 2024 06:20
@cknitt cknitt changed the title WIP: Benchmark: report on alert only Benchmark: report on alert only Oct 23, 2024
@cknitt cknitt marked this pull request as ready for review October 23, 2024 06:22
@cknitt cknitt merged commit cd5e018 into master Oct 23, 2024
20 checks passed
@cknitt cknitt deleted the benchmark-threshold branch October 23, 2024 07:42
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