Skip to content

Commit 77a6523

Browse files
author
Chuyang Deng
committed
add py2 deprecation message to estimators and models
1 parent 874b84f commit 77a6523

File tree

8 files changed

+48
-0
lines changed

8 files changed

+48
-0
lines changed

src/sagemaker/chainer/estimator.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,10 @@ def __init__(self, entry_point, use_mpi=None, num_processes=None, process_slots_
9090

9191
super(Chainer, self).__init__(entry_point, source_dir, hyperparameters,
9292
image_name=image_name, **kwargs)
93+
94+
if py_version == 'py2':
95+
logger.warning('chainer py2 container will be deprecated soon.')
96+
9397
self.py_version = py_version
9498
self.use_mpi = use_mpi
9599
self.num_processes = num_processes

src/sagemaker/chainer/model.py

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,12 +12,16 @@
1212
# language governing permissions and limitations under the License.
1313
from __future__ import absolute_import
1414

15+
import logging
16+
1517
import sagemaker
1618
from sagemaker.fw_utils import create_image_uri, model_code_key_prefix
1719
from sagemaker.model import FrameworkModel, MODEL_SERVER_WORKERS_PARAM_NAME
1820
from sagemaker.chainer.defaults import CHAINER_VERSION
1921
from sagemaker.predictor import RealTimePredictor, npy_serializer, numpy_deserializer
2022

23+
logger = logging.getLogger('sagemaker')
24+
2125

2226
class ChainerPredictor(RealTimePredictor):
2327
"""A RealTimePredictor for inference against Chainer Endpoints.
@@ -66,6 +70,9 @@ def __init__(self, model_data, role, entry_point, image=None, py_version='py3',
6670
"""
6771
super(ChainerModel, self).__init__(model_data, image, role, entry_point, predictor_cls=predictor_cls,
6872
**kwargs)
73+
if py_version == 'py2':
74+
logger.warning('chainer py2 container will be deprecated soon.')
75+
6976
self.py_version = py_version
7077
self.framework_version = framework_version
7178
self.model_server_workers = model_server_workers

src/sagemaker/mxnet/estimator.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,10 @@ def __init__(self, entry_point, source_dir=None, hyperparameters=None, py_versio
7979

8080
super(MXNet, self).__init__(entry_point, source_dir, hyperparameters,
8181
image_name=image_name, **kwargs)
82+
83+
if py_version == 'py2':
84+
logger.warning('mxnet py2 container will be deprecated soon.')
85+
8286
self.py_version = py_version
8387
self._configure_distribution(distributions)
8488

src/sagemaker/mxnet/model.py

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,12 +12,16 @@
1212
# language governing permissions and limitations under the License.
1313
from __future__ import absolute_import
1414

15+
import logging
16+
1517
import sagemaker
1618
from sagemaker.fw_utils import create_image_uri, model_code_key_prefix
1719
from sagemaker.model import FrameworkModel, MODEL_SERVER_WORKERS_PARAM_NAME
1820
from sagemaker.mxnet.defaults import MXNET_VERSION
1921
from sagemaker.predictor import RealTimePredictor, json_serializer, json_deserializer
2022

23+
logger = logging.getLogger('sagemaker')
24+
2125

2226
class MXNetPredictor(RealTimePredictor):
2327
"""A RealTimePredictor for inference against MXNet Endpoints.
@@ -66,6 +70,10 @@ def __init__(self, model_data, role, entry_point, image=None, py_version='py2',
6670
"""
6771
super(MXNetModel, self).__init__(model_data, image, role, entry_point, predictor_cls=predictor_cls,
6872
**kwargs)
73+
74+
if py_version == 'py2':
75+
logger.warning('chainer py2 container will be deprecated soon.')
76+
6977
self.py_version = py_version
7078
self.framework_version = framework_version
7179
self.model_server_workers = model_server_workers

src/sagemaker/pytorch/estimator.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,6 +74,10 @@ def __init__(self, entry_point, source_dir=None, hyperparameters=None, py_versio
7474
self.framework_version = framework_version or PYTORCH_VERSION
7575

7676
super(PyTorch, self).__init__(entry_point, source_dir, hyperparameters, image_name=image_name, **kwargs)
77+
78+
if py_version == 'py2':
79+
logger.warning('pytorch py2 container will be deprecated soon.')
80+
7781
self.py_version = py_version
7882

7983
def create_model(self, model_server_workers=None, role=None, vpc_config_override=VPC_CONFIG_DEFAULT):

src/sagemaker/pytorch/model.py

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,12 +11,17 @@
1111
# ANY KIND, either express or implied. See the License for the specific
1212
# language governing permissions and limitations under the License.
1313
from __future__ import absolute_import
14+
15+
import logging
16+
1417
import sagemaker
1518
from sagemaker.fw_utils import create_image_uri, model_code_key_prefix
1619
from sagemaker.model import FrameworkModel, MODEL_SERVER_WORKERS_PARAM_NAME
1720
from sagemaker.pytorch.defaults import PYTORCH_VERSION, PYTHON_VERSION
1821
from sagemaker.predictor import RealTimePredictor, npy_serializer, numpy_deserializer
1922

23+
logger = logging.getLogger('sagemaker')
24+
2025

2126
class PyTorchPredictor(RealTimePredictor):
2227
"""A RealTimePredictor for inference against PyTorch Endpoints.
@@ -65,6 +70,10 @@ def __init__(self, model_data, role, entry_point, image=None, py_version=PYTHON_
6570
**kwargs: Keyword arguments passed to the ``FrameworkModel`` initializer.
6671
"""
6772
super(PyTorchModel, self).__init__(model_data, image, role, entry_point, predictor_cls=predictor_cls, **kwargs)
73+
74+
if py_version == 'py2':
75+
logger.warning('pytorch py2 container will be deprecated soon.')
76+
6877
self.py_version = py_version
6978
self.framework_version = framework_version
7079
self.model_server_workers = model_server_workers

src/sagemaker/tensorflow/estimator.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -236,6 +236,10 @@ def __init__(self, training_steps=None, evaluation_steps=None, checkpoint_path=N
236236

237237
super(TensorFlow, self).__init__(image_name=image_name, **kwargs)
238238
self.checkpoint_path = checkpoint_path
239+
240+
if py_version == 'py2':
241+
LOGGER.warning('tensorflow py2 container will be deprecated soon.')
242+
239243
self.py_version = py_version
240244
self.training_steps = training_steps
241245
self.evaluation_steps = evaluation_steps

src/sagemaker/tensorflow/model.py

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,13 +12,17 @@
1212
# language governing permissions and limitations under the License.
1313
from __future__ import absolute_import
1414

15+
import logging
16+
1517
import sagemaker
1618
from sagemaker.fw_utils import create_image_uri, model_code_key_prefix
1719
from sagemaker.model import FrameworkModel, MODEL_SERVER_WORKERS_PARAM_NAME
1820
from sagemaker.predictor import RealTimePredictor
1921
from sagemaker.tensorflow.defaults import TF_VERSION
2022
from sagemaker.tensorflow.predictor import tf_json_serializer, tf_json_deserializer
2123

24+
LOGGER = logging.getLogger('sagemaker')
25+
2226

2327
class TensorFlowPredictor(RealTimePredictor):
2428
"""A ``RealTimePredictor`` for inference against TensorFlow endpoint.
@@ -67,6 +71,10 @@ def __init__(self, model_data, role, entry_point, image=None, py_version='py2',
6771
"""
6872
super(TensorFlowModel, self).__init__(model_data, image, role, entry_point, predictor_cls=predictor_cls,
6973
**kwargs)
74+
75+
if py_version == 'py2':
76+
LOGGER.warning('tensorflow py2 container will be deprecated soon.')
77+
7078
self.py_version = py_version
7179
self.framework_version = framework_version
7280
self.model_server_workers = model_server_workers

0 commit comments

Comments
 (0)