# Copyright (c) 2023, NVIDIA CORPORATION. All rights reserved. # # Licensed under the Apache License, Version 2.0 (the "License"); # you may not use this file except in compliance with the License. # You may obtain a copy of the License at # # http://www.apache.org/licenses/LICENSE-2.0 # # Unless required by applicable law or agreed to in writing, software # distributed under the License is distributed on an "AS IS" BASIS, # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. # See the License for the specific language governing permissions and # limitations under the License. import logging import time from threading import Event, Thread import gevent import numpy as np import pytest from gevent.hub import Hub as GeventHub from pytriton.client import FuturesModelClient, ModelClient from pytriton.client.exceptions import ( PyTritonClientClosedError, PyTritonClientInvalidUrlError, PyTritonClientQueueFullError, PyTritonClientTimeoutError, PyTritonClientValueError, ) from pytriton.model_config import DeviceKind from pytriton.model_config.triton_model_config import TensorSpec, TritonModelConfig from .client_common import ( ADD_SUB_WITH_BATCHING_MODEL_CONFIG, GRPC_LOCALHOST_URL, HTTP_LOCALHOST_URL, patch_server_model_addsub_no_batch_ready, ) from .utils import ( patch_grpc_client__model_up_and_ready, patch_grpc_client__server_up_and_ready, patch_http_client__model_up_and_ready, patch_http_client__server_up_and_ready, ) logging.basicConfig(level=logging.DEBUG) LOGGER = logging.getLogger("test_sync_client") ADD_SUB_WITHOUT_BATCHING_MODEL_CONFIG = TritonModelConfig( model_name="AddSub", model_version=1, batching=False, instance_group={DeviceKind.KIND_CPU: 1}, inputs=[ TensorSpec(name="a", shape=(1,), dtype=np.float32), TensorSpec(name="b", shape=(1,), dtype=np.float32), ], outputs=[ TensorSpec(name="add", shape=(1,), dtype=np.float32), TensorSpec(name="sub", shape=(1,), dtype=np.float32), ], backend_parameters={"shared-memory-socket": "dummy/path"}, ) logging.basicConfig(level=logging.DEBUG) LOGGER = logging.getLogger("test_sync_client") def test_wait_for_model_raise_error_when_invalid_url_provided(): with pytest.raises(PyTritonClientInvalidUrlError, match="Invalid url"): with FuturesModelClient(["localhost:8001"], "dummy") as client: # pytype: disable=wrong-arg-types client.wait_for_model(timeout_s=0.1).result() @patch_server_model_addsub_no_batch_ready def test_wait_for_model_passes_timeout_to_client(mocker): spy_client_close = mocker.spy(ModelClient, ModelClient.close.__name__) mock_client_wait_for_model = mocker.patch.object(ModelClient, ModelClient.wait_for_model.__name__) mock_client_wait_for_model.return_value = True spy_thread_start = mocker.spy(Thread, Thread.start.__name__) spy_thread_join = mocker.spy(Thread, Thread.join.__name__) spy_get_hub = mocker.spy(gevent, gevent.get_hub.__name__) spy_hub_destroy = mocker.spy(GeventHub, GeventHub.destroy.__name__) with FuturesModelClient( GRPC_LOCALHOST_URL, ADD_SUB_WITH_BATCHING_MODEL_CONFIG.model_name, str(ADD_SUB_WITH_BATCHING_MODEL_CONFIG.model_version), max_workers=1, ) as client: future = client.wait_for_model(15) result = future.result() assert result is True spy_client_close.assert_called_once() mock_client_wait_for_model.assert_called_with(15) spy_thread_start.assert_called_once() spy_thread_join.assert_called_once() spy_get_hub.assert_called_once() spy_hub_destroy.assert_called_once() @patch_server_model_addsub_no_batch_ready def test_infer_raises_error_when_mixed_args_convention_used(mocker): a = np.array([1], dtype=np.float32) b = np.array([1], dtype=np.float32) init_t_timeout_s = 15.0 with FuturesModelClient( GRPC_LOCALHOST_URL, ADD_SUB_WITH_BATCHING_MODEL_CONFIG.model_name, init_timeout_s=init_t_timeout_s ) as client: with pytest.raises( PyTritonClientValueError, match="Use either positional either keyword method arguments convention", ): client.infer_sample(a, b=b).result() with FuturesModelClient(GRPC_LOCALHOST_URL, ADD_SUB_WITH_BATCHING_MODEL_CONFIG.model_name) as client: with pytest.raises( PyTritonClientValueError, match="Use either positional either keyword method arguments convention", ): client.infer_batch(a, b=b).result() @patch_server_model_addsub_no_batch_ready def test_infer_sample_returns_values_creates_client(mocker): a = np.array([1], dtype=np.float32) b = np.array([2], dtype=np.float32) c = np.array([3], dtype=np.float32) init_t_timeout_s = 15.0 mock_client_wait_for_model = mocker.patch.object(ModelClient, ModelClient._wait_and_init_model_config.__name__) mock_client_infer_sample = mocker.patch.object(ModelClient, ModelClient.infer_sample.__name__) mock_client_infer_sample.return_value = c with FuturesModelClient( GRPC_LOCALHOST_URL, ADD_SUB_WITH_BATCHING_MODEL_CONFIG.model_name, init_timeout_s=init_t_timeout_s ) as client: result = client.infer_sample(a=a, b=b).result() mock_client_wait_for_model.assert_called_once_with(init_t_timeout_s) mock_client_infer_sample.assert_called_once_with(parameters=None, headers=None, a=a, b=b) # Check the Python version and use different assertions for cancel_futures assert result == c @patch_server_model_addsub_no_batch_ready def test_infer_sample_returns_values_creates_client_close_wait(mocker): a = np.array([1], dtype=np.float32) b = np.array([2], dtype=np.float32) c = np.array([3], dtype=np.float32) mock_client_infer_sample = mocker.patch.object(ModelClient, ModelClient.infer_sample.__name__) # Prevent exit from closing the client mocker.patch.object(FuturesModelClient, FuturesModelClient.__exit__.__name__) mock_client_infer_sample.return_value = c client = FuturesModelClient(GRPC_LOCALHOST_URL, ADD_SUB_WITH_BATCHING_MODEL_CONFIG.model_name) result = client.infer_sample(a, b).result() client.close(wait=True) mock_client_infer_sample.assert_called_once_with(a, b, parameters=None, headers=None) assert result == c @patch_server_model_addsub_no_batch_ready def test_infer_batch_returns_values_creates_client(mocker): a = np.array([1], dtype=np.float32) b = np.array([2], dtype=np.float32) c = np.array([3], dtype=np.float32) init_t_timeout_s = 15.0 mock_client_infer_batch = mocker.patch.object(ModelClient, ModelClient.infer_batch.__name__) mock_client_infer_batch.return_value = c with FuturesModelClient( GRPC_LOCALHOST_URL, ADD_SUB_WITH_BATCHING_MODEL_CONFIG.model_name, init_timeout_s=init_t_timeout_s ) as client: result = client.infer_batch(a=a, b=b).result() model_config = client.model_config().result() mock_client_infer_batch.assert_called_once_with(parameters=None, headers=None, a=a, b=b) assert model_config.model_name == ADD_SUB_WITH_BATCHING_MODEL_CONFIG.model_name assert result == c @patch_server_model_addsub_no_batch_ready def test_infer_sample_list_passed_arguments_returns_arguments(mocker): a = np.array([1], dtype=np.float32) b = np.array([2], dtype=np.float32) ret = np.array([3], dtype=np.float32) patch_client_infer_sample = mocker.patch.object(ModelClient, ModelClient.infer_sample.__name__) patch_client_infer_sample.return_value = ret with FuturesModelClient(GRPC_LOCALHOST_URL, ADD_SUB_WITH_BATCHING_MODEL_CONFIG.model_name) as client: return_value = client.infer_sample(a, b).result() assert return_value == ret patch_client_infer_sample.assert_called_once_with(a, b, parameters=None, headers=None) @patch_server_model_addsub_no_batch_ready def test_infer_sample_dict_passed_arguments_returns_arguments(mocker): a = np.array([1], dtype=np.float32) b = np.array([2], dtype=np.float32) ret = np.array([3], dtype=np.float32) patch_client_infer_sample = mocker.patch.object(ModelClient, ModelClient.infer_sample.__name__) patch_client_infer_sample.return_value = ret with FuturesModelClient(GRPC_LOCALHOST_URL, ADD_SUB_WITH_BATCHING_MODEL_CONFIG.model_name) as client: return_value = client.infer_sample(a=a, b=b).result() assert return_value == ret patch_client_infer_sample.assert_called_once_with(a=a, b=b, parameters=None, headers=None) @patch_server_model_addsub_no_batch_ready def test_infer_batch_list_passed_arguments_returns_arguments(mocker): a = np.array([1], dtype=np.float32) b = np.array([2], dtype=np.float32) ret = np.array([3], dtype=np.float32) patch_client_infer_batch = mocker.patch.object(ModelClient, ModelClient.infer_batch.__name__) patch_client_infer_batch.return_value = ret with FuturesModelClient(GRPC_LOCALHOST_URL, ADD_SUB_WITH_BATCHING_MODEL_CONFIG.model_name) as client: return_value = client.infer_batch(a, b).result() assert return_value == ret patch_client_infer_batch.assert_called_once_with(a, b, parameters=None, headers=None) @patch_server_model_addsub_no_batch_ready def test_infer_batch_dict_passed_arguments_returns_arguments(mocker): a = np.array([1], dtype=np.float32) b = np.array([2], dtype=np.float32) ret = np.array([3], dtype=np.float32) patch_client_infer_batch = mocker.patch.object(ModelClient, ModelClient.infer_batch.__name__) patch_client_infer_batch.return_value = ret with FuturesModelClient(GRPC_LOCALHOST_URL, ADD_SUB_WITH_BATCHING_MODEL_CONFIG.model_name) as client: return_value = client.infer_batch(a=a, b=b).result() assert return_value == ret patch_client_infer_batch.assert_called_once_with(parameters=None, headers=None, a=a, b=b) @patch_server_model_addsub_no_batch_ready def test_infer_batch_blocking_behaviour(mocker): a = np.array([1], dtype=np.float32) b = np.array([2], dtype=np.float32) c = np.array([2], dtype=np.float32) ret = np.array([3], dtype=np.float32) # Set up the queue return values to block the queue and then release it infer_called_with_b_event = Event() infer_called_with_c_event = Event() queue_is_full_event = Event() def mock_submit_side_effect(*args, **kwargs): LOGGER.debug("mock_submit_side_effect called") assert "b" in kwargs if kwargs["b"] is b: infer_called_with_b_event.set() elif kwargs["b"] is c: infer_called_with_c_event.set() if not queue_is_full_event.is_set(): LOGGER.debug("mock_submit_side_effect waiting for queue to be full") queue_is_full_event.wait() # Block until the event is set LOGGER.debug("mock_submit_side_effect returning") return ret patch_client_infer_batch = mocker.patch.object(ModelClient, ModelClient.infer_batch.__name__) patch_client_infer_batch.side_effect = mock_submit_side_effect # Set up the client with a max_queue_size of 1 to easily simulate full condition with FuturesModelClient( GRPC_LOCALHOST_URL, ADD_SUB_WITH_BATCHING_MODEL_CONFIG.model_name, max_workers=1, max_queue_size=1, non_blocking=False, ) as client: client.model_config().result() # Wait for the model to be ready LOGGER.debug("Client created") first_future = client.infer_batch(a=a, b=b) LOGGER.debug("First future created") infer_called_with_b_event.wait() # Wait for the first call to be made LOGGER.debug("First call made") blocked_thread_start_event = Event() blocked_thread_result = {} def blocked_thread(): LOGGER.debug("Blocked thread started") blocked_thread_start_event.set() LOGGER.debug("Blocked thread waiting for queue to be full") result = client.infer_batch(a=a, b=c).result() LOGGER.debug("Blocked thread got result") blocked_thread_result["ret"] = result infer_thread = Thread(target=blocked_thread) infer_thread.start() LOGGER.debug("Waiting for blocked thread to start") blocked_thread_start_event.wait() # Wait for the thread to start LOGGER.debug("Blocked thread started") time.sleep(0.1) # Wait a bit to make sure the thread is blocked assert not infer_called_with_c_event.is_set(), "infer_batch should not have been called with c yet." # The blocking call should be waiting by now, so let's release the block LOGGER.debug("Releasing queue") queue_is_full_event.set() # Wait for the blocked thread to finish LOGGER.debug("Waiting for blocked thread to finish") infer_thread.join() assert blocked_thread_result["ret"] is ret # Wait for the first future to finish assert first_future.result() is ret assert ( patch_client_infer_batch.call_count == 2 ), "infer_batch should have been called twice (one blocked, one released)." @patch_server_model_addsub_no_batch_ready def test_infer_batch_non_blocking_behaviour(mocker): a = np.array([1], dtype=np.float32) b = np.array([2], dtype=np.float32) c = np.array([2], dtype=np.float32) ret = np.array([3], dtype=np.float32) # Set up the queue return values to block the queue and then release it infer_called_with_b_event = Event() queue_is_full_event = Event() def mock_submit_side_effect(*args, **kwargs): LOGGER.debug("mock_submit_side_effect called") infer_called_with_b_event.set() if not queue_is_full_event.is_set(): LOGGER.debug("mock_submit_side_effect waiting for queue to be full") queue_is_full_event.wait() # Block until the event is set LOGGER.debug("mock_submit_side_effect returning") return ret patch_client_infer_batch = mocker.patch.object(ModelClient, ModelClient.infer_batch.__name__) patch_client_infer_batch.side_effect = mock_submit_side_effect # Set up the client with a max_queue_size of 1 to easily simulate full condition with FuturesModelClient( GRPC_LOCALHOST_URL, ADD_SUB_WITH_BATCHING_MODEL_CONFIG.model_name, max_workers=1, max_queue_size=1, non_blocking=True, ) as client: LOGGER.debug("Client created") while True: try: client.model_config().result() # Wait for the model to be ready break except PyTritonClientQueueFullError: LOGGER.debug("Waiting for model to be ready") time.sleep(0.1) pass first_future = client.infer_batch(a=a, b=b) LOGGER.debug("First future created") infer_called_with_b_event.wait() # Wait for the first call to be made LOGGER.debug("First call made") second_future = client.infer_batch(a=a, b=c) LOGGER.debug("Second future created") with pytest.raises(PyTritonClientQueueFullError): LOGGER.debug("Calling infer_batch with queue full") client.infer_batch(a=a, b=c) # The blocking call should be waiting by now, so let's release the block LOGGER.debug("Releasing queue") queue_is_full_event.set() # Wait for the first future to finish assert first_future.result() is ret assert second_future.result() is ret assert patch_client_infer_batch.call_count == 2, "infer_batch should have been called once." @patch_server_model_addsub_no_batch_ready def test_infer_batch_queue_timeout(mocker): a = np.array([1], dtype=np.float32) b = np.array([2], dtype=np.float32) c = np.array([2], dtype=np.float32) ret = np.array([3], dtype=np.float32) # Set up the queue return values to block the queue and then release it infer_called_with_b_event = Event() queue_is_full_event = Event() def mock_submit_side_effect(*args, **kwargs): LOGGER.debug("mock_submit_side_effect called") infer_called_with_b_event.set() if not queue_is_full_event.is_set(): LOGGER.debug("mock_submit_side_effect waiting for queue to be full") queue_is_full_event.wait() # Block until the event is set LOGGER.debug("mock_submit_side_effect returning") return ret patch_client_infer_batch = mocker.patch.object(ModelClient, ModelClient.infer_batch.__name__) patch_client_infer_batch.side_effect = mock_submit_side_effect # Set up the client with a max_queue_size of 1 to easily simulate full condition with FuturesModelClient( GRPC_LOCALHOST_URL, ADD_SUB_WITH_BATCHING_MODEL_CONFIG.model_name, max_workers=1, max_queue_size=1, inference_timeout_s=0.1, ) as client: LOGGER.debug("Client created") client.model_config().result() # Wait for the model to be ready first_future = client.infer_batch(a=a, b=b) LOGGER.debug("First future created") infer_called_with_b_event.wait() # Wait for the first call to be made LOGGER.debug("First call made") second_future = client.infer_batch(a=a, b=c) LOGGER.debug("Second future created") with pytest.raises(PyTritonClientQueueFullError): LOGGER.debug("Calling infer_batch with queue full") client.infer_batch(a=a, b=c) # The blocking call should be waiting by now, so let's release the block LOGGER.debug("Releasing queue") queue_is_full_event.set() # Wait for the first future to finish assert first_future.result() is ret assert second_future.result() is ret assert patch_client_infer_batch.call_count == 2, "infer_batch should have been called once." def test_init_raises_error_when_invalid_max_workers_provided(mocker): with pytest.raises(ValueError): with FuturesModelClient(GRPC_LOCALHOST_URL, ADD_SUB_WITH_BATCHING_MODEL_CONFIG.model_name, max_workers=-1): pass def test_init_raises_error_when_invalid_max_queue_size_provided(mocker): with pytest.raises(ValueError): with FuturesModelClient(GRPC_LOCALHOST_URL, ADD_SUB_WITH_BATCHING_MODEL_CONFIG.model_name, max_queue_size=-1): pass @pytest.mark.timeout(1.0) def test_init_http_passes_timeout(mocker): with FuturesModelClient("http://localhost:6669", "dummy", init_timeout_s=0.2, inference_timeout_s=0.1) as client: with pytest.raises(PyTritonClientTimeoutError): client.wait_for_model(timeout_s=0.2).result() @pytest.mark.timeout(5) def test_init_grpc_passes_timeout_5(mocker): with FuturesModelClient("grpc://localhost:6669", "dummy", init_timeout_s=0.2, inference_timeout_s=0.1) as client: with pytest.raises(PyTritonClientTimeoutError): client.wait_for_model(timeout_s=0.2).result() @pytest.mark.timeout(5) def test_init_http_spaws_several_threads(mocker): spy_thread_start = mocker.spy(Thread, Thread.start.__name__) with FuturesModelClient("http://localhost:6669", "dummy", init_timeout_s=1, inference_timeout_s=0.2) as client: timeout_s = 0.2 # The list function is used to force the evaluation of the list comprehension before iterating over the futures and # calling their result method. This is done to ensure that all the calls occur before the iteration starts, # and to verify that five threads are created. futures = list([client.wait_for_model(timeout_s=timeout_s) for _ in range(5)]) # noqa: C411 for future in futures: with pytest.raises(PyTritonClientTimeoutError): future.result() # Reusing client configuration from existing clients forces wait in other threads to finish first configuration # request. It sometimes prevents creation of a fifth thread because one of the existing threads can handle another request # before the new thread is created. This results in a race condition that affects the number of created threads. assert spy_thread_start.call_count > 1 def test_http_client_raises_error_when_used_after_close(mocker): patch_http_client__server_up_and_ready(mocker) patch_http_client__model_up_and_ready(mocker, ADD_SUB_WITH_BATCHING_MODEL_CONFIG) with ModelClient(HTTP_LOCALHOST_URL, "dummy") as client: pass with pytest.raises(PyTritonClientClosedError): client.wait_for_model(timeout_s=0.2) a = np.array([1], dtype=np.float32) with pytest.raises(PyTritonClientClosedError): client.infer_sample(a=a) with pytest.raises(PyTritonClientClosedError): client.infer_batch(a=[a]) def test_grpc_client_raises_error_when_used_after_close(mocker): patch_grpc_client__server_up_and_ready(mocker) patch_grpc_client__model_up_and_ready(mocker, ADD_SUB_WITH_BATCHING_MODEL_CONFIG) with FuturesModelClient(GRPC_LOCALHOST_URL, "dummy") as client: pass with pytest.raises(PyTritonClientClosedError): client.wait_for_model(timeout_s=0.2).result() a = np.array([1], dtype=np.float32) with pytest.raises(PyTritonClientClosedError): client.infer_sample(a=a).result() with pytest.raises(PyTritonClientClosedError): client.infer_batch(a=[a]).result()