fix: Reverting name change for code coverage

Author: chrstfu

src/sagemaker/jumpstart/hub/parser_utils.py

@@ -18,7 +18,7 @@
 from typing import Any, Dict, List, Optional
 
 
-def pascal_to_snake(camel_case_string: str) -> str:
+def camel_to_snake(camel_case_string: str) -> str:
     """Converts PascalCase to snake_case_string using a regex.
 
     This regex cannot handle whitespace ("PascalString TwoWords")

src/sagemaker/jumpstart/hub/parsers.py

@@ -27,7 +27,7 @@
     HubModelDocument,
 )
 from sagemaker.jumpstart.hub.parser_utils import (
-    pascal_to_snake,
+    camel_to_snake,
     snake_to_upper_camel,
     walk_and_apply_json,
 )
@@ -86,7 +86,7 @@ def get_model_spec_arg_keys(
         arg_keys = []
 
     if naming_convention == NamingConventionType.SNAKE_CASE:
-        arg_keys = [pascal_to_snake(key) for key in arg_keys]
+        arg_keys = [camel_to_snake(key) for key in arg_keys]
     elif naming_convention == NamingConventionType.UPPER_CAMEL_CASE:
         return arg_keys
     else:
@@ -207,7 +207,7 @@ def make_model_specs_from_describe_hub_content_response(
     default_payloads: Dict[str, Any] = {}
     if hub_model_document.default_payloads is not None:
         for alias, payload in hub_model_document.default_payloads.items():
-            default_payloads[alias] = walk_and_apply_json(payload.to_json(), pascal_to_snake)
+            default_payloads[alias] = walk_and_apply_json(payload.to_json(), camel_to_snake)
         specs["default_payloads"] = default_payloads
     specs["gated_bucket"] = hub_model_document.gated_bucket
     specs["inference_volume_size"] = hub_model_document.inference_volume_size

src/sagemaker/jumpstart/types.py

@@ -39,7 +39,7 @@
 from sagemaker.compute_resource_requirements.resource_requirements import ResourceRequirements
 from sagemaker.enums import EndpointType
 from sagemaker.jumpstart.hub.parser_utils import (
-    pascal_to_snake,
+    camel_to_snake,
     walk_and_apply_json,
 )
 
@@ -239,7 +239,7 @@ def from_json(self, json_obj: Dict[str, Any]) -> None:
             return
 
         if self._is_hub_content:
-            json_obj = walk_and_apply_json(json_obj, pascal_to_snake)
+            json_obj = walk_and_apply_json(json_obj, camel_to_snake)
 
         self.framework = json_obj.get("framework")
         self.framework_version = json_obj.get("framework_version")
@@ -293,7 +293,7 @@ def from_json(self, json_obj: Dict[str, Any]) -> None:
         """
 
         if self._is_hub_content:
-            json_obj = walk_and_apply_json(json_obj, pascal_to_snake)
+            json_obj = walk_and_apply_json(json_obj, camel_to_snake)
         self.name = json_obj["name"]
         self.type = json_obj["type"]
         self.default = json_obj["default"]
@@ -361,7 +361,7 @@ def from_json(self, json_obj: Dict[str, Any]) -> None:
         Args:
             json_obj (Dict[str, Any]): Dictionary representation of environment variable.
         """
-        json_obj = walk_and_apply_json(json_obj, pascal_to_snake)
+        json_obj = walk_and_apply_json(json_obj, camel_to_snake)
         self.name = json_obj["name"]
         self.type = json_obj["type"]
         self.default = json_obj["default"]
@@ -411,7 +411,7 @@ def from_json(self, json_obj: Optional[Dict[str, Any]]) -> None:
             return
 
         if self._is_hub_content:
-            json_obj = walk_and_apply_json(json_obj, pascal_to_snake)
+            json_obj = walk_and_apply_json(json_obj, camel_to_snake)
         self.default_content_type = json_obj["default_content_type"]
         self.supported_content_types = json_obj["supported_content_types"]
         self.default_accept_type = json_obj["default_accept_type"]
@@ -465,7 +465,7 @@ def from_json(self, json_obj: Optional[Dict[str, Any]]) -> None:
             return
 
         if self._is_hub_content:
-            json_obj = walk_and_apply_json(json_obj, pascal_to_snake)
+            json_obj = walk_and_apply_json(json_obj, camel_to_snake)
         self.raw_payload = json_obj
         self.content_type = json_obj["content_type"]
         self.body = json_obj.get("body")
@@ -538,7 +538,7 @@ def from_describe_hub_content_response(self, response: Optional[Dict[str, Any]])
         if response is None:
             return
 
-        response = walk_and_apply_json(response, pascal_to_snake)
+        response = walk_and_apply_json(response, camel_to_snake)
         self.aliases: Optional[dict] = response.get("aliases")
         self.regional_aliases = None
         self.variants: Optional[dict] = response.get("variants")
@@ -1174,7 +1174,7 @@ def __init__(self, spec: Optional[Dict[str, Any]], is_hub_content=False):
             spec (Dict[str, Any]): Dictionary representation of training config ranking.
         """
         if is_hub_content:
