Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Doc Fix: added note about cuda9.2 requirement to Java example #14140

Merged
merged 1 commit into from
Apr 4, 2019
Merged

Doc Fix: added note about cuda9.2 requirement to Java example #14140

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions scala-package/mxnet-demo/java-demo/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@ This command will pick the default values specified in the [pom](https://github.

Note: If you are planning to use GPU, please add `-Dmxnet.profile=linux-x86_64-gpu`

Note: The Maven package is built with CUDA 9.2.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Sam for bringing this up.
I think this is the correct location for this disclaimer : https://github.com/apache/incubator-mxnet/tree/master/scala-package#pre-built-maven-packages

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmmm... I never looked at this page before. Maybe we should add a link and generalize the note more and say something like "Make sure you're environment is configured correctly according to..." and link to that page then?

Users following this demo arent necessarily going to read all the docs before running. We need to keep things self-containted and succinct.


### Use customized version set
You can use the following instruction as an alternative to achieve the same result:
You may use `mvn package` to build the package,
Expand Down