[GitHub] Lyken17 commented on issue #10236: Fix the order of `path.join` and `path.expanduser`

2018-03-24 Thread GitBox
Lyken17 commented on issue #10236: Fix the order of `path.join` and `path.expanduser` URL: https://github.com/apache/incubator-mxnet/pull/10236#issuecomment-375944405 Thanks : ) This is an automated message from the Apache

[GitHub] lightingghost commented on issue #10239: Error in Building and Installing packages on Windows using Visual Studio 2017

2018-03-24 Thread GitBox
lightingghost commented on issue #10239: Error in Building and Installing packages on Windows using Visual Studio 2017 URL: https://github.com/apache/incubator-mxnet/issues/10239#issuecomment-375943999 `iomp5.lib` should be the library file needed when building `mxnet` with mkl support,

[GitHub] lightingghost commented on issue #10239: Error in Building and Installing packages on Windows using Visual Studio 2017

2018-03-24 Thread GitBox
lightingghost commented on issue #10239: Error in Building and Installing packages on Windows using Visual Studio 2017 URL: https://github.com/apache/incubator-mxnet/issues/10239#issuecomment-375943999 `iomp5.lib` should be the library file needed when building `mxnet` with mkl support,

[GitHub] piiswrong commented on issue #9933: [MXNET-23] Adding support to profile kvstore server during distributed training

2018-03-24 Thread GitBox
piiswrong commented on issue #9933: [MXNET-23] Adding support to profile kvstore server during distributed training URL: https://github.com/apache/incubator-mxnet/pull/9933#issuecomment-375943787 profile_process does sound like a good name.

[GitHub] szha commented on issue #10207: mxnet.ndarray.khatri_rao needs GPU version

2018-03-24 Thread GitBox
szha commented on issue #10207: mxnet.ndarray.khatri_rao needs GPU version URL: https://github.com/apache/incubator-mxnet/issues/10207#issuecomment-375942910 @piiswrong @cswiercz did you get a chance to sync? What's the conclusion?

[GitHub] szha commented on issue #10236: Fix the order of `path.join` and `path.expanduser`

2018-03-24 Thread GitBox
szha commented on issue #10236: Fix the order of `path.join` and `path.expanduser` URL: https://github.com/apache/incubator-mxnet/pull/10236#issuecomment-375942840 Apologies. I overlooked this. Thanks for the fix. This is an

