Skip to content
This repository was archived by the owner on Oct 26, 2026. It is now read-only.

Commit 64d5c6b

Browse files
committed
PROTO(objectui#5403): retry the noise-floor leg, and spell the delta's sign [verdict]
The first [verdict] round raced round 6: the noise-floor job downloaded that run's merged report before its merge job had uploaded it. Also fixes the sign encoding — the value's own hyphen ate the field separator, so a positive delta read as 'minus'.
1 parent cbf53d3 commit 64d5c6b

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

‎scripts-proto/verdict.mjs‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,9 @@ const name = [
3636
`stmtA-${ta.c}of${ta.s}`,
3737
`stmtB-${tb.c}of${tb.s}`,
3838
`stmtDiff-${tb.c - ta.c}`,
39-
`deltaPP-${deltaPP}`.replace('.', 'p').replace('-0p', 'minus0p'),
39+
// Spell the sign as a word: an earlier version encoded it with a hyphen and
40+
// the value's own hyphen made the sign unreadable in the artifact name.
41+
`deltaPP-${Number(deltaPP) < 0 ? 'NEG' : 'POS'}-${Math.abs(Number(deltaPP)).toFixed(4).replace('.', 'p')}`,
4042
`regressed-${regressed}`,
4143
].join('__');
4244
console.log(name);

0 commit comments

Comments
 (0)