Skip to content

Commit bae3b8b

Browse files
mariumofMarius Moisescu
and
Marius Moisescu
authored
fix: Do not specify S3 path for disabled profiler (#3546)
Co-authored-by: Marius Moisescu <[email protected]>
1 parent 39918f4 commit bae3b8b

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

src/sagemaker/debugger/profiler_config.py

+5-1
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 False
97+
):
9498
profiler_config_request["S3OutputPath"] = self.s3_output_path
9599

96100
profiler_config_request["DisableProfiler"] = self.disable_profiler

0 commit comments

Comments
 (0)