Skip to content

Commit 2938fb3

Browse files
author
Chunxi Wang
committed
Comment pytest.skip for eia mnist test
1 parent 61fc311 commit 2938fb3

File tree

3 files changed

+5
-4
lines changed

3 files changed

+5
-4
lines changed

‎buildspec.yml‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -89,9 +89,9 @@ phases:
8989
- execute-command-if-has-matching-changes "$test_cmd" "test/" "src/*.py" "setup.py" "setup.cfg" "buildspec-toolkit.yml" "artifacts/*"
9090

9191
finally:
92-
# # shut down remote GPU instance
93-
# - cleanup-gpu-instances
94-
# - cleanup-key-pairs
92+
# shut down remote GPU instance
93+
- cleanup-gpu-instances
94+
- cleanup-key-pairs
9595

9696
# # remove ECR image
9797
# - aws ecr batch-delete-image --repository-name $ECR_REPO --region $AWS_DEFAULT_REGION --image-ids imageTag=$GENERIC_TAG

‎test/integration/sagemaker/test_mnist.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ def test_mnist_gpu(sagemaker_session, image_uri, instance_type):
3434
_test_mnist_distributed(sagemaker_session, image_uri, instance_type, model_gpu_tar, mnist_gpu_script)
3535

3636

37-
@pytest.mark.skip(reason="Latest EIA version is too old - 1.3.1. Remove this after a new DLC release")
37+
# @pytest.mark.skip(reason="Latest EIA version is too old - 1.3.1. Remove this after a new DLC release")
3838
@pytest.mark.eia_test
3939
def test_mnist_eia(sagemaker_session, image_uri, instance_type, accelerator_type):
4040
instance_type = instance_type or 'ml.c4.xlarge'

‎test/resources/mnist/model_eia/mnist.py‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ def predict_fn(input_data, model):
3131
input_data = input_data.to(device)
3232
model = model.eval()
3333
with torch.no_grad():
34+
print("current torch version is: ", torch.__version__)
3435
if torch.__version__ in VERSIONS_USE_NEW_API:
3536
import torcheia
3637
# we need to set the profiling executor for EIA

0 commit comments

Comments
 (0)