Skip to content

Commit

Permalink
Move fastest
Browse files Browse the repository at this point in the history
  • Loading branch information
ehmicky committed Aug 18, 2019
1 parent 5ea3260 commit aad855c
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 10 deletions.
6 changes: 2 additions & 4 deletions src/info/main.js
Original file line number Diff line number Diff line change
Expand Up @@ -62,12 +62,10 @@ const getIterations = function({ iterations, tasks, variations, commands }) {
addIterationInfo({ iteration, tasks, variations, commands }),
)

const iterationsB = addFastestIterations(iterationsA)

// The fastest tasks will be first, then the fastest iterations within each
// task (regardless of variants or runners)
sortBy(iterationsB, ['task', 'stats.median'])
return iterationsB
sortBy(iterationsA, ['task', 'stats.median'])
return iterationsA
}

const addIterationInfo = function({
Expand Down
File renamed without changes.
18 changes: 12 additions & 6 deletions src/print/main.js
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
import { addFastestIterations } from './fastest.js'
import { dereferenceBenchmark } from './dereference.js'
import { addPrevious } from './previous.js'
import { normalizeStats, addPrintedStats } from './stats/main.js'
Expand All @@ -8,7 +9,12 @@ export const addPrintedInfo = async function(
{ show, diff, dataDir, store, verbose, nested },
{ iterations, ...benchmark },
) {
const iterationsA = dereferenceBenchmark({ benchmark, iterations })
const iterationsA = addFastestIterations(iterations)

const iterationsB = dereferenceBenchmark({
benchmark,
iterations: iterationsA,
})

const nestedNormalize = getNestedNormalize({
show,
Expand All @@ -17,17 +23,17 @@ export const addPrintedInfo = async function(
store,
nested,
})
const { previous, iterations: iterationsB } = await addPrevious({
const { previous, iterations: iterationsC } = await addPrevious({
benchmark,
iterations: iterationsA,
iterations: iterationsB,
diff,
dataDir,
store,
nestedNormalize,
})

const iterationsC = normalizeStats(iterationsB)
const iterationsD = addPrintedStats(iterationsC, verbose)
const iterationsD = normalizeStats(iterationsC)
const iterationsE = addPrintedStats(iterationsD, verbose)

const timestamp = getTimestamp(benchmark, show)

Expand All @@ -37,7 +43,7 @@ export const addPrintedInfo = async function(
...benchmark,
timestamp,
printedSystem,
iterations: iterationsD,
iterations: iterationsE,
previous,
}
}
Expand Down

0 comments on commit aad855c

Please sign in to comment.