Skip to content

Commit c844b33

Browse files
committed
fix tests
1 parent 4263904 commit c844b33

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

src/sagemaker/jumpstart/estimator.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -616,6 +616,7 @@ def _validate_model_id_and_get_type_hook():
616616
self.tolerate_vulnerable_model = estimator_init_kwargs.tolerate_vulnerable_model
617617
self.instance_count = estimator_init_kwargs.instance_count
618618
self.region = estimator_init_kwargs.region
619+
self.environment = estimator_init_kwargs.environment
619620
self.orig_predictor_cls = None
620621
self.role = estimator_init_kwargs.role
621622
self.sagemaker_session = estimator_init_kwargs.sagemaker_session

src/sagemaker/jumpstart/utils.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1671,7 +1671,7 @@ def get_model_access_config(accept_eula: Optional[bool], environment: Optional[d
16711671
Args:
16721672
accept_eula (Optional[bool]): Whether or not the EULA was accepted, optionally passed in to Estimator.fit().
16731673
"""
1674-
env_var_eula = environment.get("accept_eula")
1674+
env_var_eula = environment.get("accept_eula") if environment else None
16751675
if env_var_eula and accept_eula is not None:
16761676
raise ValueError(
16771677
"Cannot pass in both accept_eula and environment variables. "

0 commit comments

Comments
 (0)