Skip to content

Conversation

@laurenyu
Copy link
Contributor

This change provides clearer instructions for the bring-your-own model case. Relevant forum post - https://forums.aws.amazon.com/thread.jspa?threadID=269142&tstart=0

@laurenyu laurenyu requested a review from lukmis January 17, 2018 18:36
@laurenyu laurenyu merged commit a23028a into aws:master Jan 18, 2018
@laurenyu laurenyu deleted the byo-model-docs branch January 18, 2018 17:13
ragavvenkatesan pushed a commit that referenced this pull request Jan 30, 2018
* add sagemaker cli (#32)

* add sagemaker cli

* remove unnecessary close

* address PR comments

* tidy up imports

* fix imports, flake8 errors

* improve help message for bucket-name

* remove default role name

* fix log-level and py3 tests, add copyright

* update cli example scripts

* Add documentation about BYO Models (#47)

* Add test for BYO estimator using Factorization Machines algorithm as an example. (#50)

* Support multi-part uploads (#45)

* Update TensorFlow examples following API change (#44)

* Add data_type to hyperparameters (#54)

When we describe a training job the data type of the hyper parameters is
lost because we use a dict[str, str]. This adds a new field to
Hyperparameter so that we can convert the datatypes at runtime.

instead of validating with isinstance(), we cast the hp value to the type it
is meant to be. This enforces a "strongly typed" value. When we
deserialize from the API string responses it becomes easier to deal with
too.
@ragavvenkatesan ragavvenkatesan mentioned this pull request Jan 30, 2018
laurenyu added a commit to laurenyu/sagemaker-python-sdk that referenced this pull request May 31, 2018
We originally weren't honoring job names passed through fit().
This change fixes that.
apacker pushed a commit to apacker/sagemaker-python-sdk that referenced this pull request Nov 15, 2018
Additional inference discussion and analysis in LDA-Science
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants