-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Description
What did you find confusing? Please describe.
sagemaker.session.s3_input() or sagemaker.inputs.s3_input()?
While I found no code using the latter, it is the only document entry. s3_input is not listed in the Session page where I would expect it.
Describe how documentation can be improved
I'm not sure whether this is a convention of Python docs but it is confusing. If it is to be used as sagemaker.session.s3_input() then it should be documented as such.
I also found the use of sess = sagemaker.Session() but in the docs it is sagemaker.session.Session().
To save another issue, I also found from sagemaker.amazon.amazon_estimator import get_image_uri in official examples but that function was not documented.
These are quite confusing for SageMaker new comers. Hope these can be fixed somehow.
Thanks!