Skip to content

Commit ec2822b

Browse files
committed
fix: remove public function for pkl deserialization
1 parent 2ea6fa0 commit ec2822b

File tree

4 files changed

+1
-437
lines changed

4 files changed

+1
-437
lines changed

src/sagemaker/serve/detector/dependency_manager.py

Lines changed: 1 addition & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -34,27 +34,7 @@ def capture_dependencies(dependencies: dict, work_dir: Path, capture_all: bool =
3434
"""Placeholder docstring"""
3535
path = work_dir.joinpath("requirements.txt")
3636
if "auto" in dependencies and dependencies["auto"]:
37-
command = [
38-
sys.executable,
39-
Path(__file__).parent.joinpath("pickle_dependencies.py"),
40-
"--pkl_path",
41-
work_dir.joinpath(PKL_FILE_NAME),
42-
"--dest",
43-
path,
44-
]
45-
46-
if capture_all:
47-
command.append("--capture_all")
48-
49-
subprocess.run(
50-
command,
51-
env={"SETUPTOOLS_USE_DISTUTILS": "stdlib"},
52-
check=True,
53-
)
54-
55-
with open(path, "r") as f:
56-
autodetect_depedencies = f.read().splitlines()
57-
autodetect_depedencies.append("sagemaker[huggingface]>=2.199")
37+
raise ValueError("Auto requirements.txt detection is temporarily disabled in this version of sagemaker")
5838
else:
5939
autodetect_depedencies = ["sagemaker[huggingface]>=2.199"]
6040

src/sagemaker/serve/detector/pickle_dependencies.py

Lines changed: 0 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -100,25 +100,6 @@ def get_currently_used_packages():
100100
return currently_used_packages
101101

102102

103-
def get_requirements_for_pkl_file(pkl_path: Path, dest: Path):
104-
"""Placeholder docstring"""
105-
with open(pkl_path, mode="rb") as file:
106-
cloudpickle.load(file)
107-
108-
currently_used_packages = get_currently_used_packages()
109-
110-
with open(dest, mode="w+") as out:
111-
for x in get_all_installed_packages():
112-
name = x["name"]
113-
version = x["version"]
114-
# skip only for dev
115-
if name == "boto3":
116-
boto3_version = boto3.__version__
117-
out.write(f"boto3=={boto3_version}\n")
118-
elif name in currently_used_packages:
119-
out.write(f"{name}=={version}\n")
120-
121-
122103
def get_all_requirements(dest: Path):
123104
"""Placeholder docstring"""
124105
all_installed_packages = get_all_installed_packages()
@@ -146,15 +127,3 @@ def parse_args():
146127
args = parser.parse_args()
147128
return (Path(args.pkl_path), Path(args.dest), args.capture_all)
148129

149-
150-
def main():
151-
"""Placeholder docstring"""
152-
pkl_path, dest, capture_all = parse_args()
153-
if capture_all:
154-
get_all_requirements(dest)
155-
else:
156-
get_requirements_for_pkl_file(pkl_path, dest)
157-
158-
159-
if __name__ == "__main__":
160-
main()

tests/unit/sagemaker/serve/detector/test_dependency_manager.py

Lines changed: 0 additions & 141 deletions
This file was deleted.

0 commit comments

Comments
 (0)