Skip to content

Commit 0eca282

Browse files
committed
results for chrome 114
1 parent de84427 commit 0eca282

File tree

13 files changed

+3403
-3468
lines changed

13 files changed

+3403
-3468
lines changed

frameworks/keyed/blockdom/package.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,8 @@
55
"main": "dist/app.min.js",
66
"js-framework-benchmark": {
77
"frameworkVersionFromPackage": "blockdom",
8-
"frameworkHomeURL": "https://github.com/ged-odoo/blockdom"
8+
"frameworkHomeURL": "https://github.com/ged-odoo/blockdom",
9+
"issues": [1261]
910
},
1011
"scripts": {
1112
"build-dev": "rollup -c -w",

frameworks/keyed/million/package-lock.json

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

webdriver-ts-results/src/App.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,10 @@ const App = (): JSX.Element => {
1515
return (
1616
<div>
1717
{disclaimer}
18-
<p>The benchmark was run on a MacBook Pro 14 (32 GB RAM, 8/14 Cores, OSX 13.3), Chrome 113.0.5672.63 (arm64))
18+
<p>The benchmark was run on a MacBook Pro 14 (32 GB RAM, 8/14 Cores, OSX 13.4), Chrome 114.0.5735.90 (arm64))
1919
using the puppeteer benchmark driver with reduced tracing.
2020
</p>
21-
<SelectionBar/>
21+
<SelectionBar showDurationSelection={false}/>
2222
<ResultTable type={FrameworkType.KEYED}/>
2323
<ResultTable type={FrameworkType.NON_KEYED}/>
2424

webdriver-ts-results/src/Common.ts

Lines changed: 2 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -20,22 +20,6 @@ export interface Framework {
2020

2121
export enum Severity {Note, Error}
2222

23-
interface Category {
24-
id: number;
25-
text: string;
26-
issues: Array<number>;
27-
severity: Severity;
28-
}
29-
30-
export const categories: Category[] = [
31-
{id:1, text:"[Note]: Manual DOM manipulations", issues: [772], severity: Severity.Note},
32-
{id:2, text:"[Note]: View state on the model", issues: [800], severity: Severity.Note},
33-
{id:3, text:"[Note]: Explicit requestAnimationFrame calls", issues: [796], severity: Severity.Note},
34-
{id:4, text:"[Note]: Manual event delegation", issues: [801], severity: Severity.Note},
35-
{id:5, text:"[Note]: Implementation uses runtime code generation", issues: [1139], severity: Severity.Note},
36-
{id:6, text:"[Issue]: Errors in the implementation", issues: [634], severity: Severity.Error},
37-
]
38-
3923
const DEFAULT_RESULTS_KEY = 'DEFAULT';
4024

4125
interface IIssue {
@@ -52,6 +36,7 @@ export const knownIssues: IIssue[] = [
5236
{issue: 800, severity: Severity.Note, text:"[Note]: View state on the model", link: "https://github.com/krausest/js-framework-benchmark/issues/800"},
5337
{issue: 801, severity: Severity.Note, text:"[Note]: Implementation uses manual event delegation", link: "https://github.com/krausest/js-framework-benchmark/issues/801"},
5438
{issue: 1139, severity: Severity.Note, text:"[Note]: Implementation uses runtime code generation", link: "https://github.com/krausest/js-framework-benchmark/issues/1139"},
39+
{issue: 1261, severity: Severity.Note, text:"[Note]: Manual caching of (v)dom nodes", link: "https://github.com/krausest/js-framework-benchmark/issues/1261"},
5540
];
5641

5742
export function findIssue(issueNumber: number): IIssue|undefined {
@@ -215,12 +200,7 @@ export class ResultTableData {
215200
this.selectedFameworks = new Set<Framework>();
216201

217202
const allowedIssues = new Set<number>();
218-
categories.forEach(c => {
219-
if (selectedCategories.has(c.id)) {
220-
for (const i of c.issues) { allowedIssues.add(i); }
221-
}
222-
});
223-
203+
knownIssues.forEach(i => allowedIssues.add(i.issue));
224204
console.log("ResultTableData", allowedIssues, selectedCategories);
225205

226206
selectedFrameworksInDropdown.forEach(f => {

webdriver-ts-results/src/reducer.ts

Lines changed: 2 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { Benchmark, BenchmarkType, convertToMap, DisplayMode, Framework, FrameworkType, RawResult, Result, ResultTableData, SORT_BY_GEOMMEAN_CPU, categories, Severity, ResultValues, CpuDurationMode } from "./Common"
1+
import { Benchmark, BenchmarkType, convertToMap, DisplayMode, Framework, FrameworkType, RawResult, Result, ResultTableData, SORT_BY_GEOMMEAN_CPU, Severity, ResultValues, CpuDurationMode, knownIssues } from "./Common"
22
import { benchmarks as benchmark_orig, frameworks, results as rawResults } from './results';
33

44
// Temporarily disable script bootup time
@@ -96,7 +96,7 @@ let preInitialState: State = {
9696
[FrameworkType.KEYED]: undefined,
9797
[FrameworkType.NON_KEYED]: undefined
9898
},
99-
categories: new Set(categories.filter(c => c.severity != Severity.Error).map(c => c.id)),
99+
categories: new Set(knownIssues.map(ki => ki.issue)),
100100
cpuDurationMode : CpuDurationMode.Total
101101
}
102102

@@ -131,15 +131,6 @@ function extractState(state: any): Partial<State> {
131131
if (state.displayMode!==undefined) {
132132
t = { ...t, displayMode: state.displayMode };
133133
}
134-
if (state.categories!==undefined) {
135-
const newSelectedCategories = new Set<number>();
136-
for (const f of state?.categories ?? []) {
137-
for (const sc of categories) {
138-
if (f === sc.id) newSelectedCategories.add(sc.id);
139-
}
140-
}
141-
t = { ...t, categories: newSelectedCategories };
142-
}
143134
return t;
144135
}
145136

@@ -312,11 +303,6 @@ export const reducer = (state = initialState, action: Action): State => {
312303
const t = { ...state, categories };
313304
return { ...t, resultTables: updateResultTable(t) };
314305
}
315-
case 'SELECT_ALL_CATEGORIES': {
316-
const newCategories = (action.data.add) ? new Set(categories.map(c => c.id)) : new Set<number>();
317-
const t = { ...state, categories: newCategories };
318-
return { ...t, resultTables: updateResultTable(t) };
319-
}
320306
default:
321307
return state;
322308
}

0 commit comments

Comments
 (0)