From 91ed516b7d57a95f82a56032f5cc908c1b3732fc Mon Sep 17 00:00:00 2001 From: yifeim Date: Thu, 14 Oct 2021 14:10:08 -0700 Subject: [PATCH 1/3] install requirements from relative path this allows it to correctly find the path to install from whl files. --- src/sagemaker_inference/model_server.py | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/src/sagemaker_inference/model_server.py b/src/sagemaker_inference/model_server.py index 6e8adc5..fc9b403 100644 --- a/src/sagemaker_inference/model_server.py +++ b/src/sagemaker_inference/model_server.py @@ -180,10 +180,15 @@ def _terminate(signo, frame): # pylint: disable=unused-argument def _install_requirements(): logger.info("installing packages from requirements.txt...") - pip_install_cmd = [sys.executable, "-m", "pip", "install", "-r", REQUIREMENTS_PATH] + if sys.version_info >= (3, 5): + pip_install_cmd = [sys.executable, "-m", "pip", "install", "-r", "requirements.txt"] + kw = {"cwd": code_dir} + else: + pip_install_cmd = [sys.executable, "-m", "pip", "install", "-r", REQUIREMENTS_PATH] + kw = {} try: - subprocess.check_call(pip_install_cmd) + subprocess.check_call(pip_install_cmd, **kw) except subprocess.CalledProcessError: logger.error("failed to install required packages, exiting") raise ValueError("failed to install required packages") From e1dd62e0d379bb5c89cb1f9ec3ba125eed086fc8 Mon Sep 17 00:00:00 2001 From: yifeim Date: Thu, 14 Oct 2021 16:55:33 -0700 Subject: [PATCH 2/3] check_call cwd works since py36 --- src/sagemaker_inference/model_server.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/sagemaker_inference/model_server.py b/src/sagemaker_inference/model_server.py index fc9b403..b432573 100644 --- a/src/sagemaker_inference/model_server.py +++ b/src/sagemaker_inference/model_server.py @@ -180,7 +180,7 @@ def _terminate(signo, frame): # pylint: disable=unused-argument def _install_requirements(): logger.info("installing packages from requirements.txt...") - if sys.version_info >= (3, 5): + if sys.version_info >= (3, 6): pip_install_cmd = [sys.executable, "-m", "pip", "install", "-r", "requirements.txt"] kw = {"cwd": code_dir} else: From c008970239b2e8f8d53d8fd929368e193bd993a2 Mon Sep 17 00:00:00 2001 From: Yifei Ma Date: Fri, 15 Oct 2021 14:48:42 -0700 Subject: [PATCH 3/3] fix: fix whl installation error from inference requirements