-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Closed
Labels
status: pending releaseThe fix have been merged but not yet released to PyPIThe fix have been merged but not yet released to PyPItype: bugtype: documentation
Description
This issue was reported by AWS customer Nishank Singla from GE Healthcare.
This is a small correction for some default variables used in the TensorFlow Estimator class.
It looks like in the TF 1.12 upgrade, someone meant to replace TF_VERSION
with LATEST_VERSION
:
LATEST_VERSION = '1.12' |
TF_VERSION is still set to 1.11:
TF_VERSION = '1.11' |
And LATEST_VERSION is not actually used as the default in the constructor:
sagemaker-python-sdk/src/sagemaker/tensorflow/estimator.py
Lines 231 to 233 in 2763f9a
if framework_version is None: | |
LOGGER.warning(fw.empty_framework_version_warning(TF_VERSION, TF_VERSION)) | |
self.framework_version = framework_version or TF_VERSION |
Metadata
Metadata
Assignees
Labels
status: pending releaseThe fix have been merged but not yet released to PyPIThe fix have been merged but not yet released to PyPItype: bugtype: documentation