Skip to content

Commit 813858a

Browse files
nargokulpintaoz-aws
authored andcommitted
Enable the Recipe tests marked with @pytest.mark.skip(reason="Hyperpod recipe code unavailable" (#1642)
1 parent 854bafc commit 813858a

File tree

3 files changed

+0
-12
lines changed

3 files changed

+0
-12
lines changed

tests/unit/sagemaker/modules/train/sm_recipes/test_utils.py

-6
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,6 @@ def temporary_recipe():
4848
yield f.name
4949

5050

51-
# TODO: To be removed on 12/5/2024
52-
@pytest.mark.skip(reason="Hyperpod recipe code unavailable")
5351
def test_load_base_recipe_with_overrides(temporary_recipe, training_recipes_cfg):
5452
expected_epochs = 20
5553
expected_layers = 15
@@ -71,8 +69,6 @@ def test_load_base_recipe_with_overrides(temporary_recipe, training_recipes_cfg)
7169
)
7270

7371

74-
# TODO: To be removed on 12/5/2024
75-
@pytest.mark.skip(reason="Hyperpod recipe code unavailable")
7672
@pytest.mark.parametrize(
7773
"test_case",
7874
[
@@ -130,8 +126,6 @@ def test_load_base_recipe_types(
130126
assert mock_retrieve.call_args.args[0] == url
131127

132128

133-
# TODO: To be removed on 12/5/2024
134-
@pytest.mark.skip(reason="Hyperpod recipe code unavailable")
135129
@pytest.mark.parametrize(
136130
"test_case",
137131
[

tests/unit/sagemaker/modules/train/test_model_trainer.py

-2
Original file line numberDiff line numberDiff line change
@@ -850,8 +850,6 @@ def mock_upload_data(path, bucket, key_prefix):
850850
)
851851

852852

853-
# TODO: To be removed on 12/5/2024
854-
@pytest.mark.skip(reason="Hyperpod recipe code unavailable")
855853
def test_model_trainer_gpu_recipe_full_init(modules_session):
856854
training_recipe = "training/llama/p4_hf_llama3_70b_seq8k_gpu"
857855
recipe_overrides = {"run": {"results_dir": "/opt/ml/model"}}

tests/unit/test_pytorch.py

-4
Original file line numberDiff line numberDiff line change
@@ -844,7 +844,6 @@ def test_predictor_with_component_name(sagemaker_session, component_name):
844844
assert predictor._get_component_name() == component_name
845845

846846

847-
@pytest.mark.skip(reason="Hyperpod recipe code unavailable")
848847
def test_training_recipe_for_cpu(sagemaker_session):
849848
container_log_level = '"logging.INFO"'
850849

@@ -878,7 +877,6 @@ def test_training_recipe_for_cpu(sagemaker_session):
878877
)
879878

880879

881-
@pytest.mark.skip(reason="Hyperpod recipe code unavailable")
882880
@pytest.mark.parametrize(
883881
"recipe, model",
884882
[
@@ -935,7 +933,6 @@ def test_training_recipe_for_gpu(sagemaker_session, recipe, model):
935933
assert pytorch.distribution.items() == expected_distribution.items()
936934

937935

938-
@pytest.mark.skip(reason="Hyperpod recipe code unavailable")
939936
def test_training_recipe_with_override(sagemaker_session):
940937
container_log_level = '"logging.INFO"'
941938

@@ -973,7 +970,6 @@ def test_training_recipe_with_override(sagemaker_session):
973970
assert pytorch.image_uri == IMAGE_URI
974971

975972

976-
@pytest.mark.skip(reason="Hyperpod recipe code unavailable")
977973
def test_training_recipe_gpu_custom_source_dir(sagemaker_session):
978974
container_log_level = '"logging.INFO"'
979975

0 commit comments

Comments
 (0)