-            spec = walk_and_apply_json(spec, pascal_to_snake)
+            spec = walk_and_apply_json(spec, camel_to_snake)
         self.from_json(spec)
 
     def from_json(self, json_obj: Dict[str, Any]) -> None:
@@ -1280,7 +1280,7 @@ def from_json(self, json_obj: Dict[str, Any]) -> None:
             json_obj (Dict[str, Any]): Dictionary representation of spec.
         """
         if self._is_hub_content:
-            json_obj = walk_and_apply_json(json_obj, pascal_to_snake)
+            json_obj = walk_and_apply_json(json_obj, camel_to_snake)
         self.model_id: str = json_obj.get("model_id")
         self.url: str = json_obj.get("url")
         self.version: str = json_obj.get("version")
@@ -1509,7 +1509,7 @@ def __init__(
             ValueError: If the component field is invalid.
         """
         if is_hub_content:
-            component = walk_and_apply_json(component, pascal_to_snake)
+            component = walk_and_apply_json(component, camel_to_snake)
         self.component_name = component_name
         super().__init__(component, is_hub_content)
         self.from_json(component)
@@ -1562,8 +1562,8 @@ def __init__(
                 The list of components that are used to construct the resolved config.
         """
         if is_hub_content:
-            config = walk_and_apply_json(config, pascal_to_snake)
-            base_fields = walk_and_apply_json(base_fields, pascal_to_snake)
+            config = walk_and_apply_json(config, camel_to_snake)
+            base_fields = walk_and_apply_json(base_fields, camel_to_snake)
         self.base_fields = base_fields
         self.config_components: Dict[str, JumpStartConfigComponent] = config_components
         self.benchmark_metrics: Dict[str, List[JumpStartBenchmarkStat]] = (
@@ -1729,7 +1729,7 @@ def from_json(self, json_obj: Dict[str, Any]) -> None:
         """
         super().from_json(json_obj)
         if self._is_hub_content:
-            json_obj = walk_and_apply_json(json_obj, pascal_to_snake)
+            json_obj = walk_and_apply_json(json_obj, camel_to_snake)
         self.inference_config_components: Optional[Dict[str, JumpStartConfigComponent]] = (
             {
                 component_name: JumpStartConfigComponent(component_name, component)

src/sagemaker/jumpstart/utils.py

@@ -33,7 +33,7 @@
 
 from sagemaker.jumpstart import constants, enums
 from sagemaker.jumpstart import accessors
-from sagemaker.jumpstart.hub.parser_utils import pascal_to_snake, snake_to_upper_camel
+from sagemaker.jumpstart.hub.parser_utils import camel_to_snake, snake_to_upper_camel
 from sagemaker.s3 import parse_s3_url
 from sagemaker.jumpstart.exceptions import (
     DeprecatedJumpStartModelError,
@@ -1149,7 +1149,7 @@ def get_jumpstart_configs(
         return (
             {
                 config_name: metadata_configs.configs[
-                    pascal_to_snake(snake_to_upper_camel(config_name))
+                    camel_to_snake(snake_to_upper_camel(config_name))
                 ]
                 for config_name in config_names
             }

tests/unit/sagemaker/jumpstart/hub/test_parser_utils.py

@@ -12,7 +12,7 @@
 # language governing permissions and limitations under the License.
 from __future__ import absolute_import
 import pytest
-from sagemaker.jumpstart.hub.parser_utils import pascal_to_snake
+from sagemaker.jumpstart.hub.parser_utils import camel_to_snake
 
 REGION = "us-east-1"
 ACCOUNT_ID = "123456789123"
@@ -31,4 +31,4 @@
     ],
 )
 def test_parse_(input_string, expected):
-    assert expected == pascal_to_snake(input_string)
+    assert expected == camel_to_snake(input_string)

tests/unit/sagemaker/jumpstart/hub/test_utils.py

@@ -268,7 +268,7 @@ def test_walk_and_apply_json():
     }
 
     result = parser_utils.walk_and_apply_json(
-        test_json, parser_utils.pascal_to_snake, ["ignore_my_children"]
+        test_json, parser_utils.camel_to_snake, ["ignore_my_children"]
     )
     assert result == {
         "camel_case_key": "value",
@@ -294,7 +294,7 @@ def test_walk_and_apply_json_no_stop():
         "CamelCaseObjectListKey": {"instance.ml.type.xlarge": [{"ShouldChangeMe": "string"}]},
     }
 
-    result = parser_utils.walk_and_apply_json(test_json, parser_utils.pascal_to_snake)
+    result = parser_utils.walk_and_apply_json(test_json, parser_utils.camel_to_snake)
     assert result == {
         "camel_case_key": "value",
         "camel_case_object_key": {