Skip to content

Commit 7e307bf

Browse files
author
Malav Shastri
committed
feat: Add curatedHub telemetry support
1 parent 559ef2e commit 7e307bf

File tree

3 files changed

+9
-6
lines changed

3 files changed

+9
-6
lines changed

src/sagemaker/jumpstart/factory/estimator.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -432,7 +432,7 @@ def _add_sagemaker_session_to_kwargs(kwargs: JumpStartKwargs) -> JumpStartKwargs
432432
kwargs.sagemaker_session = (
433433
kwargs.sagemaker_session
434434
or get_default_jumpstart_session_with_user_agent_suffix(
435-
kwargs.model_id, kwargs.model_version
435+
kwargs.model_id, kwargs.model_version, kwargs.hub_arn
436436
)
437437
)
438438
return kwargs

src/sagemaker/jumpstart/factory/model.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -152,12 +152,14 @@ def _add_sagemaker_session_to_kwargs(
152152
kwargs: Union[JumpStartModelInitKwargs, JumpStartModelDeployKwargs]
153153
) -> JumpStartModelInitKwargs:
154154
"""Sets session in kwargs based on default or override, returns full kwargs."""
155+
155156
kwargs.sagemaker_session = (
156157
kwargs.sagemaker_session
157158
or get_default_jumpstart_session_with_user_agent_suffix(
158-
kwargs.model_id, kwargs.model_version
159+
kwargs.model_id, kwargs.model_version, kwargs.hub_arn
159160
)
160161
)
162+
161163
return kwargs
162164

163165

src/sagemaker/jumpstart/utils.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1012,24 +1012,25 @@ def get_jumpstart_configs(
10121012
)
10131013

10141014

1015-
def get_jumpstart_user_agent_extra_suffix(model_id: str, model_version: str) -> str:
1015+
def get_jumpstart_user_agent_extra_suffix(model_id: Optional[str], model_version: Optional[str], is_hub_content: Optional[bool]) -> str:
10161016
"""Returns the model-specific user agent string to be added to requests."""
10171017
sagemaker_python_sdk_headers = get_user_agent_extra_suffix()
10181018
jumpstart_specific_suffix = f"md/js_model_id#{model_id} md/js_model_ver#{model_version}"
1019+
hub_specific_suffix = f"md/js_is_hub_content#{is_hub_content}"
10191020
return (
10201021
sagemaker_python_sdk_headers
10211022
if os.getenv(constants.ENV_VARIABLE_DISABLE_JUMPSTART_TELEMETRY, None)
1022-
else f"{sagemaker_python_sdk_headers} {jumpstart_specific_suffix}"
1023+
else f"{sagemaker_python_sdk_headers} {jumpstart_specific_suffix} {hub_specific_suffix}"
10231024
)
10241025

10251026

10261027
def get_default_jumpstart_session_with_user_agent_suffix(
1027-
model_id: str, model_version: str
1028+
model_id: Optional[str] = None, model_version: Optional[str] = None, is_hub_content: Optional[bool] = False
10281029
) -> Session:
10291030
"""Returns default JumpStart SageMaker Session with model-specific user agent suffix."""
10301031
botocore_session = botocore.session.get_session()
10311032
botocore_config = botocore.config.Config(
1032-
user_agent_extra=get_jumpstart_user_agent_extra_suffix(model_id, model_version),
1033+
user_agent_extra=get_jumpstart_user_agent_extra_suffix(model_id, model_version, is_hub_content),
10331034
)
10341035
botocore_session.set_default_client_config(botocore_config)
10351036
# shallow copy to not affect default session constant

0 commit comments

Comments
 (0)