[GitHub] stu1130 commented on issue #13891: Fixed java benchmark failing error by fixing the classpath

2019-01-15 Thread GitBox
stu1130 commented on issue #13891: Fixed java benchmark failing error by fixing the classpath URL: https://github.com/apache/incubator-mxnet/pull/13891#issuecomment-454600664 @mxnet-label-bot add [pr-awaiting-merge] Thanks @piyushghai

[GitHub] stu1130 commented on issue #13890: Improve bulking in Gluon

2019-01-15 Thread GitBox
stu1130 commented on issue #13890: Improve bulking in Gluon URL: https://github.com/apache/incubator-mxnet/pull/13890#issuecomment-454600502 Thanks for your contribution @ptrendx @mxnet-label-bot add [pr-awaiting-review]

[incubator-mxnet] Diff for: [GitHub] lanking520 merged pull request #13856: License fixed with copyrights on 1.4.x release branch

2019-01-15 Thread GitBox
diff --git a/3rdparty/googletest b/3rdparty/googletest index ec44c6c1675..eb9225ce361 16 --- a/3rdparty/googletest +++ b/3rdparty/googletest @@ -1 +1 @@ -Subproject commit ec44c6c1675c25b9827aacd08c02433cccde7780 +Subproject commit eb9225ce361affe561592e0912320b9db84985d0 diff --git a/LICENSE

[incubator-mxnet] branch v1.4.x updated (84beaf1 -> a9bdeb5)

