Skip to content

Commit 4cd4a58

Browse files
author
Syed Jafri
committed
fix: unskip dpo trainer integ tests
1 parent 59ea1ab commit 4cd4a58

1 file changed

Lines changed: 3 additions & 5 deletions

File tree

sagemaker-train/tests/integ/train/test_dpo_trainer_integration.py

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,15 +22,14 @@
2222
import pytest
2323

2424

25-
@pytest.mark.skip(reason="Skipping GPU resource intensive test")
2625
def test_dpo_trainer_lora_complete_workflow(sagemaker_session):
2726
"""Test complete DPO training workflow with LORA."""
2827
# Create DPOTrainer instance with comprehensive configuration
2928
trainer = DPOTrainer(
3029
model="meta-textgeneration-llama-3-2-1b-instruct",
3130
training_type=TrainingType.LORA,
3231
model_package_group="sdk-test-finetuned-models",
33-
training_dataset="arn:aws:sagemaker:us-west-2:729646638167:hub-content/sdktest/DataSet/dpo-oss-test-data/0.0.1",
32+
training_dataset="s3://mc-flows-sdk-testing/input_data/dpo/preference_dataset_train_256.jsonl",
3433
s3_output_path="s3://mc-flows-sdk-testing/output/",
3534
accept_eula=True
3635
)
@@ -61,16 +60,15 @@ def test_dpo_trainer_lora_complete_workflow(sagemaker_session):
6160
assert training_job.output_model_package_arn is not None
6261

6362

64-
@pytest.mark.skip(reason="Skipping GPU resource intensive test")
6563
def test_dpo_trainer_with_validation_dataset(sagemaker_session):
6664
"""Test DPO trainer with both training and validation datasets."""
6765

6866
dpo_trainer = DPOTrainer(
6967
model="meta-textgeneration-llama-3-2-1b-instruct",
7068
training_type=TrainingType.LORA,
7169
model_package_group="sdk-test-finetuned-models",
72-
training_dataset="arn:aws:sagemaker:us-west-2:729646638167:hub-content/sdktest/DataSet/dpo-oss-test-data/0.0.1",
73-
validation_dataset="arn:aws:sagemaker:us-west-2:729646638167:hub-content/sdktest/DataSet/dpo-oss-test-data/0.0.1",
70+
training_dataset="s3://mc-flows-sdk-testing/input_data/dpo/preference_dataset_train_256.jsonl",
71+
validation_dataset="s3://mc-flows-sdk-testing/input_data/dpo/preference_dataset_train_256.jsonl",
7472
s3_output_path="s3://mc-flows-sdk-testing/output/",
7573
accept_eula=True
7674
)

0 commit comments

Comments
 (0)