Address pr comments

Author: icywang86rui

CHANGELOG.rst

@@ -13,7 +13,7 @@ CHANGELOG
 * build: added pylint
 * build: upgrade docker-compose to 1.23
 * enhancement: Frameworks: update warning for not setting framework_version as we aren't planning a breaking change anymore
-* feature: Estimator: add script mode and py3 support for TensorFlow
+* feature: Estimator: add script mode and Python 3 support for TensorFlow
 
 1.14.1
 ======

src/sagemaker/fw_utils.py

@@ -31,7 +31,7 @@
                                   'If you would like to use version {latest}, ' \
                                   'please add framework_version={latest} to your constructor.'
 
-EMPTY_FRAMEWORK_VERSION_ERROR = 'framework_version is required for this estimator. ' \
+EMPTY_FRAMEWORK_VERSION_ERROR = 'framework_version is required for script mode estimator. ' \
                                 'Please add framework_version={} to your constructor to avoid this error.'
 
 VALID_PY_VERSIONS = ['py2', 'py3']

src/sagemaker/tensorflow/estimator.py

@@ -318,7 +318,7 @@ def _prepare_init_params_from_job_description(cls, job_details, model_channel_na
                                                                                        model_channel_name)
 
         # Move some of the tensorflow specific init params from hyperparameters into the main init params.
-        for argument in ['checkpoint_path', 'training_steps', 'evaluation_steps', 'model_dir']:
+        for argument in ('checkpoint_path', 'training_steps', 'evaluation_steps', 'model_dir'):
             value = init_params['hyperparameters'].pop(argument, None)
             if value is not None:
                 init_params[argument] = value