Skip to content

Commit 16605c1

Browse files
author
Marius Moisescu
committed
fix: do not specify S3 path for disabled profiler
1 parent ea0d053 commit 16605c1

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

src/sagemaker/debugger/profiler_config.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,11 @@ def _to_request_dict(self):
9090
"""
9191
profiler_config_request = {}
9292

93-
if self.s3_output_path is not None:
93+
if (
94+
self.s3_output_path is not None
95+
and self.disable_profiler is not None
96+
and self.disable_profiler is True
97+
):
9498
profiler_config_request["S3OutputPath"] = self.s3_output_path
9599

96100
profiler_config_request["DisableProfiler"] = self.disable_profiler

0 commit comments

Comments
 (0)