Skip to content

Commit

Permalink
Alert on benchmark regression
Browse files Browse the repository at this point in the history 
Very low tresshold to test things out

Contribute to CURA-11527
  • Loading branch information
@jellespijker
jellespijker committed Jan 9, 2024
1 parent 085665f commit f61cffd
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions .github/workflows/benchmark.yml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -55,4 +55,6 @@ jobs:
output_file_path: "build/Release/benchmark_result.json"
data_dir: "dev/bench"
tool: "googlecpp"
alert_threshold: "101%"
alert_comment_cc_users: "@nallath, @jellespijker, @wawanbreton, @casperlamboo, @saumyaj3, @HellAholic"
secrets: inherit

1 comment on commit f61cffd

@rburema
Copy link
Member

@rburema rburema commented on f61cffd Jan 9, 2024

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 'C++ Benchmark'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.01.

Benchmark suite Current: f61cffd Previous: c8bfce0 Ratio
InfillTest/Infill_generate_connect/1/1200 552.5844320000033 ms/iter 541.5780194999797 ms/iter 1.02
WallTestFixture/InsetOrderOptimizer_getInsetOrder/9999 0.013897532822012006 ms/iter 0.013668677353758275 ms/iter 1.02

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

CC: @nallath @jellespijker @wawanbreton @casperlamboo @saumyaj3 @HellAholic

Please sign in to comment.