fix: Do not specify S3 path for disabled profiler (#3546)

Co-authored-by: Marius Moisescu <[email protected]>
aws · Dec 19, 2022 · bae3b8b · bae3b8b
1 parent 39918f4
commit bae3b8b
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/sagemaker/debugger/profiler_config.py b/src/sagemaker/debugger/profiler_config.py
@@ -90,7 +90,11 @@ def _to_request_dict(self):
         """
         profiler_config_request = {}
 
-        if self.s3_output_path is not None:
+        if (
+            self.s3_output_path is not None
+            and self.disable_profiler is not None
+            and self.disable_profiler is False
+        ):
             profiler_config_request["S3OutputPath"] = self.s3_output_path
 
         profiler_config_request["DisableProfiler"] = self.disable_profiler