-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Conversation
Note - |
docs/_static/mxnet-theme/index.html
Outdated
<h3>Introducing - Gluon</h3> | ||
<p>We’re happy to introduce a new elegant, easy to use, imperative interface for MXNet. | ||
<h3>MXNet Model Server</h3> | ||
<p>Apache MXNet Model Server (MMS) is a flexible and easy to use tool for serving deep learning models exported from MXNet or the Open Neural Network Exchange (ONNX). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not an apache package. @hen would that be a problem?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please change line 30 to:
Model Server for Apache MXNet
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And get rid of the (MMS) since it doesn't match up here! Thanks!
docs/_static/mxnet-theme/index.html
Outdated
</p> | ||
<a href="http://gluon.mxnet.io">Learn More</a> | ||
<a href="https://github.com/awslabs/mxnet-model-server/blob/master/README.md">Learn More</a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Link should take to: https://github.com/awslabs/mxnet-model-server
docs/_static/mxnet-theme/index.html
Outdated
<h3>Introducing - Gluon</h3> | ||
<p>We’re happy to introduce a new elegant, easy to use, imperative interface for MXNet. | ||
<h3>MXNet Model Server</h3> | ||
<p>Apache MXNet Model Server (MMS) is a flexible and easy to use tool for serving deep learning models exported from MXNet or the Open Neural Network Exchange (ONNX). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we cannot call out ONNX support yet - since 0.2 goes out on 2/5
Where is the tutorial tab? |
docs/_static/mxnet-theme/index.html
Outdated
<h3>Apache MXNet 1.0 Released</h3> | ||
<p>We're excited to announce the release of MXNet 1.0! Check out the release notes for latest updates.</p> | ||
<a href="https://github.com/apache/incubator-mxnet/releases/tag/1.0.0">Learn More</a> | ||
<h3>Apache MXNet 1.0.1 Released</h3> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update the website version to 1.1.0
docs/_static/mxnet-theme/index.html
Outdated
<a href="https://github.com/apache/incubator-mxnet/releases/tag/1.0.0">Learn More</a> | ||
<h3>Apache MXNet 1.0.1 Released</h3> | ||
<p>We're excited to announce the release of MXNet 1.0.1! Check out the release notes for latest updates.</p> | ||
<a href="https://github.com/apache/incubator-mxnet/releases/tag/1.0.1">Learn More</a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mbaijal are we supposed to have 1.1.0.rc0
in the link or 1.1.0
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ideally, It should be 1.1.0
everywhere. Once the RC is created, no src can be modified so it must be the final release tag.
However, This would lead to a broken link on the website and I think this should only be merged onto the release branch for now.
…ator-mxnet into branch_01232018
Note that the version numbering has changed from 1.0.1 to 1.1.0 for the release |
MXNet model server is actually another project. I wonder whether its appropriate to advertise it here. |
I suggest we advertise MMS as a related project - we need to build the brand around MXNet. In a few weeks we also want to add Tensorboard-MXNet etc. |
Rephrased title |
Description
This PR has the following changes:
Checklist
Essentials