Skip to content

Commit 7a52481

Browse files
committed
fix integ test and use list comprehension instead of map
1 parent 979348e commit 7a52481

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

src/sagemaker/predictor.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -303,7 +303,7 @@ def _get_model_names(self):
303303
EndpointConfigName=self._endpoint_config_name
304304
)
305305
production_variants = endpoint_config["ProductionVariants"]
306-
return map(lambda d: d["ModelName"], production_variants)
306+
return [d["ModelName"] for d in production_variants]
307307

308308

309309
class _CsvSerializer(object):

tests/integ/test_mxnet_train.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -81,9 +81,8 @@ def _deploy_estimator_and_assert_instance_type(estimator, instance_type):
8181
predictor = estimator.deploy(1, instance_type)
8282

8383
model_name = predictor._model_names[0]
84-
endpoint_name = predictor.endpoint
8584
config_name = sagemaker_session.sagemaker_client.describe_endpoint(
86-
EndpointName=endpoint_name
85+
EndpointName=predictor.endpoint_name
8786
)["EndpointConfigName"]
8887
config = sagemaker_session.sagemaker_client.describe_endpoint_config(
8988
EndpointConfigName=config_name
@@ -94,7 +93,7 @@ def _deploy_estimator_and_assert_instance_type(estimator, instance_type):
9493

9594
assert config["ProductionVariants"][0]["InstanceType"] == instance_type
9695

97-
return (model_name, endpoint_name, config_name)
96+
return (model_name, predictor.endpoint_name, config_name)
9897

9998
estimator = MXNet.attach(mxnet_training_job, sagemaker_session)
10099
estimator.base_job_name = "test-mxnet-deploy-twice"

0 commit comments

Comments
 (0)