Skip to content

Commit fce450d

Browse files
authored
Merge branch 'krausest:master' into master
2 parents c006381 + 3eb5279 commit fce450d

File tree

9 files changed

+871
-735
lines changed

9 files changed

+871
-735
lines changed

README.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
**Please note: There's an [issue](https://github.com/krausest/js-framework-benchmark/issues/1655) with chrome 124 such that we don't get reliable values. Currently all PRs are blocked by this issue.**
2+
13

24
# js-framework-benchmark
35

webdriver-ts-results/src/components/BoxPlotTable/BoxPlotTableChart.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,7 @@ const BoxPlotTableChart = ({ traces }: Props) => {
5050
},
5151
options: {
5252
animation: false,
53+
maintainAspectRatio: false,
5354
scales: {
5455
x: {
5556
type: "category",

webdriver-ts/package-lock.json

Lines changed: 617 additions & 581 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

webdriver-ts/package.json

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -21,27 +21,27 @@
2121
"author": "",
2222
"license": "Apache-2.0",
2323
"devDependencies": {
24-
"@types/node": "20.11.0",
25-
"@types/ramda": "0.29.9",
26-
"@types/selenium-webdriver": "4.1.21",
27-
"@types/semver": "7.5.6",
24+
"@types/node": "20.12.7",
25+
"@types/ramda": "0.29.12",
26+
"@types/selenium-webdriver": "4.1.22",
27+
"@types/semver": "7.5.8",
2828
"@types/yargs": "17.0.32",
29-
"@vitest/coverage-v8": "^1.2.0",
29+
"@vitest/coverage-v8": "^1.5.2",
3030
"ts-node": "^10.9.2",
31-
"typescript": "5.3.3",
32-
"vitest": "^1.2.0"
31+
"typescript": "5.4.5",
32+
"vitest": "^1.5.2"
3333
},
3434
"dependencies": {
35-
"chromedriver": "120.0.1",
35+
"chromedriver": "124.0.1",
3636
"cross-env": "7.0.3",
37-
"lighthouse": "11.4.0",
38-
"playwright": "1.40.1",
39-
"playwright-firefox": "1.40.1",
40-
"playwright-webkit": "1.40.1",
41-
"puppeteer-core": "21.7.0",
42-
"ramda": "0.29.1",
43-
"selenium-webdriver": "4.16.0",
44-
"semver": "7.5.4",
37+
"lighthouse": "12.0.0",
38+
"playwright": "1.43.1",
39+
"playwright-firefox": "1.43.1",
40+
"playwright-webkit": "1.43.1",
41+
"puppeteer-core": "22.7.1",
42+
"ramda": "0.30.0",
43+
"selenium-webdriver": "4.20.0",
44+
"semver": "7.6.0",
4545
"yargs": "17.7.2"
4646
}
4747
}

webdriver-ts/src/benchmarkRunner.ts

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -93,8 +93,7 @@ async function runBenchmakLoopStartup(
9393
let res = await forkAndCallBenchmark(framework, benchmarkInfo, benchmarkOptions);
9494
if (Array.isArray(res.result)) {
9595
results = results.concat(res.result as StartupBenchmarkResult[]);
96-
}
97-
else {
96+
} else {
9897
results.push(res.result);
9998
}
10099
if (res.warnings) {
@@ -338,6 +337,7 @@ async function main() {
338337
.array("framework")
339338
.array("benchmark")
340339
.number("count")
340+
.number("puppeteerSleep")
341341
.string("chromeBinary").argv;
342342

343343
console.log("args", args);
@@ -381,8 +381,11 @@ async function main() {
381381
resultsDirectory: "results",
382382
tracesDirectory: "traces",
383383
allowThrottling: !args.nothrottling,
384+
puppeteerSleep: args.puppeteerSleep ?? 0,
384385
};
385386

387+
config.PUPPETEER_WAIT_MS = benchmarkOptions.puppeteerSleep;
388+
386389
if (args.count) {
387390
benchmarkOptions.numIterationsForCPUBenchmarks = args.count;
388391
config.NUM_ITERATIONS_FOR_BENCHMARK_CPU_DROP_SLOWEST_COUNT = 0;
@@ -412,14 +415,12 @@ async function main() {
412415
let matchesDirectoryArg = (directoryName: string) =>
413416
frameworkArgument.length === 0 || frameworkArgument.some((arg: string) => arg == directoryName);
414417
let frameworks = await initializeFrameworks(benchmarkOptions, matchesDirectoryArg);
415-
runFrameworks = frameworks.filter(
416-
(f) => f.keyed || config.BENCHMARK_RUNNER !== BenchmarkRunner.WEBDRIVER_AFTERFRAME
417-
);
418+
runFrameworks = frameworks.filter((f) => f.keyed || config.BENCHMARK_RUNNER !== BenchmarkRunner.WEBDRIVER_AFTERFRAME);
418419

419-
if (args.type=='keyed') {
420+
if (args.type == "keyed") {
420421
runFrameworks = runFrameworks.filter((f) => f.keyed);
421422
console.log("run only keyed frameworks");
422-
} else if (args.type=='non-keyed') {
423+
} else if (args.type == "non-keyed") {
423424
runFrameworks = runFrameworks.filter((f) => !f.keyed);
424425
console.log("run only non-keyed frameworks");
425426
}

0 commit comments

Comments
 (0)