[incubator-mxnet] branch master updated: Fix the order of `path.join` and `path.expanduser` (#10236)

2018-03-24 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new 0ca6dd2 Fix the order of

[GitHub] szha closed pull request #10236: Fix the order of `path.join` and `path.expanduser`

2018-03-24 Thread GitBox
szha closed pull request #10236: Fix the order of `path.join` and `path.expanduser` URL: https://github.com/apache/incubator-mxnet/pull/10236 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[GitHub] cjolivier01 commented on issue #10234: [MXNET-135] mx.image.imread support s3

2018-03-24 Thread GitBox
cjolivier01 commented on issue #10234: [MXNET-135] mx.image.imread support s3 URL: https://github.com/apache/incubator-mxnet/pull/10234#issuecomment-375941742 unit tests? This is an automated message from the Apache Git

[GitHub] cjolivier01 commented on a change in pull request #10234: [MXNET-135] mx.image.imread support s3

2018-03-24 Thread GitBox
cjolivier01 commented on a change in pull request #10234: [MXNET-135] mx.image.imread support s3 URL: https://github.com/apache/incubator-mxnet/pull/10234#discussion_r176927212 ## File path: src/io/image_io.cc ## @@ -217,17 +217,28 @@ void Imread(const nnvm::NodeAttrs&

[GitHub] cjolivier01 commented on a change in pull request #10234: [MXNET-135] mx.image.imread support s3

2018-03-24 Thread GitBox
cjolivier01 commented on a change in pull request #10234: [MXNET-135] mx.image.imread support s3 URL: https://github.com/apache/incubator-mxnet/pull/10234#discussion_r176927167 ## File path: src/io/image_io.cc ## @@ -217,17 +217,28 @@ void Imread(const nnvm::NodeAttrs&

[GitHub] cjolivier01 commented on a change in pull request #10234: [MXNET-135] mx.image.imread support s3

2018-03-24 Thread GitBox
cjolivier01 commented on a change in pull request #10234: [MXNET-135] mx.image.imread support s3 URL: https://github.com/apache/incubator-mxnet/pull/10234#discussion_r176927089 ## File path: src/io/image_io.cc ## @@ -217,17 +217,28 @@ void Imread(const nnvm::NodeAttrs&

[GitHub] cjolivier01 commented on a change in pull request #10234: [MXNET-135] mx.image.imread support s3

2018-03-24 Thread GitBox
cjolivier01 commented on a change in pull request #10234: [MXNET-135] mx.image.imread support s3 URL: https://github.com/apache/incubator-mxnet/pull/10234#discussion_r176927070 ## File path: src/io/image_io.cc ## @@ -217,17 +217,28 @@ void Imread(const nnvm::NodeAttrs&

[GitHub] piiswrong commented on a change in pull request #10014: [MXNET-81] Fix crash with mx.nd.ones

2018-03-24 Thread GitBox
piiswrong commented on a change in pull request #10014: [MXNET-81] Fix crash with mx.nd.ones URL: https://github.com/apache/incubator-mxnet/pull/10014#discussion_r176927022 ## File path: src/engine/threaded_engine.cc ## @@ -309,6 +309,13 @@ void

[GitHub] piiswrong commented on a change in pull request #10014: [MXNET-81] Fix crash with mx.nd.ones

2018-03-24 Thread GitBox
piiswrong commented on a change in pull request #10014: [MXNET-81] Fix crash with mx.nd.ones URL: https://github.com/apache/incubator-mxnet/pull/10014#discussion_r176927010 ## File path: src/engine/threaded_engine.cc ## @@ -309,6 +309,13 @@ void

[GitHub] piiswrong commented on issue #10234: [MXNET-135] mx.image.imread support s3

2018-03-24 Thread GitBox
piiswrong commented on issue #10234: [MXNET-135] mx.image.imread support s3 URL: https://github.com/apache/incubator-mxnet/pull/10234#issuecomment-375940739 This is a lot more complicated and likely slower This is an

[GitHub] piiswrong commented on issue #10236: Fix the order of `path.join` and `path.expanduser`

2018-03-24 Thread GitBox
piiswrong commented on issue #10236: Fix the order of `path.join` and `path.expanduser` URL: https://github.com/apache/incubator-mxnet/pull/10236#issuecomment-375940625 @szha Because expanduser only takes one argument. This simply own't run. ``` >>> os.path.expanduser('~/xxx',

[GitHub] conradwt commented on issue #621: Support for other Device Types, OpenCL AMD GPU

2018-03-24 Thread GitBox
conradwt commented on issue #621: Support for other Device Types, OpenCL AMD GPU URL: https://github.com/apache/incubator-mxnet/issues/621#issuecomment-375930949 Hi All, are there any updates regarding this topic because I would like to see OpenCL be the default for MXNet as well as other

[GitHub] harryprince commented on issue #10147: Mxnet R Package Installation Doc Bug

2018-03-24 Thread GitBox
harryprince commented on issue #10147: Mxnet R Package Installation Doc Bug URL: https://github.com/apache/incubator-mxnet/issues/10147#issuecomment-375933405 I recommend someone officially public a stable version mxnet package to CRAN, just like `pip install mxnet-cu80`, which can make

[GitHub] harryprince opened a new issue #10147: Mxnet R Package Installation Doc Bug

2018-03-24 Thread GitBox
harryprince opened a new issue #10147: Mxnet R Package Installation Doc Bug URL: https://github.com/apache/incubator-mxnet/issues/10147 Wrong: ``` # current repo doc: https://github.com/apache/incubator-mxnet/tree/master/R-package cran <- getOption("repos") cran["dmlc"] <-

[GitHub] harryprince closed issue #10147: Mxnet R Package Installation Doc Bug

2018-03-24 Thread GitBox
harryprince closed issue #10147: Mxnet R Package Installation Doc Bug URL: https://github.com/apache/incubator-mxnet/issues/10147 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] dl002 opened a new issue #10239: Error in Building and Installing packages on Windows using Visual Studio 2017

2018-03-24 Thread GitBox
dl002 opened a new issue #10239: Error in Building and Installing packages on Windows using Visual Studio 2017 URL: https://github.com/apache/incubator-mxnet/issues/10239 When trying to build on Windows following instructions

[GitHub] eric-haibin-lin opened a new issue #10238: Incorrect API Doc on website

2018-03-24 Thread GitBox
eric-haibin-lin opened a new issue #10238: Incorrect API Doc on website URL: https://github.com/apache/incubator-mxnet/issues/10238 I compared the API documentation in the source code and on the website. They are different. Is the website refreshed at all? Specifically, the sparse

[GitHub] conradwt commented on issue #621: Support for other Device Types, OpenCL AMD GPU

2018-03-24 Thread GitBox
conradwt commented on issue #621: Support for other Device Types, OpenCL AMD GPU URL: https://github.com/apache/incubator-mxnet/issues/621#issuecomment-375930949 Hi All, are there any updates regarding this topic because I would like to see OpenCL be the default for many of these ML

[GitHub] szha commented on issue #8601: Why is the accuracy overestimated?

2018-03-24 Thread GitBox
szha commented on issue #8601: Why is the accuracy overestimated? URL: https://github.com/apache/incubator-mxnet/issues/8601#issuecomment-375920480 @leleamol @marcoabreu please refrain from manually adding "Need Triage" label while triaging, and be sure to remove the "Need Triage" label

[GitHub] szha commented on issue #8646: Bug in FullyConnected???

2018-03-24 Thread GitBox
szha commented on issue #8646: Bug in FullyConnected??? URL: https://github.com/apache/incubator-mxnet/issues/8646#issuecomment-375920464 @anirudh2290 @eric-haibin-lin please refrain from manually adding "Need Triage" label while triaging. Thanks.

[GitHub] szha commented on issue #8601: Why is the accuracy overestimated?

2018-03-24 Thread GitBox
szha commented on issue #8601: Why is the accuracy overestimated? URL: https://github.com/apache/incubator-mxnet/issues/8601#issuecomment-375920480 @leleamol @marcoabreu please refrain from manually adding "Need Triage" label while triaging. Thanks.

[GitHub] szha commented on issue #8673: Unexpected autograd behaviors (about variables and scope)

2018-03-24 Thread GitBox
szha commented on issue #8673: Unexpected autograd behaviors (about variables and scope) URL: https://github.com/apache/incubator-mxnet/issues/8673#issuecomment-375920439 @anirudh2290 @eric-haibin-lin please refrain from manually adding "Need Triage" label while triaging. Thanks.

[GitHub] szha commented on issue #8969: A couple of issues in benchmark_score.py

2018-03-24 Thread GitBox
szha commented on issue #8969: A couple of issues in benchmark_score.py URL: https://github.com/apache/incubator-mxnet/issues/8969#issuecomment-375920392 @gautamkmr @sandeep-krishnamurthy please refrain from manually adding "Need Triage" label while triaging. Thanks.

[GitHub] szha commented on issue #8953: Run 'fine-tuning' example found【Wrong number of inputs】on "python3.5 mxnet 0.11.0 centos7 "

2018-03-24 Thread GitBox
szha commented on issue #8953: Run 'fine-tuning' example found【Wrong number of inputs】on "python3.5 mxnet 0.11.0 centos7 " URL: https://github.com/apache/incubator-mxnet/issues/8953#issuecomment-375920417 @leleamol @sandeep-krishnamurthy please refrain from manually adding "Need Triage"

[GitHub] szha commented on issue #9520: Image augumention crash when set MXNET_CPU_WORKER_NTHREADS bigger than 3

2018-03-24 Thread GitBox
szha commented on issue #9520: Image augumention crash when set MXNET_CPU_WORKER_NTHREADS bigger than 3 URL: https://github.com/apache/incubator-mxnet/issues/9520#issuecomment-375920306 @sandeep-krishnamurthy @vdantu please refrain from manually adding "Need Triage" label while

[incubator-mxnet] branch master updated: fix word language model script and readme (#10225)

2018-03-24 Thread sxjscience
This is an automated email from the ASF dual-hosted git repository. sxjscience pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new 200f541 fix word language model

[GitHub] sxjscience commented on issue #10225: fix word language model script and readme

2018-03-24 Thread GitBox
sxjscience commented on issue #10225: fix word language model script and readme URL: https://github.com/apache/incubator-mxnet/pull/10225#issuecomment-375919910 I’ll merge it first. We will submit another PR to fix the network structure.

[GitHub] sxjscience closed pull request #10225: fix word language model script and readme

2018-03-24 Thread GitBox
sxjscience closed pull request #10225: fix word language model script and readme URL: https://github.com/apache/incubator-mxnet/pull/10225 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[GitHub] sxjscience closed issue #10224: Language model example cannot be run

2018-03-24 Thread GitBox
sxjscience closed issue #10224: Language model example cannot be run URL: https://github.com/apache/incubator-mxnet/issues/10224 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] szha commented on issue #10236: Fix the order of `path.join` and `path.expanduser`

2018-03-24 Thread GitBox
szha commented on issue #10236: Fix the order of `path.join` and `path.expanduser` URL: https://github.com/apache/incubator-mxnet/pull/10236#issuecomment-375914310 In what case would the order of these two function calls matter?

[GitHub] cjolivier01 commented on issue #10228: [MXNET-134] Don't do imperative bulk when profiling aggregate stats

2018-03-24 Thread GitBox
cjolivier01 commented on issue #10228: [MXNET-134] Don't do imperative bulk when profiling aggregate stats URL: https://github.com/apache/incubator-mxnet/pull/10228#issuecomment-375901906 I was thinking about this last night and I think I can come up with a more elegant solution. However,

[GitHub] larroy commented on issue #10214: Bugfix for USE_SSE in CMake

2018-03-24 Thread GitBox
larroy commented on issue #10214: Bugfix for USE_SSE in CMake URL: https://github.com/apache/incubator-mxnet/pull/10214#issuecomment-375902753 LGTM This is an automated message from the Apache Git Service. To respond to the

[GitHub] harryprince commented on issue #10147: Mxnet R Package Installation Doc Bug

2018-03-24 Thread GitBox
harryprince commented on issue #10147: Mxnet R Package Installation Doc Bug URL: https://github.com/apache/incubator-mxnet/issues/10147#issuecomment-375893415 I summarise an installation example on GPU/linux in detail for Chinese users: https://segmentfault.com/a/119013899075

[GitHub] marcoabreu commented on issue #10218: [Do Not Merge]Create CentOS gcc4.8 & MKLDNN ci-test

2018-03-24 Thread GitBox
marcoabreu commented on issue #10218: [Do Not Merge]Create CentOS gcc4.8 & MKLDNN ci-test URL: https://github.com/apache/incubator-mxnet/pull/10218#issuecomment-375875037 That's the way to go, thanks a lot! This is an

[GitHub] dl002 opened a new issue #10237: No setupenv.cmd present in the folder

2018-03-24 Thread GitBox
dl002 opened a new issue #10237: No setupenv.cmd present in the folder URL: https://github.com/apache/incubator-mxnet/issues/10237 The installation docs [here](https://github.com/apache/incubator-mxnet/blob/master/docs/install/windows_setup.md) say the following at [Installing the

[GitHub] tdomhan commented on issue #1740: Installed MxNet on Win7 computer, but when I use import mxnet I get an error

2018-03-24 Thread GitBox
tdomhan commented on issue #1740: Installed MxNet on Win7 computer, but when I use import mxnet I get an error URL: https://github.com/apache/incubator-mxnet/issues/1740#issuecomment-375865684 In case someone else finds this issue to use dumpbin: 1. Open the Visual Studio developer

[GitHub] harryprince commented on issue #10147: Mxnet R Package Installation Doc Bug

2018-03-24 Thread GitBox
harryprince commented on issue #10147: Mxnet R Package Installation Doc Bug URL: https://github.com/apache/incubator-mxnet/issues/10147#issuecomment-375860684 same issue like: #10088 #7985 This is an automated

[GitHub] TaoLv commented on issue #10218: [Do Not Merge]Create CentOS gcc4.8 & MKLDNN ci-test

2018-03-24 Thread GitBox
TaoLv commented on issue #10218: [Do Not Merge]Create CentOS gcc4.8 & MKLDNN ci-test URL: https://github.com/apache/incubator-mxnet/pull/10218#issuecomment-375859617 GCC version check is added. Currently, I don't know which versions of msvc and clang support std::align, so I restrict this

[GitHub] TaoLv commented on issue #10218: [Do Not Merge]Create CentOS gcc4.8 & MKLDNN ci-test

2018-03-24 Thread GitBox
TaoLv commented on issue #10218: [Do Not Merge]Create CentOS gcc4.8 & MKLDNN ci-test URL: https://github.com/apache/incubator-mxnet/pull/10218#issuecomment-375859617 GCC version check is added. Currently, I don't know which versions of msvc and clang support std::align, so I restrict this

[GitHub] Lyken17 opened a new pull request #10236: Fix the order of `path.join` and `path.expanduser`

2018-03-24 Thread GitBox
Lyken17 opened a new pull request #10236: Fix the order of `path.join` and `path.expanduser` URL: https://github.com/apache/incubator-mxnet/pull/10236 ## Description ## It seems the order of `join` and `expanduser` are somehow reversed. This is a pull request to fix the tiny bug.

[GitHub] HudsonHuang commented on issue #9461: The fault in the process of running example

2018-03-24 Thread GitBox
HudsonHuang commented on issue #9461: The fault in the process of running example URL: https://github.com/apache/incubator-mxnet/issues/9461#issuecomment-375852843 @zhangguotai Could you tell me how did you solved it? I meet the same problem.

[GitHub] zheng-da commented on issue #10215: Avoid MKLDNN in-place data conversion in NDArray synchronously

2018-03-24 Thread GitBox
zheng-da commented on issue #10215: Avoid MKLDNN in-place data conversion in NDArray synchronously URL: https://github.com/apache/incubator-mxnet/pull/10215#issuecomment-375851756 Yes, it's out of the scope of this PR. NNVM is the right place for graph optimization. I'm just curious what

[GitHub] asitstands opened a new issue #10235: Build fails with USE_CUDNN = 0

2018-03-24 Thread GitBox
asitstands opened a new issue #10235: Build fails with USE_CUDNN = 0 URL: https://github.com/apache/incubator-mxnet/issues/10235 @zheng-da I got the following errors during the build with `USE_CUDNN = 0`. It looks like introduced with #10116. ```

[GitHub] TaoLv commented on issue #10215: Avoid MKLDNN in-place data conversion in NDArray synchronously

2018-03-24 Thread GitBox
TaoLv commented on issue #10215: Avoid MKLDNN in-place data conversion in NDArray synchronously URL: https://github.com/apache/incubator-mxnet/pull/10215#issuecomment-375851062 @zheng-da Thanks for the explaination and the great effort for this issue. I agree that end users have no need