Skip to content

test: accept exact queue-delay boundary (>=) in seq-batcher timing as… #8899

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
mc-nv merged 5 commits into from
Aug 5, 2026
+14 −10
Merged

test: accept exact queue-delay boundary (>=) in seq-batcher timing as… #8899

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
934cd7d
test: accept exact queue-delay boundary (>=) in seq-batcher timing as…
akhilraj9 Jul 22, 2026
205dc42
fix: update copyright year and align error messages with >= operator
akhilraj9 Jul 23, 2026
f32047e
fix: update copyright year to 2026 in sequence_util.py
akhilraj9 Jul 23, 2026
c151b58
fix: resolve flake8 F403/F405/F824 in sequence_util.py
akhilraj9 Jul 30, 2026
42db918
Merge branch 'main' into asaraswathi/tri-1372-fix-ci-test-l0_sequence…
pskiran1 Jul 30, 2026
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
24 changes: 14 additions & 10 deletions qa/common/sequence_util.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#!/usr/bin/env python3

# Copyright 2019-2025, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
# Copyright 2019-2026, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
#
# Redistribution and use in source and binary forms, with or without
# modification, are permitted provided that the following conditions
Expand Down Expand Up @@ -40,6 +40,11 @@
import tritonclient.http as httpclient
from tritonclient.utils import *

# QA-suite convention: tritonclient.utils is imported via `*` here and in 20+
# other QA files. Silence flake8's star-import warnings (F403/F405) for this
# file only; proper cleanup is tracked separately.
# flake8: noqa: F403,F405

@akhilraj9 akhilraj9 Jul 30, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Scoped # flake8: noqa: F403,F405 for the QA-suite import * convention. Note it also masks two pre-existing in0→dummy_in0 typos in the string-tensor path — tracked for a proper fix in TRI-1643.

if sys.version_info >= (3, 0):
import queue
else:
Expand Down Expand Up @@ -93,7 +98,6 @@ def clear_deferred_exceptions(self):
_deferred_exceptions = []

def add_deferred_exception(self, ex):
global _deferred_exceptions
with _deferred_exceptions_lock:
_deferred_exceptions.append(ex)

Expand Down Expand Up @@ -674,8 +678,8 @@ def check_sequence(
)
if gt_ms is not None:
self.assertTrue(
(end_ms - start_ms) > gt_ms,
"expected greater than "
(end_ms - start_ms) >= gt_ms,
"expected greater than or equal to "
+ str(gt_ms)
+ "ms response time, got "
+ str(end_ms - start_ms)
Expand Down Expand Up @@ -707,8 +711,8 @@ def check_sequence(
)
if gt_ms is not None:
self.assertTrue(
(seq_end_ms - seq_start_ms) > gt_ms,
"sequence expected greater than "
(seq_end_ms - seq_start_ms) >= gt_ms,
"sequence expected greater than or equal to "
+ str(gt_ms)
+ "ms response time, got "
+ str(seq_end_ms - seq_start_ms)
Expand Down Expand Up @@ -883,8 +887,8 @@ def check_sequence_async(
)
if gt_ms is not None:
self.assertTrue(
(seq_end_ms - seq_start_ms) > gt_ms,
"sequence expected greater than "
(seq_end_ms - seq_start_ms) >= gt_ms,
"sequence expected greater than or equal to "
+ str(gt_ms)
+ "ms response time, got "
+ str(seq_end_ms - seq_start_ms)
Expand Down Expand Up @@ -1100,8 +1104,8 @@ def check_sequence_shape_tensor_io(
)
if gt_ms is not None:
self.assertTrue(
(seq_end_ms - seq_start_ms) > gt_ms,
"sequence expected greater than "
(seq_end_ms - seq_start_ms) >= gt_ms,
"sequence expected greater than or equal to "
+ str(gt_ms)
+ "ms response time, got "
+ str(seq_end_ms - seq_start_ms)
Expand Down
Loading