Skip to content

Run 4 perf tests instead of 2 #80

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

Merged
j-f1 merged 7 commits into from
Sep 25, 2020
Merged

Run 4 perf tests instead of 2 #80

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
847f881
Run Prettier
j-f1 Sep 25, 2020
7537193
Average over four tests instead of two
j-f1 Sep 25, 2020
b2bb780
More advanced algorithm
j-f1 Sep 25, 2020
af6dbf2
spelling
j-f1 Sep 25, 2020
107c9cf
Return the results
j-f1 Sep 25, 2020
f45e86e
Update index.js
j-f1 Sep 25, 2020
5c8472e
Update index.js
j-f1 Sep 25, 2020
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
More advanced algorithm
  • Loading branch information
@j-f1
j-f1 committed Sep 25, 2020
commit b2bb780a316150c424ab21565e101f2f827c8a70
14 changes: 9 additions & 5 deletions ci/perf-tester/src/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,11 +32,15 @@ const {
toBool,
} = require("./utils.js");

const benchmarkCount = 4;
const runBunchmarks = () =>
Promise.all(Array(benchmarkCount).fill().map(runBenchmark)).then(
averageBenchmarks
);
const benchmarkParallel = 2;
const benchmarkSerial = 2;
const runBunchmarks = async () => {
for (let i = 0; i < benchmarkSerial; i++) {
await Promise.all(
Array(benchmarkParallel).fill().map(runBenchmark)
).then(averageBenchmarks);
}
};

async function run(octokit, context, token) {
const { number: pull_number } = context.issue;
Expand Down