Skip to content

Commit 0529828

Browse files
committed
Only define override when needed
1 parent c053583 commit 0529828

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

src/sagemaker/tensorflow/estimator.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -177,9 +177,11 @@ def __init__(
177177
fw.python_deprecation_warning(self._framework_name, defaults.LATEST_PY2_VERSION)
178178
)
179179
self.framework_version = framework_version
180-
self.override_fw_version = "2.16" if self.framework_version and "2.16" in self.framework_version else None
181-
## TODO: Revert
182-
print(f"ABCD123 setting self.override_fw_version to {self.override_fw_version}")
180+
if self.framework_version and "2.16" in self.framework_version:
181+
self.override_fw_version="2.16"
182+
## TODO: Revert
183+
print(f"ABCD123 setting self.override_fw_version to {self.override_fw_version}")
184+
183185
# TF training and inference versions do not have a one-to-one connection. This mismatch
184186
# is accommodated by the underlying dictionary. The key of dictionary relates to the inference
185187
# version and the value relates to training version.

0 commit comments

Comments
 (0)