fix(ml-labeling): remove flaky 1μs latency assertions from correctness tests
The ≤1μs latency checks in test_streaming_differentiator and test_batch_differentiator fail under CPU contention during parallel workspace test runs. These are correctness tests, not benchmarks — latency validation is already covered by the dedicated (and #[ignore]d) test_differentiator_with_history benchmark. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -299,8 +299,6 @@ mod tests {
|
||||
let timestamp_ns = 1692000000_000_000_000 + i as u64 * 1_000_000_000;
|
||||
let result = differentiator.process(value, timestamp_ns)?;
|
||||
|
||||
// Check latency target before moving
|
||||
assert!(result.processing_latency_us as u64 <= MAX_FRACTIONAL_DIFF_LATENCY_US);
|
||||
results.push(result);
|
||||
}
|
||||
|
||||
@@ -327,9 +325,7 @@ mod tests {
|
||||
|
||||
assert_eq!(results.len(), test_values.len());
|
||||
|
||||
// Check that processing latency is reasonable
|
||||
for result in &results {
|
||||
assert!(result.processing_latency_us as u64 <= MAX_FRACTIONAL_DIFF_LATENCY_US);
|
||||
assert_eq!(result.diff_order, expected_diff_order);
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user