address nits

Author: gwang111

src/sagemaker/jumpstart/model.py

@@ -817,14 +817,12 @@ def deploy(
                 f"{EndpointType.INFERENCE_COMPONENT_BASED} is not supported for Proprietary models."
             )
 
-        print(self.additional_model_data_sources)
         self.additional_model_data_sources = _add_model_access_configs_to_model_data_sources(
             self.additional_model_data_sources,
             deploy_kwargs.model_access_configs,
             deploy_kwargs.model_id,
             deploy_kwargs.region,
         )
-        print(self.additional_model_data_sources)
 
         try:
             predictor = super(JumpStartModel, self).deploy(**deploy_kwargs.to_kwargs_dict())

src/sagemaker/jumpstart/utils.py

@@ -1548,7 +1548,7 @@ def _add_model_access_configs_to_model_data_sources(
     Args:
         model_data_sources (DeploymentConfigMetadata): Model data sources that will be updated
         model_access_configs (DeploymentConfigMetadata): Config holding accept_eula field
-        model_id (DeploymentConfigMetadata): Jumpstart mode id.
+        model_id (DeploymentConfigMetadata): Jumpstart model id.
         region (str): Region where the user is operating in.
     Returns:
         List[Dict[str, Any]]: List of model data sources with accept EULA configs applied

tests/unit/sagemaker/jumpstart/test_utils.py

@@ -2253,7 +2253,7 @@ def test_multiple_public_additional_model_data_source_should_pass_through_both(s
             + self.MOCK_PUBLIC_DEPLOY_CONFIG_ADDITIONAL_MODEL_DATA_SOURCE_POST_CALL
         )
 
-    def test_public_additional_model_data_source_with_model_access_config_should_ignored_it(self):
+    def test_public_additional_model_data_source_with_model_access_config_should_ignore_it(self):
         # WHERE / WHEN
         additional_model_data_sources = utils._add_model_access_configs_to_model_data_sources(
             model_data_sources=self.MOCK_PUBLIC_DEPLOY_CONFIG_ADDITIONAL_MODEL_DATA_SOURCE_PRE_CALL,