Skip to content

Commit 5984ea4

Browse files
committed
fix: unit tests, clarify_check_step.py with s3_upload_mode value
1 parent d110015 commit 5984ea4

File tree

3 files changed

+5
-4
lines changed

3 files changed

+5
-4
lines changed

src/sagemaker/image_uri_config/clarify.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
"us-west-2": "306415355426",
3030
"us-gov-west-1": "598674086554"
3131
},
32-
"repository": "cr-clarify-processing"
32+
"repository": "sagemaker-clarify-processing"
3333
}
3434
}
3535
}

src/sagemaker/workflow/clarify_check_step.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@
2929
ModelConfig,
3030
ModelPredictedLabelConfig,
3131
SHAPConfig,
32+
ProcessingOutputHandler,
3233
_upload_analysis_config,
3334
SageMakerClarifyProcessor,
3435
_set,
@@ -391,7 +392,7 @@ def _generate_processing_job_parameters(
391392
source=SageMakerClarifyProcessor._CLARIFY_OUTPUT,
392393
destination=data_config.s3_output_path,
393394
output_name="analysis_result",
394-
s3_upload_mode="EndOfJob",
395+
s3_upload_mode=ProcessingOutputHandler.get_s3_upload_mode(analysis_config),
395396
)
396397
return dict(config_input=config_input, data_input=data_input, result_output=result_output)
397398

tests/unit/test_clarify.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1721,11 +1721,11 @@ def test_invalid_analysis_config(data_config, data_bias_config, model_config):
17211721

17221722
class TestProcessingOutputHandler:
17231723
def test_get_s3_upload_mode_image(self):
1724-
analysis_config = {"dataset_type": DatasetType.IMAGE}
1724+
analysis_config = {"dataset_type": DatasetType.IMAGE.value}
17251725
s3_upload_mode = ProcessingOutputHandler.get_s3_upload_mode(analysis_config)
17261726
assert s3_upload_mode == ProcessingOutputHandler.S3UploadMode.CONTINUOUS.value
17271727

17281728
def test_get_s3_upload_mode_text(self):
1729-
analysis_config = {"dataset_type": DatasetType.TEXTCSV}
1729+
analysis_config = {"dataset_type": DatasetType.TEXTCSV.value}
17301730
s3_upload_mode = ProcessingOutputHandler.get_s3_upload_mode(analysis_config)
17311731
assert s3_upload_mode == ProcessingOutputHandler.S3UploadMode.ENDOFJOB.value

0 commit comments

Comments
 (0)