address more comments and fix doc

Author: Captainia

doc/doc_utils/jumpstart_doc_utils.py

@@ -244,7 +244,7 @@ def create_proprietary_model_table():
         proprietary_content_entries.append(
             "     - `{} <{}>`__ |external-link|\n".format("Source", model_spec.get("url"))
         )
-    return marketpkace_content_intro + proprietary_content_entries + ["\n"]
+    return [marketpkace_content_intro] + proprietary_content_entries + ["\n"]
 
 
 def create_jumpstart_model_table():

src/sagemaker/jumpstart/filters.py

@@ -52,6 +52,13 @@ class SpecialSupportedFilterKeys(str, Enum):
     MODEL_TYPE = "model_type"
 
 
+class ProprietaryModelFilterIdentifiers(str, Enum):
+    """Enum class for proprietary model filter keys."""
+
+    PROPRIETARY = "proprietary"
+    MARKETPLACE = "marketplace"
+
+
 FILTER_OPERATOR_STRING_MAPPINGS = {
     FilterOperators.EQUALS: ["===", "==", "equals", "is"],
     FilterOperators.NOT_EQUALS: ["!==", "!=", "not equals", "is not"],

src/sagemaker/jumpstart/model.py

@@ -631,7 +631,7 @@ def deploy(
             and endpoint_type == EndpointType.INFERENCE_COMPONENT_BASED
         ):
             raise ValueError(
-                "EndpointType.INFERENCE_COMPONENT_BASED is not supported for Proprietary models."
+                f"{EndpointType.INFERENCE_COMPONENT_BASED} is not supported for Proprietary models."
             )
 
         try:

src/sagemaker/jumpstart/notebook_utils.py

@@ -29,6 +29,7 @@
 from sagemaker.jumpstart.enums import JumpStartScriptScope, JumpStartModelType
 from sagemaker.jumpstart.filters import (
     SPECIAL_SUPPORTED_FILTER_KEYS,
+    ProprietaryModelFilterIdentifiers,
     BooleanValues,
     Identity,
     SpecialSupportedFilterKeys,
@@ -227,7 +228,6 @@ def list_jumpstart_scripts(  # pylint: disable=redefined-builtin
         sagemaker_session (sagemaker.session.Session): Optional. The SageMaker Session to
             use to perform the model search. (Default: DEFAULT_JUMPSTART_SAGEMAKER_SESSION).
     """
-
     if (isinstance(filter, Constant) and filter.resolved_value == BooleanValues.TRUE) or (
         isinstance(filter, str) and filter.lower() == BooleanValues.TRUE.lower()
     ):
@@ -360,8 +360,7 @@ def _generate_jumpstart_model_versions(  # pylint: disable=redefined-builtin
         key = model_filter.key
         all_keys.add(key)
         if model_filter.key == SpecialSupportedFilterKeys.MODEL_TYPE and model_filter.value in {
-            "marketplace",
-            "proprietary",
+            identifier.value for identifier in ProprietaryModelFilterIdentifiers
         }:
             model_filter.set_value(JumpStartModelType.PROPRIETARY.value)
         model_filters.add(model_filter)