Skip to content

Commit 1afd3e4

Browse files
authored
ci: add centrality metric to priority unused includes workflow
1 parent 2a0b44d commit 1afd3e4

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

.github/workflows/prioritize-unused-includes.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ jobs:
3535
python set_edge_weights.py priority-unused-includes.csv include-analysis.js --config chromium --metric expanded_size > priority-unused-includes.expanded_size.csv
3636
python set_edge_weights.py priority-unused-includes.csv include-analysis.js --config chromium --metric prevalence > priority-unused-includes.prevalence.csv
3737
python set_edge_weights.py priority-unused-includes.csv include-analysis.js --config chromium --metric includer_size > priority-unused-includes.includer_size.csv
38+
python set_edge_weights.py priority-unused-includes.csv include-analysis.js --config chromium --metric centrality > priority-unused-includes.centrality.csv
3839
- uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
3940
with:
4041
name: weighted-unused-includes
@@ -44,6 +45,7 @@ jobs:
4445
priority-unused-includes.expanded_size.csv
4546
priority-unused-includes.prevalence.csv
4647
priority-unused-includes.includer_size.csv
48+
priority-unused-includes.centrality.csv
4749
- run: npm install @actions/cache
4850
- uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
4951
with:
@@ -55,6 +57,7 @@ jobs:
5557
const expandedSizeData = fs.readFileSync('./priority-unused-includes.expanded_size.csv', 'utf8').trim().split('\n');
5658
const prevalenceData = fs.readFileSync('./priority-unused-includes.prevalence.csv', 'utf8').trim().split('\n');
5759
const includerSizeData = fs.readFileSync('./priority-unused-includes.includer_size.csv', 'utf8').trim().split('\n');
60+
const centralityData = fs.readFileSync('./priority-unused-includes.centrality.csv', 'utf8').trim().split('\n');
5861
let totalAddedSize = 0;
5962
6063
const unusedIncludes = await Promise.all(data.split('\n').map(async (line, idx) => {
@@ -81,6 +84,7 @@ jobs:
8184
parseFloat(prevalenceData[idx].trim().split(',')[4]),
8285
parseInt(expandedSizeData[idx].trim().split(',')[4]),
8386
parseInt(includerSizeData[idx].trim().split(',')[4]),
87+
parseInt(centralityData[idx].trim().split(',')[4]),
8488
cacheHit,
8589
];
8690
}));
@@ -93,6 +97,7 @@ jobs:
9397
{ data: 'Prevalence', header: true },
9498
{ data: 'Expanded Size', header: true },
9599
{ data: 'Includer Size', header: true },
100+
{ data: 'Centrality', header: true },
96101
],
97102
// Sort by added size, then convert it back to string or it won't render
98103
...includes
@@ -103,6 +108,7 @@ jobs:
103108
`${prevalence.toFixed(2)}%`,
104109
expandedSize.toLocaleString(),
105110
includerSize.toLocaleString(),
111+
centrality.toFixed(3),
106112
]),
107113
]);
108114
}

0 commit comments

Comments
 (0)