fix(web-probe): ignore terminal duration snap in jump detector (#767)
Co-authored-by: Codex <codex@noreply.local>
This commit is contained in:
@@ -4491,7 +4491,8 @@ function detectTurnTimingNonMonotonic(columns, rows) {
|
|||||||
const sampleDeltaSeconds = elapsedSecondsBetween(previous.ts, row.ts);
|
const sampleDeltaSeconds = elapsedSecondsBetween(previous.ts, row.ts);
|
||||||
const delta = current - previous.value;
|
const delta = current - previous.value;
|
||||||
const allowedIncreaseSeconds = sampleDeltaSeconds + alertThresholds.turnTimingSampleSlackSeconds;
|
const allowedIncreaseSeconds = sampleDeltaSeconds + alertThresholds.turnTimingSampleSlackSeconds;
|
||||||
if (delta > allowedIncreaseSeconds) {
|
const terminalTransition = !isTerminalTurnStatus(previous.status) && isTerminalTurnStatus(cell.status);
|
||||||
|
if (delta > allowedIncreaseSeconds && !terminalTransition) {
|
||||||
totalElapsedForwardJumps.push({
|
totalElapsedForwardJumps.push({
|
||||||
columnId: column.id,
|
columnId: column.id,
|
||||||
columnLabel: column.label,
|
columnLabel: column.label,
|
||||||
@@ -4501,9 +4502,11 @@ function detectTurnTimingNonMonotonic(columns, rows) {
|
|||||||
fromSeq: previous.seq,
|
fromSeq: previous.seq,
|
||||||
fromTs: previous.ts,
|
fromTs: previous.ts,
|
||||||
fromValue: previous.value,
|
fromValue: previous.value,
|
||||||
|
fromStatus: previous.status ?? null,
|
||||||
toSeq: row.seq ?? null,
|
toSeq: row.seq ?? null,
|
||||||
toTs: row.ts ?? null,
|
toTs: row.ts ?? null,
|
||||||
toValue: current,
|
toValue: current,
|
||||||
|
toStatus: cell.status ?? null,
|
||||||
delta,
|
delta,
|
||||||
sampleDeltaSeconds,
|
sampleDeltaSeconds,
|
||||||
allowedIncreaseSeconds,
|
allowedIncreaseSeconds,
|
||||||
@@ -4628,7 +4631,7 @@ function detectTurnTimingNonMonotonic(columns, rows) {
|
|||||||
});
|
});
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
previousByMetric.set(metric, { value: current, seq: row.seq ?? null, ts: row.ts ?? null });
|
previousByMetric.set(metric, { value: current, seq: row.seq ?? null, ts: row.ts ?? null, status: cell.status ?? null });
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user