2019-01-15 Thread lanking
This is an automated email from the ASF dual-hosted git repository. lanking pushed a change to branch v1.4.x in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 84beaf1 Fix incorrect delete in MXExecutorReshape exception handling (#13376) (#13824) add a9bdeb5

[GitHub] lanking520 merged pull request #13856: License fixed with copyrights on 1.4.x release branch

2019-01-15 Thread GitBox
lanking520 merged pull request #13856: License fixed with copyrights on 1.4.x release branch URL: https://github.com/apache/incubator-mxnet/pull/13856 As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing list, comm...@mxnet.apache.org

[GitHub] stu1130 commented on a change in pull request #13848: [MXNET-1291] solve pylint errors in examples with issue no.12205

2019-01-15 Thread GitBox
stu1130 commented on a change in pull request #13848: [MXNET-1291] solve pylint errors in examples with issue no.12205 URL: https://github.com/apache/incubator-mxnet/pull/13848#discussion_r248107682 ## File path: example/cnn_text_classification/data_helpers.py ## @@ -81,8

[GitHub] stu1130 commented on a change in pull request #13848: [MXNET-1291] solve pylint errors in examples with issue no.12205

2019-01-15 Thread GitBox
stu1130 commented on a change in pull request #13848: [MXNET-1291] solve pylint errors in examples with issue no.12205 URL: https://github.com/apache/incubator-mxnet/pull/13848#discussion_r248109412 ## File path: example/autoencoder/model.py ## @@ -53,7 +54,7 @@ def

[GitHub] stu1130 commented on a change in pull request #13848: [MXNET-1291] solve pylint errors in examples with issue no.12205

2019-01-15 Thread GitBox
stu1130 commented on a change in pull request #13848: [MXNET-1291] solve pylint errors in examples with issue no.12205 URL: https://github.com/apache/incubator-mxnet/pull/13848#discussion_r248106161 ## File path: example/cnn_text_classification/data_helpers.py ## @@ -15,6

[GitHub] stu1130 commented on a change in pull request #13848: [MXNET-1291] solve pylint errors in examples with issue no.12205

2019-01-15 Thread GitBox
stu1130 commented on a change in pull request #13848: [MXNET-1291] solve pylint errors in examples with issue no.12205 URL: https://github.com/apache/incubator-mxnet/pull/13848#discussion_r248105482 ## File path: example/cnn_chinese_text_classification/data_helpers.py ##

[GitHub] stu1130 commented on a change in pull request #13848: [MXNET-1291] solve pylint errors in examples with issue no.12205

2019-01-15 Thread GitBox
stu1130 commented on a change in pull request #13848: [MXNET-1291] solve pylint errors in examples with issue no.12205 URL: https://github.com/apache/incubator-mxnet/pull/13848#discussion_r248109730 ## File path: example/distributed_training/cifar10_dist.py ## @@ -124,38

[GitHub] stu1130 commented on a change in pull request #13848: [MXNET-1291] solve pylint errors in examples with issue no.12205

2019-01-15 Thread GitBox
stu1130 commented on a change in pull request #13848: [MXNET-1291] solve pylint errors in examples with issue no.12205 URL: https://github.com/apache/incubator-mxnet/pull/13848#discussion_r248107754 ## File path: example/cnn_text_classification/text_cnn.py ## @@ -19,12

[GitHub] stu1130 commented on a change in pull request #13848: [MXNET-1291] solve pylint errors in examples with issue no.12205

2019-01-15 Thread GitBox
stu1130 commented on a change in pull request #13848: [MXNET-1291] solve pylint errors in examples with issue no.12205 URL: https://github.com/apache/incubator-mxnet/pull/13848#discussion_r248107682 ## File path: example/cnn_text_classification/data_helpers.py ## @@ -81,8

[GitHub] NRauschmayr commented on issue #13310: [MXNET-703] Update to TensorRT 5, ONNX IR 3. Fix inference bugs.

2019-01-15 Thread GitBox
NRauschmayr commented on issue #13310: [MXNET-703] Update to TensorRT 5, ONNX IR 3. Fix inference bugs. URL: https://github.com/apache/incubator-mxnet/pull/13310#issuecomment-454593897 @KellenSunderland Sure, that's ok! There is no hurry for the changes which I suggested.

[GitHub] fhieber commented on issue #13862: [Bug] unravel_index no longer works with magic '-1' in shape parameter after 1.3.1

2019-01-15 Thread GitBox
fhieber commented on issue #13862: [Bug] unravel_index no longer works with magic '-1' in shape parameter after 1.3.1 URL: https://github.com/apache/incubator-mxnet/issues/13862#issuecomment-454592661 Could someone test this also with the 1.4 release candidate? If this is present there

[GitHub] ptrendx commented on issue #13890: Improve bulking in Gluon

2019-01-15 Thread GitBox
ptrendx commented on issue #13890: Improve bulking in Gluon URL: https://github.com/apache/incubator-mxnet/pull/13890#issuecomment-454592050 Also adding @KellenSunderland This is an automated message from the Apache Git

[GitHub] KellenSunderland commented on issue #12328: [WIP] [MXNET-857] Add initial NVTX profiler implementation

2019-01-15 Thread GitBox
KellenSunderland commented on issue #12328: [WIP] [MXNET-857] Add initial NVTX profiler implementation URL: https://github.com/apache/incubator-mxnet/pull/12328#issuecomment-454591869 @stu1130 @sandeep-krishnamurthy @roywei @anirudh2290 @lupesko I'm about to resume work to finish this

[GitHub] KellenSunderland commented on issue #13311: [MXNET-703] Minor refactor of TensorRT code

2019-01-15 Thread GitBox
KellenSunderland commented on issue #13311: [MXNET-703] Minor refactor of TensorRT code URL: https://github.com/apache/incubator-mxnet/pull/13311#issuecomment-454590672 Rebased and fixed the last nit about static_cast. I believe this can be merged if a committer could do a quick review.

[GitHub] KellenSunderland commented on a change in pull request #13311: [MXNET-703] Minor refactor of TensorRT code

2019-01-15 Thread GitBox
KellenSunderland commented on a change in pull request #13311: [MXNET-703] Minor refactor of TensorRT code URL: https://github.com/apache/incubator-mxnet/pull/13311#discussion_r248101123 ## File path: src/executor/trt_graph_executor.cc ## @@ -434,7 +435,7 @@ Executor

[GitHub] stu1130 commented on issue #13683: fix bug in nag optimizer

2019-01-15 Thread GitBox
stu1130 commented on issue #13683: fix bug in nag optimizer URL: https://github.com/apache/incubator-mxnet/pull/13683#issuecomment-454589276 @mxnet-label-bot update [pr-work-in-progress] This is an automated message from the

[incubator-mxnet] branch master updated: Temporarily disable website testing (#13887)

2019-01-15 Thread kellen
This is an automated email from the ASF dual-hosted git repository. kellen 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 9d42812 Temporarily disable website

[GitHub] KellenSunderland merged pull request #13887: Temporarily disable website builds

2019-01-15 Thread GitBox
KellenSunderland merged pull request #13887: Temporarily disable website builds URL: https://github.com/apache/incubator-mxnet/pull/13887 As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing list, comm...@mxnet.apache.org

[incubator-mxnet] Diff for: [GitHub] KellenSunderland merged pull request #13887: Temporarily disable website builds

2019-01-15 Thread GitBox
diff --git a/ci/jenkins/Jenkinsfile_website b/ci/jenkins/Jenkinsfile_website index acdd2be4d00..95ea02b5d24 100644 --- a/ci/jenkins/Jenkinsfile_website +++ b/ci/jenkins/Jenkinsfile_website @@ -37,10 +37,11 @@ core_logic: { custom_steps.compile_unix_cpu_openblas() ]) -

[GitHub] stu1130 edited a comment on issue #13609: [MXNET-1258]fix unittest for ROIAlign Operator

2019-01-15 Thread GitBox
stu1130 edited a comment on issue #13609: [MXNET-1258]fix unittest for ROIAlign Operator URL: https://github.com/apache/incubator-mxnet/pull/13609#issuecomment-454586816 @mxnet-label-bot update [pr-work-in-progress] let us know once you've done the check

[GitHub] stu1130 commented on issue #13609: [MXNET-1258]fix unittest for ROIAlign Operator

2019-01-15 Thread GitBox
stu1130 commented on issue #13609: [MXNET-1258]fix unittest for ROIAlign Operator URL: https://github.com/apache/incubator-mxnet/pull/13609#issuecomment-454586816 @mxnet-label-bot update [pr-work-in-progress] This is an

[GitHub] ptrendx commented on issue #13890: Improve bulking in Gluon

2019-01-15 Thread GitBox
ptrendx commented on issue #13890: Improve bulking in Gluon URL: https://github.com/apache/incubator-mxnet/pull/13890#issuecomment-454586193 Perf comparison: ResNet-50 v1d from GluonCV on synthetic data, using slightly modified script to enable synthetic data, DGX1-V 32G. - Default

[GitHub] stu1130 commented on issue #13083: [MXNET-345] Hardmax operator

2019-01-15 Thread GitBox
stu1130 commented on issue #13083: [MXNET-345] Hardmax operator URL: https://github.com/apache/incubator-mxnet/pull/13083#issuecomment-454585417 @mxnet-label-bot update [pr-work-in-progress] I assume  mean yes!  This is

[GitHub] zachgk commented on issue #13856: License fixed with copyrights on 1.4.x release branch

2019-01-15 Thread GitBox
zachgk commented on issue #13856: License fixed with copyrights on 1.4.x release branch URL: https://github.com/apache/incubator-mxnet/pull/13856#issuecomment-454584714 @lanking520 @gigasquid @aaronmarkham @ddavydenko This

[GitHub] stu1130 commented on issue #12821: [MXNET-1095] [WIP] Support higher order gradient for cos, sin, negative, mul

2019-01-15 Thread GitBox
stu1130 commented on issue #12821: [MXNET-1095] [WIP] Support higher order gradient for cos,sin,negative,mul URL: https://github.com/apache/incubator-mxnet/pull/12821#issuecomment-454583892 @sxjscience any plan to work on this? Thanks

[GitHub] apeforest commented on issue #11203: ConvTranspose2d giving incorrect output

2019-01-15 Thread GitBox
apeforest commented on issue #11203: ConvTranspose2d giving incorrect output URL: https://github.com/apache/incubator-mxnet/issues/11203#issuecomment-454579934 Finally got some bandwidth. Looking into this issue again. This

[incubator-mxnet] Diff for: [GitHub] sandeep-krishnamurthy closed pull request #12408: [MXNET-875] pass global step and names of params to callback function stat_func i…

2019-01-15 Thread GitBox
diff --git a/python/mxnet/monitor.py b/python/mxnet/monitor.py index e3185a1281a..732728ec2d4 100644 --- a/python/mxnet/monitor.py +++ b/python/mxnet/monitor.py @@ -67,7 +67,7 @@ def stat_helper(name, array): array = NDArray(array, writable=False) if not self.activated or

[GitHub] stu1130 commented on issue #12641: Add docker file which includes sample jupyter notebook (#12611)

2019-01-15 Thread GitBox
stu1130 commented on issue #12641: Add docker file which includes sample jupyter notebook (#12611) URL: https://github.com/apache/incubator-mxnet/pull/12641#issuecomment-454578753 @nswamy ping again thanks This is an

[GitHub] sandeep-krishnamurthy closed pull request #12408: [MXNET-875] pass global step and names of params to callback function stat_func i…

2019-01-15 Thread GitBox
sandeep-krishnamurthy closed pull request #12408: [MXNET-875] pass global step and names of params to callback function stat_func i… URL: https://github.com/apache/incubator-mxnet/pull/12408 As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing

[GitHub] stu1130 edited a comment on issue #12408: [MXNET-875] pass global step and names of params to callback function stat_func i…

2019-01-15 Thread GitBox
stu1130 edited a comment on issue #12408: [MXNET-875] pass global step and names of params to callback function stat_func i… URL: https://github.com/apache/incubator-mxnet/pull/12408#issuecomment-454578066 @knwng since the PR has been inactive for a while, we will close it. Feel free to

[GitHub] stu1130 commented on issue #12408: [MXNET-875] pass global step and names of params to callback function stat_func i…

2019-01-15 Thread GitBox
stu1130 commented on issue #12408: [MXNET-875] pass global step and names of params to callback function stat_func i… URL: https://github.com/apache/incubator-mxnet/pull/12408#issuecomment-454578066 @knwng since the PR is inactive for a while, we will close it. Feel free to reopen it if

[GitHub] stu1130 commented on issue #12328: [WIP] [MXNET-857] Add initial NVTX profiler implementation

2019-01-15 Thread GitBox
stu1130 commented on issue #12328: [WIP] [MXNET-857] Add initial NVTX profiler implementation URL: https://github.com/apache/incubator-mxnet/pull/12328#issuecomment-454576932 @KellenSunderland ping for the update. Thanks

[GitHub] stu1130 commented on a change in pull request #10792: [MXNET-400] support string type for kvstore key in cpp-package

2019-01-15 Thread GitBox
stu1130 commented on a change in pull request #10792: [MXNET-400] support string type for kvstore key in cpp-package URL: https://github.com/apache/incubator-mxnet/pull/10792#discussion_r248069285 ## File path: cpp-package/example/test_kvstore.cpp ## @@ -0,0 +1,132 @@ +/*

[GitHub] stu1130 commented on a change in pull request #10792: [MXNET-400] support string type for kvstore key in cpp-package

2019-01-15 Thread GitBox
stu1130 commented on a change in pull request #10792: [MXNET-400] support string type for kvstore key in cpp-package URL: https://github.com/apache/incubator-mxnet/pull/10792#discussion_r248077985 ## File path: cpp-package/example/test_kvstore.cpp ## @@ -0,0 +1,132 @@ +/*

[GitHub] zachgk commented on a change in pull request #13807: [MXNET-1180] Java Image API

2019-01-15 Thread GitBox
zachgk commented on a change in pull request #13807: [MXNET-1180] Java Image API URL: https://github.com/apache/incubator-mxnet/pull/13807#discussion_r248083486 ## File path: scala-package/examples/pom.xml ## @@ -128,5 +129,11 @@ slf4j-simple 1.7.5 +

[GitHub] zachgk commented on a change in pull request #12690: [MXNET-1000] get Ndarray real value and form it from a NDArray

2019-01-15 Thread GitBox
zachgk commented on a change in pull request #12690: [MXNET-1000] get Ndarray real value and form it from a NDArray URL: https://github.com/apache/incubator-mxnet/pull/12690#discussion_r248080616 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/NDArray.scala

[incubator-mxnet] Diff for: [GitHub] sandeep-krishnamurthy merged pull request #13868: update Spectral Normalization Code

2019-01-15 Thread GitBox
diff --git a/example/gluon/sn_gan/model.py b/example/gluon/sn_gan/model.py index 6040adb4eea..cfd7f93e8da 100644 --- a/example/gluon/sn_gan/model.py +++ b/example/gluon/sn_gan/model.py @@ -21,7 +21,7 @@ import mxnet as mx from mxnet import nd -from mxnet import gluon +from mxnet import gluon,

[incubator-mxnet] branch master updated (078a6c2 -> 3a97487)

2019-01-15 Thread skm
This is an automated email from the ASF dual-hosted git repository. skm pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 078a6c2 Fix SN-GAN example doc (#13877) add 3a97487 update Spectral Normalization Code (#13868) No new

[GitHub] sandeep-krishnamurthy closed issue #13847: Spectralnormalization GAN seems to be seriously broken

2019-01-15 Thread GitBox
sandeep-krishnamurthy closed issue #13847: Spectralnormalization GAN seems to be seriously broken URL: https://github.com/apache/incubator-mxnet/issues/13847 This is an automated message from the Apache Git Service. To

[GitHub] zachgk commented on a change in pull request #12690: [MXNET-1000] get Ndarray real value and form it from a NDArray

2019-01-15 Thread GitBox
zachgk commented on a change in pull request #12690: [MXNET-1000] get Ndarray real value and form it from a NDArray URL: https://github.com/apache/incubator-mxnet/pull/12690#discussion_r248079764 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/NDArray.scala

[GitHub] sandeep-krishnamurthy merged pull request #13868: update Spectral Normalization Code

2019-01-15 Thread GitBox
sandeep-krishnamurthy merged pull request #13868: update Spectral Normalization Code URL: https://github.com/apache/incubator-mxnet/pull/13868 As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing list, comm...@mxnet.apache.org

[GitHub] zachgk commented on a change in pull request #12690: [MXNET-1000] get Ndarray real value and form it from a NDArray

2019-01-15 Thread GitBox
zachgk commented on a change in pull request #12690: [MXNET-1000] get Ndarray real value and form it from a NDArray URL: https://github.com/apache/incubator-mxnet/pull/12690#discussion_r248081895 ## File path: scala-package/core/src/test/scala/org/apache/mxnet/NDArraySuite.scala

[GitHub] zachgk commented on a change in pull request #12690: [MXNET-1000] get Ndarray real value and form it from a NDArray

2019-01-15 Thread GitBox
zachgk commented on a change in pull request #12690: [MXNET-1000] get Ndarray real value and form it from a NDArray URL: https://github.com/apache/incubator-mxnet/pull/12690#discussion_r247710541 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/NDArray.scala

[incubator-mxnet] branch master updated: Fix SN-GAN example doc (#13877)

2019-01-15 Thread skm
This is an automated email from the ASF dual-hosted git repository. skm 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 078a6c2 Fix SN-GAN example doc (#13877)

[incubator-mxnet] Diff for: [GitHub] sandeep-krishnamurthy merged pull request #13877: Fix SN-GAN example doc

2019-01-15 Thread GitBox
diff --git a/example/README.md b/example/README.md index 2123104a148..dea7e289e6c 100644 --- a/example/README.md +++ b/example/README.md @@ -95,7 +95,7 @@ If your tutorial depends on specific packages, simply add them to this provision * [Gluon Examples](gluon) - several examples using the Gluon

[GitHub] sandeep-krishnamurthy merged pull request #13877: Fix SN-GAN example doc

2019-01-15 Thread GitBox
sandeep-krishnamurthy merged pull request #13877: Fix SN-GAN example doc URL: https://github.com/apache/incubator-mxnet/pull/13877 As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing list, comm...@mxnet.apache.org

[GitHub] lebeg commented on issue #11549: Pip package can be much faster (OpenCV version?)

2019-01-15 Thread GitBox
lebeg commented on issue #11549: Pip package can be much faster (OpenCV version?) URL: https://github.com/apache/incubator-mxnet/issues/11549#issuecomment-454567165 Actually, we think we found the reason for the slowdown in CMake builds. It's due to the fact that even with MKL set to be

[GitHub] lebeg commented on a change in pull request #13681: Making MKL-DNN default on MXNet master

2019-01-15 Thread GitBox
lebeg commented on a change in pull request #13681: Making MKL-DNN default on MXNet master URL: https://github.com/apache/incubator-mxnet/pull/13681#discussion_r248077722 ## File path: CMakeLists.txt ## @@ -20,7 +20,7 @@ mxnet_option(USE_F16C "Build with x86

[GitHub] aaronmarkham commented on issue #13831: OSError: [Errno 14] Bad address on website CI tests

2019-01-15 Thread GitBox
aaronmarkham commented on issue #13831: OSError: [Errno 14] Bad address on website CI tests URL: https://github.com/apache/incubator-mxnet/issues/13831#issuecomment-454563102 I tried to reproduce with several simultaneous docker containers and running it locally. It doesn't get the OS

[GitHub] aaronmarkham opened a new issue #13892: switching versions while on a specific API page loses the anchor

2019-01-15 Thread GitBox
aaronmarkham opened a new issue #13892: switching versions while on a specific API page loses the anchor URL: https://github.com/apache/incubator-mxnet/issues/13892 ## Description Say you're looking a specific API for a released version, then want to look at in master. You're on:

[GitHub] piyushghai opened a new pull request #13891: Fixed java benchmark failing error by fixing the classpath

2019-01-15 Thread GitBox
piyushghai opened a new pull request #13891: Fixed java benchmark failing error by fixing the classpath URL: https://github.com/apache/incubator-mxnet/pull/13891 ## Description ## As title. ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for

[incubator-mxnet] Diff for: [GitHub] lanking520 merged pull request #13889: fix the GPU flavor problem for Scala

2019-01-15 Thread GitBox
diff --git a/scala-package/deploy/pom.xml b/scala-package/deploy/pom.xml index c51aa9a9209..542bf6c1688 100644 --- a/scala-package/deploy/pom.xml +++ b/scala-package/deploy/pom.xml @@ -19,6 +19,7 @@ +${project.parent.basedir}/.. mxnet-full_2.11-${platform}-${flavor}

[incubator-mxnet] branch master updated: fix the fetching GPU problem (#13889)

2019-01-15 Thread lanking
This is an automated email from the ASF dual-hosted git repository. lanking 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 634d527 fix the fetching GPU problem

[GitHub] lanking520 merged pull request #13889: fix the GPU flavor problem for Scala

2019-01-15 Thread GitBox
lanking520 merged pull request #13889: fix the GPU flavor problem for Scala URL: https://github.com/apache/incubator-mxnet/pull/13889 As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing list, comm...@mxnet.apache.org

[GitHub] ptrendx opened a new pull request #13890: Improve bulking in Gluon

2019-01-15 Thread GitBox
ptrendx opened a new pull request #13890: Improve bulking in Gluon URL: https://github.com/apache/incubator-mxnet/pull/13890 ## Description ## This PR improves performance of hybridized Gluon models on the GPU by better bulking of ops (running GPU ops without synchronization). ##

[GitHub] DickJC123 commented on a change in pull request #13749: Add NHWC layout support to Pooling (cpu, gpu cuda, gpu cuDNN)

2019-01-15 Thread GitBox
DickJC123 commented on a change in pull request #13749: Add NHWC layout support to Pooling (cpu, gpu cuda, gpu cuDNN) URL: https://github.com/apache/incubator-mxnet/pull/13749#discussion_r248057077 ## File path: .gitmodules ## @@ -1,6 +1,6 @@ [submodule

[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.

2019-01-15 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new 3597c1e Bump the publish

[GitHub] larroy commented on a change in pull request #13311: [MXNET-703] Minor refactor of TensorRT code

2019-01-15 Thread GitBox
larroy commented on a change in pull request #13311: [MXNET-703] Minor refactor of TensorRT code URL: https://github.com/apache/incubator-mxnet/pull/13311#discussion_r248047439 ## File path: src/executor/trt_graph_executor.cc ## @@ -434,7 +435,7 @@ Executor

[GitHub] larroy commented on a change in pull request #13311: [MXNET-703] Minor refactor of TensorRT code

2019-01-15 Thread GitBox
larroy commented on a change in pull request #13311: [MXNET-703] Minor refactor of TensorRT code URL: https://github.com/apache/incubator-mxnet/pull/13311#discussion_r248045566 ## File path: src/executor/trt_graph_executor.cc ## @@ -188,7 +187,7 @@ void

[GitHub] piyushghai commented on a change in pull request #13807: [MXNET-1180] Java Image API

2019-01-15 Thread GitBox
piyushghai commented on a change in pull request #13807: [MXNET-1180] Java Image API URL: https://github.com/apache/incubator-mxnet/pull/13807#discussion_r248045486 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/Image.scala ## @@ -56,7 +56,7 @@ object

[GitHub] stu1130 commented on a change in pull request #13715: Add C++ Predictor class for inference

2019-01-15 Thread GitBox
stu1130 commented on a change in pull request #13715: Add C++ Predictor class for inference URL: https://github.com/apache/incubator-mxnet/pull/13715#discussion_r247764014 ## File path: cpp-package/example/cpp-predictor/predictor.h ## @@ -0,0 +1,228 @@ +/* + * Licensed to

[GitHub] stu1130 commented on a change in pull request #13715: Add C++ Predictor class for inference

2019-01-15 Thread GitBox
stu1130 commented on a change in pull request #13715: Add C++ Predictor class for inference URL: https://github.com/apache/incubator-mxnet/pull/13715#discussion_r247739340 ## File path: cpp-package/example/cpp-predictor/cpp_pred.cpp ## @@ -0,0 +1,83 @@ +/* + * Licensed to

[GitHub] stu1130 commented on a change in pull request #13715: Add C++ Predictor class for inference

2019-01-15 Thread GitBox
stu1130 commented on a change in pull request #13715: Add C++ Predictor class for inference URL: https://github.com/apache/incubator-mxnet/pull/13715#discussion_r247764297 ## File path: cpp-package/example/cpp-predictor/predictor.h ## @@ -0,0 +1,228 @@ +/* + * Licensed to

[GitHub] apeforest commented on a change in pull request #13749: Add NHWC layout support to Pooling (cpu, gpu cuda, gpu cuDNN)

2019-01-15 Thread GitBox
apeforest commented on a change in pull request #13749: Add NHWC layout support to Pooling (cpu, gpu cuda, gpu cuDNN) URL: https://github.com/apache/incubator-mxnet/pull/13749#discussion_r248038506 ## File path: .gitmodules ## @@ -1,6 +1,6 @@ [submodule

[GitHub] stu1130 commented on issue #13888: Reduce verbosity of container builds (wget output)

2019-01-15 Thread GitBox
stu1130 commented on issue #13888: Reduce verbosity of container builds (wget output) URL: https://github.com/apache/incubator-mxnet/pull/13888#issuecomment-454520044 @mxnet-label-bot add [pr-awaiting-review] Thanks for your contribution @larroy

[GitHub] larroy commented on issue #13842: Avoid adding SegfaultLogger if process already has sig handler.

2019-01-15 Thread GitBox
larroy commented on issue #13842: Avoid adding SegfaultLogger if process already has sig handler. URL: https://github.com/apache/incubator-mxnet/pull/13842#issuecomment-454520039 Good catch. So the JVM has a handler for segfault installed?

[GitHub] stu1130 commented on issue #13889: fix the GPU flavor problem for Scala

2019-01-15 Thread GitBox
stu1130 commented on issue #13889: fix the GPU flavor problem for Scala URL: https://github.com/apache/incubator-mxnet/pull/13889#issuecomment-454519843 @mxnet-label-bot add [pr-awaiting-review] This is an automated message

[incubator-mxnet] branch master updated (c4b4246 -> bc7ea31)

2019-01-15 Thread lanking
This is an automated email from the ASF dual-hosted git repository. lanking pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from c4b4246 Fix permissions of ci/docker/install/ubuntu_publish.sh (#13840) add bc7ea31 Avoid adding

[GitHub] lanking520 merged pull request #13842: Avoid adding SegfaultLogger if process already has sig handler.

2019-01-15 Thread GitBox
lanking520 merged pull request #13842: Avoid adding SegfaultLogger if process already has sig handler. URL: https://github.com/apache/incubator-mxnet/pull/13842 As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing list, comm...@mxnet.apache.org

[incubator-mxnet] Diff for: [GitHub] lanking520 merged pull request #13842: Avoid adding SegfaultLogger if process already has sig handler.

2019-01-15 Thread GitBox
diff --git a/src/initialize.cc b/src/initialize.cc index de7edd1b145..8d0e3c30421 100644 --- a/src/initialize.cc +++ b/src/initialize.cc @@ -44,7 +44,11 @@ class LibraryInitializer { LibraryInitializer() { dmlc::InitLogging("mxnet"); #if MXNET_USE_SIGNAL_HANDLER && DMLC_LOG_STACK_TRACE -

[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.

2019-01-15 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new 1e99e61 Bump the publish

[GitHub] marcoabreu commented on a change in pull request #13767: [MXNET-1191] Maven Jenkins Pipeline with Static Binary and GPU

2019-01-15 Thread GitBox
marcoabreu commented on a change in pull request #13767: [MXNET-1191] Maven Jenkins Pipeline with Static Binary and GPU URL: https://github.com/apache/incubator-mxnet/pull/13767#discussion_r248021281 ## File path: ci/docker/install/ubuntu_base.sh ## @@ -0,0 +1,40 @@

[GitHub] larroy commented on issue #9507: Segmentation Fault

2019-01-15 Thread GitBox
larroy commented on issue #9507: Segmentation Fault URL: https://github.com/apache/incubator-mxnet/issues/9507#issuecomment-454508473 https://github.com/apache/incubator-mxnet/pull/13886 This is an automated message from the

[GitHub] larroy commented on issue #13886: Fixes for wine detection tutorial

2019-01-15 Thread GitBox
larroy commented on issue #13886: Fixes for wine detection tutorial URL: https://github.com/apache/incubator-mxnet/pull/13886#issuecomment-454507969 https://github.com/apache/incubator-mxnet/issues/9094 This is an automated

[incubator-mxnet] branch master updated (4fe5461 -> c4b4246)

2019-01-15 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 4fe5461 fix ssd quantization script error (#13843) add c4b4246 Fix permissions of

[incubator-mxnet] Diff for: [GitHub] marcoabreu merged pull request #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh

2019-01-15 Thread GitBox
diff --git a/ci/docker/install/ubuntu_publish.sh b/ci/docker/install/ubuntu_publish.sh old mode 100644 new mode 100755 With regards, Apache Git Services

[GitHub] marcoabreu commented on issue #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh

2019-01-15 Thread GitBox
marcoabreu commented on issue #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh URL: https://github.com/apache/incubator-mxnet/pull/13840#issuecomment-454507512 Thanks a lot for the fix, Pedro! This

[GitHub] marcoabreu merged pull request #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh

2019-01-15 Thread GitBox
marcoabreu merged pull request #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh URL: https://github.com/apache/incubator-mxnet/pull/13840 As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing list,

[GitHub] lanking520 opened a new pull request #13889: fix the GPU flavor problem for Scala

2019-01-15 Thread GitBox
lanking520 opened a new pull request #13889: fix the GPU flavor problem for Scala URL: https://github.com/apache/incubator-mxnet/pull/13889 ## Description ## Currently the pom file configuration did not correctly find the `libcuda` name as well as path problems to fetch the

[GitHub] piyushghai commented on issue #13875: importing mxnet causing subprocess to crash

2019-01-15 Thread GitBox
piyushghai commented on issue #13875: importing mxnet causing subprocess to crash URL: https://github.com/apache/incubator-mxnet/issues/13875#issuecomment-454503199 @mxnet-label-bot update [Build, MKL] @azai91 @mseth10 Can we have a look at this crash ?

[GitHub] larroy opened a new pull request #13888: Reduce verbosity of container builds (wget output)

2019-01-15 Thread GitBox
larroy opened a new pull request #13888: Reduce verbosity of container builds (wget output) URL: https://github.com/apache/incubator-mxnet/pull/13888 ## Description ## minor: see title. ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for

[GitHub] larroy commented on issue #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh

2019-01-15 Thread GitBox
larroy commented on issue #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh URL: https://github.com/apache/incubator-mxnet/pull/13840#issuecomment-454500735 Since docker cache is not working well (This fixes it) we have docker image being rebuilt in PRs and

[GitHub] aaronmarkham edited a comment on issue #13831: OSError: [Errno 14] Bad address on website CI tests

2019-01-15 Thread GitBox
aaronmarkham edited a comment on issue #13831: OSError: [Errno 14] Bad address on website CI tests URL: https://github.com/apache/incubator-mxnet/issues/13831#issuecomment-454498017 I'm wondering if CI is running out of memory. Apparently that can cause this error. Reference:

[GitHub] aaronmarkham commented on issue #13831: OSError: [Errno 14] Bad address on website CI tests

2019-01-15 Thread GitBox
aaronmarkham commented on issue #13831: OSError: [Errno 14] Bad address on website CI tests URL: https://github.com/apache/incubator-mxnet/issues/13831#issuecomment-454498017 I'm wondering if CI is running out of memory. Apparently that can cause this error.

[GitHub] larroy commented on issue #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh

2019-01-15 Thread GitBox
larroy commented on issue #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh URL: https://github.com/apache/incubator-mxnet/pull/13840#issuecomment-454497840 Yes this is another issue, with failing download of clang tidy script. Both are valid.

[GitHub] stu1130 commented on issue #13887: Temporarily disable website builds

2019-01-15 Thread GitBox
stu1130 commented on issue #13887: Temporarily disable website builds URL: https://github.com/apache/incubator-mxnet/pull/13887#issuecomment-454494551 @mxnet-label-bot add [pr-awaiting-merge] Thanks for your contribution @KellenSunderland

[GitHub] stsukrov commented on a change in pull request #13549: Runtime feature detection

2019-01-15 Thread GitBox
stsukrov commented on a change in pull request #13549: Runtime feature detection URL: https://github.com/apache/incubator-mxnet/pull/13549#discussion_r247998714 ## File path: src/mxfeatures.cc ## @@ -0,0 +1,112 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

[GitHub] stsukrov commented on a change in pull request #13549: Runtime feature detection

2019-01-15 Thread GitBox
stsukrov commented on a change in pull request #13549: Runtime feature detection URL: https://github.com/apache/incubator-mxnet/pull/13549#discussion_r247998351 ## File path: python/mxnet/mxfeatures.py ## @@ -0,0 +1,103 @@ +# Licensed to the Apache Software Foundation

[GitHub] KellenSunderland edited a comment on issue #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh

2019-01-15 Thread GitBox
KellenSunderland edited a comment on issue #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh URL: https://github.com/apache/incubator-mxnet/pull/13840#issuecomment-454486877 @larroy Right now on master I see a problem saying:

[GitHub] KellenSunderland commented on issue #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh

2019-01-15 Thread GitBox
KellenSunderland commented on issue #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh URL: https://github.com/apache/incubator-mxnet/pull/13840#issuecomment-454486877 @larroy Right now on master I see a problem saying:

[GitHub] KellenSunderland commented on issue #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh

2019-01-15 Thread GitBox
KellenSunderland commented on issue #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh URL: https://github.com/apache/incubator-mxnet/pull/13840#issuecomment-454485760 Don't consider my comments blocking, I just didn't have enough context to merge. Merge away if

[GitHub] piyushghai commented on issue #13881: Cmake : could not find mkl (missing MKLDNN_LIBRARY)

2019-01-15 Thread GitBox
piyushghai commented on issue #13881: Cmake : could not find mkl (missing MKLDNN_LIBRARY) URL: https://github.com/apache/incubator-mxnet/issues/13881#issuecomment-454485506 Thank you for submitting the issue! I'm labeling it so the MXNet community members can help resolve it.

[GitHub] piyushghai commented on issue #13881: Cmake : could not find mkl (missing MKLDNN_LIBRARY)

2019-01-15 Thread GitBox
piyushghai commented on issue #13881: Cmake : could not find mkl (missing MKLDNN_LIBRARY) URL: https://github.com/apache/incubator-mxnet/issues/13881#issuecomment-454485651 @mxnet-label-bot update [MKL, MKLDNN] This is an

[GitHub] piyushghai commented on issue #13885: imagenet example failure to run properly

2019-01-15 Thread GitBox
piyushghai commented on issue #13885: imagenet example failure to run properly URL: https://github.com/apache/incubator-mxnet/issues/13885#issuecomment-454484747 Thank you for submitting the issue! I'm labeling it so the MXNet community members can help resolve it. @mxnet-label-bot add

[GitHub] KellenSunderland commented on issue #13887: Temporarily disable website builds

2019-01-15 Thread GitBox
KellenSunderland commented on issue #13887: Temporarily disable website builds URL: https://github.com/apache/incubator-mxnet/pull/13887#issuecomment-454484136 @larroy I think the trick is this PR just disables the flaky section. It still will report back a success to Jenkins after a

[GitHub] piyushghai commented on issue #13883: Raspberrypi 3 Failed to build docker image

2019-01-15 Thread GitBox
piyushghai commented on issue #13883: Raspberrypi 3 Failed to build docker image URL: https://github.com/apache/incubator-mxnet/issues/13883#issuecomment-454483725 @mxnet-label-bot add [Edge devices, Docker] This is an

[GitHub] larroy commented on issue #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh

2019-01-15 Thread GitBox
larroy commented on issue #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh URL: https://github.com/apache/incubator-mxnet/pull/13840#issuecomment-454482627 ubuntu 14 docker image is failing across the board due to the permissions in this script.

<    1   2   3   >