-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Open
Description
Hello,
You give the ability to use LocalSession which is potentially really helpful. I am still struggling to make it work by using local-stack and pytests.
I find everywhere small obstacles from the code, Eg. here in utils you are recreating resources (eg s3) instead of using the s3 resource of the sagemaker_session
which a user can initialize it with local s3_url_endpoint
sagemaker-python-sdk/src/sagemaker/utils.py
Line 532 in 72c12bb
s3 = boto_session.resource("s3", region_name=sagemaker_session.boto_region_name) |
So instead of doing this
boto_session = sagemaker_session.boto_session
s3 = boto_session.resource("s3", region_name=boto_session.region_name)
you could simple do this if s3_resource
is initialized
s3 = sagemaker_session.s3_resource