Skip to content

Commit e75da02

Browse files
Small bug fix for serve integration test (#5363)
* Fixing serve unit tests * Bug fix for inference --------- Co-authored-by: rsareddy0329 <rsareddy0329@gmail.com>
1 parent 8a803f7 commit e75da02

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

sagemaker-serve/src/sagemaker/serve/model_builder.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -943,23 +943,23 @@ def _is_model_customization(self) -> bool:
943943
# ModelTrainer with model customization
944944
if isinstance(self.model, ModelTrainer) and hasattr(self.model, '_latest_training_job'):
945945
# Check model_package_config first (new location)
946-
if (hasattr(self.model._latest_training_job, 'model_package_config') and self.model._latest_training_job.model_package_config != Unassigned
947-
and getattr(self.model._latest_training_job.model_package_config, 'source_model_package_arn', Unassigned) != Unassigned):
946+
if (hasattr(self.model._latest_training_job, 'model_package_config') and self.model._latest_training_job.model_package_config != Unassigned()
947+
and getattr(self.model._latest_training_job.model_package_config, 'source_model_package_arn', Unassigned()) != Unassigned()):
948948
return True
949949
# Fallback to serverless_job_config (legacy location)
950-
if (hasattr(self.model._latest_training_job, 'serverless_job_config') and self.model._latest_training_job.serverless_job_config != Unassigned
951-
and hasattr(self.model._latest_training_job, 'output_model_package_arn') and self.model._latest_training_job.output_model_package_arn!= Unassigned):
950+
if (hasattr(self.model._latest_training_job, 'serverless_job_config') and self.model._latest_training_job.serverless_job_config != Unassigned()
951+
and hasattr(self.model._latest_training_job, 'output_model_package_arn') and self.model._latest_training_job.output_model_package_arn!= Unassigned()):
952952
return True
953953

954954
# BaseTrainer with model customization
955955
if isinstance(self.model, BaseTrainer) and hasattr(self.model, '_latest_training_job'):
956956
# Check model_package_config first (new location)
957-
if (hasattr(self.model._latest_training_job, 'model_package_config') and self.model._latest_training_job.model_package_config != Unassigned
958-
and getattr(self.model._latest_training_job.model_package_config, 'source_model_package_arn', Unassigned) != Unassigned):
957+
if (hasattr(self.model._latest_training_job, 'model_package_config') and self.model._latest_training_job.model_package_config != Unassigned()
958+
and getattr(self.model._latest_training_job.model_package_config, 'source_model_package_arn', Unassigned()) != Unassigned()):
959959
return True
960960
# Fallback to serverless_job_config (legacy location)
961-
if (hasattr(self.model._latest_training_job, 'serverless_job_config') and self.model._latest_training_job.serverless_job_config != Unassigned
962-
and hasattr(self.model._latest_training_job, 'output_model_package_arn') and self.model._latest_training_job.output_model_package_arn!= Unassigned):
961+
if (hasattr(self.model._latest_training_job, 'serverless_job_config') and self.model._latest_training_job.serverless_job_config != Unassigned()
962+
and hasattr(self.model._latest_training_job, 'output_model_package_arn') and self.model._latest_training_job.output_model_package_arn!= Unassigned()):
963963
return True
964964

965965
return False

0 commit comments

Comments
 (0)