Skip to content

Commit a7c0aaf

Browse files
authored
Fix SageMaker Session handling in Horovod test (#165)
1 parent c4d6b85 commit a7c0aaf

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

test/integration/sagemaker/test_horovod.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@
2121

2222

2323
def test_distributed_training_horovod(sagemaker_session,
24-
sagemaker_local_session,
2524
instance_type,
2625
ecr_image,
2726
tmpdir,
@@ -39,12 +38,13 @@ def test_distributed_training_horovod(sagemaker_session,
3938
script_mode=True,
4039
hyperparameters={'sagemaker_mpi_enabled': True,
4140
'sagemaker_mpi_custom_mpi_options': mpi_options,
42-
'sagemaker_mpi_num_of_processes_per_host': 1})
41+
'sagemaker_mpi_num_of_processes_per_host': 1},
42+
sagemaker_session=sagemaker_session)
4343

4444
estimator.fit()
4545

4646
model_data_source = sagemaker.local.data.get_data_source_instance(
47-
estimator.model_data, sagemaker.session.Session())
47+
estimator.model_data, sagemaker_session)
4848

4949
for filename in model_data_source.get_file_list():
5050
assert os.path.basename(filename) == 'model.tar.gz'

0 commit comments

Comments
 (0)