-
Notifications
You must be signed in to change notification settings - Fork 6.8k
[Doc] Change the description for pip packages #12584
Conversation
Thanks for the contribution @TaoLv |
docs/install/index.md
Outdated
|
||
*mxnet-cu92mkl* means the package is built with CUDA/cuDNN and MKL-DNN enabled and the CUDA version is 9.2. | ||
|
||
MKL pip package is experimental in some old versions of 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.
Wouldn't it be more accurate to say "All MKL pip packages are experimental prior to version x.y.z."
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.
I'm not sure. But I think MKL pip packages should once be GA in some old versions, before it's changed to MKL-DNN backend. @szha May I have your comments?
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.
the pip packages w/ direct mklml integration has always been experimental until being replaced by mkldnn. the USE_MKLML_EXPERIMENTAL flag was on in all of them.
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.
Thanks for the clarification. Will change the doc.
@sandeep-krishnamurthy Please change label to pr-awaiting-merge. This PR has been approved. |
Description
@aaronmarkham @pengzhao-intel
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.
Changes
Comments