Skip to content

Performance: Add comment with perf changes in PRs #56899

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
WunderBart wants to merge 10 commits into from
Closed

Performance: Add comment with perf changes in PRs #56899

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
3fd0fe2
Introduce performance metrics comparison step
WunderBart Dec 8, 2023
3e498ba
tmp: simulate regression
WunderBart Dec 8, 2023
616bae4
Collect all regressed metrics
WunderBart Dec 8, 2023
74417ca
Rename to check-performance-regression
WunderBart Dec 8, 2023
3960e67
Print regression percentage
WunderBart Dec 8, 2023
eaed08c
Add a fail-safe for missing result files
WunderBart Dec 8, 2023
4425e8c
Fix file filter
WunderBart Dec 8, 2023
b992876
Ensure results json has correct format
WunderBart Dec 8, 2023
6616ca9
Revert "tmp: simulate regression"
WunderBart Dec 8, 2023
2eb27e9
Try 30% max regression allowed
WunderBart Dec 8, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Print regression percentage
  • Loading branch information
@WunderBart
WunderBart committed Dec 8, 2023
commit 3960e67e3b614d2ce83e0d1fea53f2e0ff2a7aab
7 changes: 6 additions & 1 deletion bin/check-performance-regression.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,13 @@ fs.readdir( resultsPath, ( err, files ) => {
console.log( 'Performance regression detected:' );
regressedMetrics.forEach(
( { file, metric, branchValue, trunkValue } ) => {
const percent = (
( ( branchValue - trunkValue ) / trunkValue ) *
100
).toFixed( 2 );

console.log(
`- In ${ file }: ${ metric } increased from ${ trunkValue } to ${ branchValue }`
`- In ${ file }: ${ metric } increased by ${ percent }% (from ${ trunkValue } to ${ branchValue })`
);
}
);
Expand Down