Support tensorflow 1.9 and bump version to 1.9.2 #365
Conversation
README.rst
Outdated
| @@ -242,7 +242,7 @@ MXNet SageMaker Estimators | |||
|
|
|||
| By using MXNet SageMaker ``Estimators``, you can train and host MXNet models on Amazon SageMaker. | |||
|
|
|||
| Supported versions of MXNet: ``1.2.1``, ``1.1.0``, ``1.0.0``, ``0.12.1``. | |||
| Supported versions of MXNet: ``1.2.1``, ``1.1.0``, ``1.0.0``, ``0.12.1``. We recommend to use the latest version and our integration test will only test the latest version. | |||
There was a problem hiding this comment.
I would add the sentence in a new line after the list of versions and change it to something like "We recommend you use the latest version as that's where we spend most of our development efforts". Maybe @djarpin or @eslesar-aws has a better wording suggestion.
There was a problem hiding this comment.
"We recommend that you use the latest supported version, because that's where we focus most of our development efforts."
Codecov Report
@@ Coverage Diff @@
## master #365 +/- ##
======================================
Coverage 92.9% 92.9%
======================================
Files 51 51
Lines 3567 3567
======================================
Hits 3314 3314
Misses 253 253
Continue to review full report at Codecov.
|
Extending SageMaker PyTorch containers
Issue #, if available:
Description of changes:
Merge Checklist
Put an
xin the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.