Skip to content

Commit fd10a25

Browse files
authored
fix: use correct TF version in empty framework_version warning (#756)
1 parent 397fb90 commit fd10a25

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/sagemaker/tensorflow/estimator.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -228,7 +228,7 @@ def __init__(self, training_steps=None, evaluation_steps=None, checkpoint_path=N
228228
**kwargs: Additional kwargs passed to the Framework constructor.
229229
"""
230230
if framework_version is None:
231-
LOGGER.warning(fw.empty_framework_version_warning(TF_VERSION, TF_VERSION))
231+
LOGGER.warning(fw.empty_framework_version_warning(TF_VERSION, self.LATEST_VERSION))
232232
self.framework_version = framework_version or TF_VERSION
233233

234234
super(TensorFlow, self).__init__(image_name=image_name, **kwargs)

tests/unit/test_tf_estimator.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -697,7 +697,7 @@ def test_empty_framework_version(warning, sagemaker_session):
697697
framework_version=None)
698698

699699
assert estimator.framework_version == defaults.TF_VERSION
700-
warning.assert_called_with(defaults.TF_VERSION, defaults.TF_VERSION)
700+
warning.assert_called_with(defaults.TF_VERSION, estimator.LATEST_VERSION)
701701

702702

703703
def _deprecated_args_msg(args):

0 commit comments

Comments
 (0)