Skip to content

Commit 9409345

Browse files
authored
Fix flake8 errors (#621)
1 parent de9199e commit 9409345

File tree

2 files changed

+14
-14
lines changed

2 files changed

+14
-14
lines changed

tests/integ/test_mxnet_train.py

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -102,16 +102,16 @@ def test_deploy_model_with_update_non_existing_endpoint(mxnet_training_job, sage
102102
'please use an existing endpoint name'.format(endpoint_name)
103103

104104
with timeout_and_delete_endpoint_by_name(endpoint_name, sagemaker_session):
105-
desc = sagemaker_session.sagemaker_client.describe_training_job(TrainingJobName=mxnet_training_job)
106-
model_data = desc['ModelArtifacts']['S3ModelArtifacts']
107-
script_path = os.path.join(DATA_DIR, 'mxnet_mnist', 'mnist.py')
108-
model = MXNetModel(model_data, 'SageMakerRole', entry_point=script_path,
109-
py_version=PYTHON_VERSION, sagemaker_session=sagemaker_session)
110-
model.deploy(1, 'ml.t2.medium', endpoint_name=endpoint_name)
111-
sagemaker_session.describe_endpoint(EndpointName=endpoint_name)
112-
113-
with pytest.raises(ValueError, message=expected_error_message):
114-
model.deploy(1, 'ml.m4.xlarge', update_endpoint=True, endpoint_name='non-existing-endpoint')
105+
desc = sagemaker_session.sagemaker_client.describe_training_job(TrainingJobName=mxnet_training_job)
106+
model_data = desc['ModelArtifacts']['S3ModelArtifacts']
107+
script_path = os.path.join(DATA_DIR, 'mxnet_mnist', 'mnist.py')
108+
model = MXNetModel(model_data, 'SageMakerRole', entry_point=script_path,
109+
py_version=PYTHON_VERSION, sagemaker_session=sagemaker_session)
110+
model.deploy(1, 'ml.t2.medium', endpoint_name=endpoint_name)
111+
sagemaker_session.describe_endpoint(EndpointName=endpoint_name)
112+
113+
with pytest.raises(ValueError, message=expected_error_message):
114+
model.deploy(1, 'ml.m4.xlarge', update_endpoint=True, endpoint_name='non-existing-endpoint')
115115

116116

117117
@pytest.mark.continuous_testing

tests/unit/test_tuner.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -428,7 +428,7 @@ def test_stop_tuning_job(sagemaker_session, tuner):
428428

429429
def test_stop_tuning_job_no_tuning_job(tuner):
430430
with pytest.raises(ValueError) as e:
431-
tuner.stop_tuning_job()
431+
tuner.stop_tuning_job()
432432
assert 'No tuning job available' in str(e)
433433

434434

@@ -612,7 +612,7 @@ def test_transfer_learning_tuner(sagemaker_session):
612612
def test_continuous_parameter():
613613
cont_param = ContinuousParameter(0.1, 1e-2)
614614
assert isinstance(cont_param, ParameterRange)
615-
assert cont_param.__name__ is 'Continuous'
615+
assert cont_param.__name__ == 'Continuous'
616616

617617

618618
def test_continuous_parameter_ranges():
@@ -627,7 +627,7 @@ def test_continuous_parameter_ranges():
627627
def test_integer_parameter():
628628
int_param = IntegerParameter(1, 2)
629629
assert isinstance(int_param, ParameterRange)
630-
assert int_param.__name__ is 'Integer'
630+
assert int_param.__name__ == 'Integer'
631631

632632

633633
def test_integer_parameter_ranges():
@@ -642,7 +642,7 @@ def test_integer_parameter_ranges():
642642
def test_categorical_parameter_list():
643643
cat_param = CategoricalParameter(['a', 'z'])
644644
assert isinstance(cat_param, ParameterRange)
645-
assert cat_param.__name__ is 'Categorical'
645+
assert cat_param.__name__ == 'Categorical'
646646

647647

648648
def test_categorical_parameter_list_ranges():

0 commit comments

Comments
 (0)