[GitHub] larroy commented on issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler

2018-06-13 Thread GitBox
larroy commented on issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler URL: https://github.com/apache/incubator-mxnet/issues/11249#issuecomment-397084307 @nswamy wouldn't these kind of heavy tests would be more suitable for nightly?

[GitHub] larroy commented on issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler

2018-06-13 Thread GitBox
larroy commented on issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler URL: https://github.com/apache/incubator-mxnet/issues/11249#issuecomment-397084126

[GitHub] marcoabreu commented on issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler

2018-06-13 Thread GitBox
marcoabreu commented on issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler URL: https://github.com/apache/incubator-mxnet/issues/11249#issuecomment-397080903 I don't think we assign any limits to Docker. But why does a unittest

[GitHub] lanking520 commented on issue #11264: [MXNET-543] disable scalatest on Spark

2018-06-13 Thread GitBox
lanking520 commented on issue #11264: [MXNET-543] disable scalatest on Spark URL: https://github.com/apache/incubator-mxnet/pull/11264#issuecomment-397072489 @CodingCat It also breaks scalatest on Mac, may not be only the memory issues

[GitHub] CodingCat commented on issue #11264: [MXNET-543] disable scalatest on Spark

2018-06-13 Thread GitBox
CodingCat commented on issue #11264: [MXNET-543] disable scalatest on Spark URL: https://github.com/apache/incubator-mxnet/pull/11264#issuecomment-397072732 it happens consistently in mac? This is an automated message from

[GitHub] nswamy commented on issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler

2018-06-13 Thread GitBox
nswamy commented on issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler URL: https://github.com/apache/incubator-mxnet/issues/11249#issuecomment-397066182 This is due to the Spark tests that were newly merged

[GitHub] anirudh2290 commented on issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler

2018-06-13 Thread GitBox
anirudh2290 commented on issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler URL: https://github.com/apache/incubator-mxnet/issues/11249#issuecomment-397065586 @nswamy havent seen this yet on 1.2 but need to merge PRs on master before

[GitHub] nswamy commented on issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler

2018-06-13 Thread GitBox
nswamy commented on issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler URL: https://github.com/apache/incubator-mxnet/issues/11249#issuecomment-397064830 I see this on the master branch, is it braking on the 1.2.0 branch as well?

[GitHub] rahul003 commented on issue #10391: [MXNET-139] Tutorial for mixed precision training with float16

2018-06-13 Thread GitBox
rahul003 commented on issue #10391: [MXNET-139] Tutorial for mixed precision training with float16 URL: https://github.com/apache/incubator-mxnet/pull/10391#issuecomment-397061801 @ Reviewers, please check the tutorial now. I think we can merge it and keep updating it if you have

[GitHub] rahul003 commented on issue #10391: [MXNET-139] Tutorial for mixed precision training with float16

2018-06-13 Thread GitBox
rahul003 commented on issue #10391: [MXNET-139] Tutorial for mixed precision training with float16 URL: https://github.com/apache/incubator-mxnet/pull/10391#issuecomment-397061801 @reviewers, please check the tutorial now. I think we can merge it and keep updating it if you have

[GitHub] lanking520 commented on issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler

2018-06-13 Thread GitBox
lanking520 commented on issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler URL: https://github.com/apache/incubator-mxnet/issues/11249#issuecomment-397061511 I broke my Mac's Test, currently working on this...

[GitHub] eric-haibin-lin commented on a change in pull request #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds.

2018-06-13 Thread GitBox
eric-haibin-lin commented on a change in pull request #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds. URL: https://github.com/apache/incubator-mxnet/pull/11245#discussion_r195210060 ## File path: tests/python/unittest/test_ndarray.py ## @@ -711,9

[GitHub] anirudh2290 commented on issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler

2018-06-13 Thread GitBox
anirudh2290 commented on issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler URL: https://github.com/apache/incubator-mxnet/issues/11249#issuecomment-397054461 Happens here too:

[GitHub] anirudh2290 opened a new issue #11263: Memory corruption issue with Scala test

2018-06-13 Thread GitBox
anirudh2290 opened a new issue #11263: Memory corruption issue with Scala test URL: https://github.com/apache/incubator-mxnet/issues/11263 ``` *** Error in `java': double free or corruption (fasttop): 0x7f91e0db8270 *** === Backtrace: =

[GitHub] anirudh2290 commented on issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler

2018-06-13 Thread GitBox
anirudh2290 commented on issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler URL: https://github.com/apache/incubator-mxnet/issues/11249#issuecomment-397054461 Happens here too:

[GitHub] zhreshold commented on issue #11258: [MXNET-540] update python gpu build from source instructions on windows

2018-06-13 Thread GitBox
zhreshold commented on issue #11258: [MXNET-540] update python gpu build from source instructions on windows URL: https://github.com/apache/incubator-mxnet/pull/11258#issuecomment-397049564 please rebase to master since CI is down

[GitHub] zhreshold commented on a change in pull request #10911: Fix engine stop/start

2018-06-13 Thread GitBox
zhreshold commented on a change in pull request #10911: Fix engine stop/start URL: https://github.com/apache/incubator-mxnet/pull/10911#discussion_r195199407 ## File path: src/engine/threaded_engine_pooled.cc ## @@ -42,14 +43,38 @@ namespace engine { */ class

[GitHub] marcoabreu commented on issue #5677: Constant Initializer only works on keys ending with weight

2018-06-13 Thread GitBox
marcoabreu commented on issue #5677: Constant Initializer only works on keys ending with weight URL: https://github.com/apache/incubator-mxnet/issues/5677#issuecomment-397048244 Why should it be closed @kalyc ? This is an

[GitHub] anirudhacharya commented on issue #10889: [MXNET-382] Shape and Size Operator

2018-06-13 Thread GitBox
anirudhacharya commented on issue #10889: [MXNET-382] Shape and Size Operator URL: https://github.com/apache/incubator-mxnet/pull/10889#issuecomment-397020847 @reminisce @piiswrong ping for review. This is an automated

[GitHub] kalyc commented on issue #4159: How to bind 3 inputs using mxnet.io.NDArrayIter?

2018-06-13 Thread GitBox
kalyc commented on issue #4159: How to bind 3 inputs using mxnet.io.NDArrayIter? URL: https://github.com/apache/incubator-mxnet/issues/4159#issuecomment-397047093 Hey @anjishnu thanks for submitting the issue. Were you able to resolve it?

[GitHub] anirudh2290 commented on a change in pull request #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds.

2018-06-13 Thread GitBox
anirudh2290 commented on a change in pull request #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds. URL: https://github.com/apache/incubator-mxnet/pull/11245#discussion_r195197561 ## File path: tests/python/unittest/test_ndarray.py ## @@ -711,9

[GitHub] kalyc commented on issue #5677: Constant Initializer only works on keys ending with weight

2018-06-13 Thread GitBox
kalyc commented on issue #5677: Constant Initializer only works on keys ending with weight URL: https://github.com/apache/incubator-mxnet/issues/5677#issuecomment-397046724 Hello @jmacglashan - Thanks for submitting this issue. Can this be closed?

[GitHub] ThomasDelteil commented on issue #11243: weird gpu memory usage

2018-06-13 Thread GitBox
ThomasDelteil commented on issue #11243: weird gpu memory usage URL: https://github.com/apache/incubator-mxnet/issues/11243#issuecomment-397046550 @dwSun MXNet is built using asynchronous operations. When you load data, run a forward pass or backward pass, the operations are enqueued on

[GitHub] ThomasDelteil commented on issue #11243: weird gpu memory usage

2018-06-13 Thread GitBox
ThomasDelteil commented on issue #11243: weird gpu memory usage URL: https://github.com/apache/incubator-mxnet/issues/11243#issuecomment-397046550 @dwSun MXNet is built using asynchronous operations. When you load data, run a forward pass or backward pass, the operations are enqueued on

[GitHub] zhreshold commented on issue #6257: Merge AMD-HIP port

2018-06-13 Thread GitBox
zhreshold commented on issue #6257: Merge AMD-HIP port URL: https://github.com/apache/incubator-mxnet/issues/6257#issuecomment-397046110 This change will break all legacy build systems and documents with USE_CUDA=1 enabled. I suggest keep USE_CUDA and add USE_GPU as a dispatcher to turn

[GitHub] kpmurali commented on a change in pull request #11261: [MXNET-541] Correcting the install command for version 0.11.0

2018-06-13 Thread GitBox
kpmurali commented on a change in pull request #11261: [MXNET-541] Correcting the install command for version 0.11.0 URL: https://github.com/apache/incubator-mxnet/pull/11261#discussion_r195195799 ## File path: docs/install/index.md ## @@ -635,9 +659,9 @@ pip install

[GitHub] azai91 commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators

2018-06-13 Thread GitBox
azai91 commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators URL: https://github.com/apache/incubator-mxnet/pull/11129#discussion_r195193780 ## File path: src/common/exec_utils.h ## @@ -98,10 +98,13 @@ inline bool

[GitHub] zhreshold commented on issue #11200: Training Imagenet on AlexNet failed with corrupted JPEG format images

2018-06-13 Thread GitBox
zhreshold commented on issue #11200: Training Imagenet on AlexNet failed with corrupted JPEG format images URL: https://github.com/apache/incubator-mxnet/issues/11200#issuecomment-397043235 @absalama What's the arguments you used with im2rec.py? If you don't specify `--pass-through`, it

[GitHub] ThomasDelteil commented on issue #11091: Symbolic .json file not compatible with .params file generated since MXNet 1.2

2018-06-13 Thread GitBox
ThomasDelteil commented on issue #11091: Symbolic .json file not compatible with .params file generated since MXNet 1.2 URL: https://github.com/apache/incubator-mxnet/issues/11091#issuecomment-397036787 A more in-depth analysis written by @piiswrong about the cause of the issue and

[GitHub] ThomasDelteil commented on issue #11091: Symbolic .json file not compatible with .params file generated since MXNet 1.2

2018-06-13 Thread GitBox
ThomasDelteil commented on issue #11091: Symbolic .json file not compatible with .params file generated since MXNet 1.2 URL: https://github.com/apache/incubator-mxnet/issues/11091#issuecomment-397036787 A more in-depth analysis written by @piiswrong about the cause of the issue and

[GitHub] azai91 commented on issue #11129: [MXNET-497]Test kAddTo request for mkldnn operators

2018-06-13 Thread GitBox
azai91 commented on issue #11129: [MXNET-497]Test kAddTo request for mkldnn operators URL: https://github.com/apache/incubator-mxnet/pull/11129#issuecomment-397033017 Address regression issue in this diff #11262 This is an

[GitHub] ThomasDelteil commented on a change in pull request #11210: [MXNET-532] Clarify documentation of save_params(), load_params(), export()

2018-06-13 Thread GitBox
ThomasDelteil commented on a change in pull request #11210: [MXNET-532] Clarify documentation of save_params(), load_params(), export() URL: https://github.com/apache/incubator-mxnet/pull/11210#discussion_r195183961 ## File path: python/mxnet/gluon/block.py ## @@ -309,6

[GitHub] kpmurali opened a new pull request #11261: [MXNET-541] Correcting the install command for version 0.11.0

2018-06-13 Thread GitBox
kpmurali opened a new pull request #11261: [MXNET-541] Correcting the install command for version 0.11.0 URL: https://github.com/apache/incubator-mxnet/pull/11261 ## Description ## Correcting the install command for version 0.11.0 Install command based on

[GitHub] lanking520 commented on issue #11149: Unreasonable performance of resnext models provided in model_zoo, evaluated by score.py

2018-06-13 Thread GitBox
lanking520 commented on issue #11149: Unreasonable performance of resnext models provided in model_zoo, evaluated by score.py URL: https://github.com/apache/incubator-mxnet/issues/11149#issuecomment-397028840 @HuichuanLiu Many thanks for your hard work. I have checked with MXNet Gluon

[GitHub] ThomasDelteil commented on a change in pull request #11254: add blocklist

2018-06-13 Thread GitBox
ThomasDelteil commented on a change in pull request #11254: add blocklist URL: https://github.com/apache/incubator-mxnet/pull/11254#discussion_r195174456 ## File path: python/mxnet/gluon/block.py ## @@ -841,6 +842,140 @@ def hybrid_forward(self, F, x, *args, **kwargs):

[incubator-mxnet] branch master updated (fc102c3 -> c54f9f7)

2018-06-13 Thread indhub
This is an automated email from the ASF dual-hosted git repository. indhub pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from fc102c3 Fix build.py when CCACHE_DIR is set. (#11226) add c54f9f7 [MXNET-534] Add examples to

[GitHub] indhub closed pull request #11225: [MXNET-534] Add examples to example list

2018-06-13 Thread GitBox
indhub closed pull request #11225: [MXNET-534] Add examples to example list URL: https://github.com/apache/incubator-mxnet/pull/11225 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 is a

[GitHub] anirudhacharya commented on issue #10889: [MXNET-382] Shape and Size Operator

2018-06-13 Thread GitBox
anirudhacharya commented on issue #10889: [MXNET-382] Shape and Size Operator URL: https://github.com/apache/incubator-mxnet/pull/10889#issuecomment-397020847 @reminisce ping for review. This is an automated message from the

[GitHub] eric-haibin-lin commented on issue #11223: Allow specifying AdaGrad initial accumulator value

2018-06-13 Thread GitBox
eric-haibin-lin commented on issue #11223: Allow specifying AdaGrad initial accumulator value URL: https://github.com/apache/incubator-mxnet/pull/11223#issuecomment-397018537 I'm not sure if this is necessary. Can the user just specify `eps`?? I agree eps is not a good name but it has the

[GitHub] eric-haibin-lin commented on issue #11255: P100 + MXNet is slow

2018-06-13 Thread GitBox
eric-haibin-lin commented on issue #11255: P100 + MXNet is slow URL: https://github.com/apache/incubator-mxnet/issues/11255#issuecomment-397014896 did you use multiple worker for data loader? This is an automated message

[GitHub] larroy commented on issue #9853: Flaky test_operator.test_binary_op

2018-06-13 Thread GitBox
larroy commented on issue #9853: Flaky test_operator.test_binary_op URL: https://github.com/apache/incubator-mxnet/issues/9853#issuecomment-397011088 1060292419 This is an automated message from the Apache Git Service. To

[GitHub] larroy commented on issue #9853: Flaky test_operator.test_binary_op

2018-06-13 Thread GitBox
larroy commented on issue #9853: Flaky test_operator.test_binary_op URL: https://github.com/apache/incubator-mxnet/issues/9853#issuecomment-397011088 seed 1060292419 This is an automated message from the Apache Git Service.

[GitHub] lanking520 commented on a change in pull request #11152: updating Scala IntelliJ tutorial & installation instructions [WIP]

2018-06-13 Thread GitBox
lanking520 commented on a change in pull request #11152: updating Scala IntelliJ tutorial & installation instructions [WIP] URL: https://github.com/apache/incubator-mxnet/pull/11152#discussion_r195157557 ## File path: docs/tutorials/scala/README.md ## @@ -1,6 +1,34 @@ #

[GitHub] zheng-da commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators

2018-06-13 Thread GitBox
zheng-da commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators URL: https://github.com/apache/incubator-mxnet/pull/11129#discussion_r195148473 ## File path: src/common/exec_utils.h ## @@ -98,10 +98,13 @@ inline bool

[GitHub] zheng-da commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators

2018-06-13 Thread GitBox
zheng-da commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators URL: https://github.com/apache/incubator-mxnet/pull/11129#discussion_r195152303 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -128,12 +201,18 @@ void

[GitHub] zheng-da commented on issue #11129: [MXNET-497]Test kAddTo request for mkldnn operators

2018-06-13 Thread GitBox
zheng-da commented on issue #11129: [MXNET-497]Test kAddTo request for mkldnn operators URL: https://github.com/apache/incubator-mxnet/pull/11129#issuecomment-396994511 @pengzhao-intel I think we should fix bugs when adding tests. We can change the title of this PR to something like "fix

[GitHub] larroy commented on issue #11259: [MXNET-184] Fix flaky test test_operator.test_binary_op due to numerical errors

2018-06-13 Thread GitBox
larroy commented on issue #11259: [MXNET-184] Fix flaky test test_operator.test_binary_op due to numerical errors URL: https://github.com/apache/incubator-mxnet/pull/11259#issuecomment-396991029 SA https://github.com/apache/incubator-mxnet/pull/11260

[GitHub] larroy commented on issue #11259: [MXNET-184] Fix flaky test test_operator.test_binary_op due to numerical errors

2018-06-13 Thread GitBox
larroy commented on issue #11259: [MXNET-184] Fix flaky test test_operator.test_binary_op due to numerical errors URL: https://github.com/apache/incubator-mxnet/pull/11259#issuecomment-396988910 Now they are much closer, still not equal.

[GitHub] larroy commented on issue #11259: [MXNET-184] Fix flaky test test_operator.test_binary_op due to numerical errors

2018-06-13 Thread GitBox
larroy commented on issue #11259: [MXNET-184] Fix flaky test test_operator.test_binary_op due to numerical errors URL: https://github.com/apache/incubator-mxnet/pull/11259#issuecomment-396982166 There must be a float32 conversion somewhere in the engine: Isolated the failing case:

[GitHub] larroy commented on issue #11259: [MXNET-184] Fix flaky test test_operator.test_binary_op due to numerical errors

2018-06-13 Thread GitBox
larroy commented on issue #11259: [MXNET-184] Fix flaky test test_operator.test_binary_op due to numerical errors URL: https://github.com/apache/incubator-mxnet/pull/11259#issuecomment-396977710 Still has issues with MXNET_TEST_SEED=1768433044

[GitHub] larroy opened a new pull request #11260: [WIP] flaky 9853 bis

2018-06-13 Thread GitBox
larroy opened a new pull request #11260: [WIP] flaky 9853 bis URL: https://github.com/apache/incubator-mxnet/pull/11260 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your

[GitHub] sriharikarnam commented on issue #6257: Merge AMD-HIP port

2018-06-13 Thread GitBox
sriharikarnam commented on issue #6257: Merge AMD-HIP port URL: https://github.com/apache/incubator-mxnet/issues/6257#issuecomment-396970488 We are in-progress of disscusiion with sub module owners. We will be ready with HIP port of MXNet, which supports both CUDA and HIP path Need

[GitHub] marcoabreu commented on issue #11200: Training Imagenet on AlexNet failed with corrupted JPEG format images

2018-06-13 Thread GitBox
marcoabreu commented on issue #11200: Training Imagenet on AlexNet failed with corrupted JPEG format images URL: https://github.com/apache/incubator-mxnet/issues/11200#issuecomment-396966558 @szha @eric-haibin-lin could you help me out here please?

[GitHub] szha commented on a change in pull request #11254: add blocklist

2018-06-13 Thread GitBox
szha commented on a change in pull request #11254: add blocklist URL: https://github.com/apache/incubator-mxnet/pull/11254#discussion_r195114422 ## File path: docs/api/python/gluon/gluon.md ## @@ -116,6 +116,8 @@ net.hybridize() Block HybridBlock +BlockList

[GitHub] marcoabreu commented on issue #11247: add seed_aug parameter for ImageRecordItr to fix random seed for augm…

2018-06-13 Thread GitBox
marcoabreu commented on issue #11247: add seed_aug parameter for ImageRecordItr to fix random seed for augm… URL: https://github.com/apache/incubator-mxnet/pull/11247#issuecomment-396966250 Hi, our tests are in https://github.com/apache/incubator-mxnet/tree/master/tests/cpp Yes,

[GitHub] BenLag2906 commented on issue #10529: Pb between c++ and python

2018-06-13 Thread GitBox
BenLag2906 commented on issue #10529: Pb between c++ and python URL: https://github.com/apache/incubator-mxnet/issues/10529#issuecomment-396965220 To add more explanation, i try to translate kohillyang code in c++, but many difficulties persist.

[GitHub] marcoabreu commented on a change in pull request #11199: [MXNET-538] Add XUnit to python tests

2018-06-13 Thread GitBox
marcoabreu commented on a change in pull request #11199: [MXNET-538] Add XUnit to python tests URL: https://github.com/apache/incubator-mxnet/pull/11199#discussion_r195113206 ## File path: Jenkinsfile ## @@ -689,16 +764,21 @@ try { node('mxnetwindows-cpu') {

[GitHub] chinakook commented on issue #11254: add blocklist

2018-06-13 Thread GitBox
chinakook commented on issue #11254: add blocklist URL: https://github.com/apache/incubator-mxnet/pull/11254#issuecomment-396965169 BlockList is necessary. This is an automated message from the Apache Git Service. To respond

[incubator-mxnet] branch master updated (74479b8 -> fc102c3)

2018-06-13 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 74479b8 [MXNET-386] ongoing maintenance on NDArray (#11126) add fc102c3 Fix build.py when

[GitHub] marcoabreu closed pull request #11226: Fix build.py in docker for osx

2018-06-13 Thread GitBox
marcoabreu closed pull request #11226: Fix build.py in docker for osx URL: https://github.com/apache/incubator-mxnet/pull/11226 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 is a foreign

[GitHub] larroy opened a new pull request #11259: Fix flaky test test_operator.test_binary_op due to numerical errors

2018-06-13 Thread GitBox
larroy opened a new pull request #11259: Fix flaky test test_operator.test_binary_op due to numerical errors URL: https://github.com/apache/incubator-mxnet/pull/11259 Use float64 computations as the reference numpy implementation operates in double and not float. f64(f32(f64(.))) %

[GitHub] larroy commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests.

2018-06-13 Thread GitBox
larroy commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests. URL: https://github.com/apache/incubator-mxnet/pull/10827#discussion_r195084977 ## File path: tests/jenkins/run_test_installation_docs.sh ##

[GitHub] yjcn commented on issue #11242: Same model with c_predict_api gets an incorrect result but it is right in python.

2018-06-13 Thread GitBox
yjcn commented on issue #11242: Same model with c_predict_api gets an incorrect result but it is right in python. URL: https://github.com/apache/incubator-mxnet/issues/11242#issuecomment-396913910 @kalyc Thanks for your reply. This is python ssd 's result.

[GitHub] absalama commented on issue #11200: Training Imagenet on AlexNet failed with corrupted JPEG format images

2018-06-13 Thread GitBox
absalama commented on issue #11200: Training Imagenet on AlexNet failed with corrupted JPEG format images URL: https://github.com/apache/incubator-mxnet/issues/11200#issuecomment-396905607 First, does im2rec tool delete corrupted images or mcyk format images when it transfers the raw

[GitHub] wenyangchu commented on issue #11247: add seed_aug parameter for ImageRecordItr to fix random seed for augm…

2018-06-13 Thread GitBox
wenyangchu commented on issue #11247: add seed_aug parameter for ImageRecordItr to fix random seed for augm… URL: https://github.com/apache/incubator-mxnet/pull/11247#issuecomment-396900736 Hi @marcoabreu or any others, another question is that, I have a test failure in python3 cpu win

[GitHub] wenyangchu commented on issue #11247: add seed_aug parameter for ImageRecordItr to fix random seed for augm…

2018-06-13 Thread GitBox
wenyangchu commented on issue #11247: add seed_aug parameter for ImageRecordItr to fix random seed for augm… URL: https://github.com/apache/incubator-mxnet/pull/11247#issuecomment-396900736 Hi @marcoabreu, another question is that, I have a test failure in python3 cpu win and python3

[GitHub] yasserzamani commented on issue #63: update python gpu build from source instructions on windows

2018-06-13 Thread GitBox
yasserzamani commented on issue #63: update python gpu build from source instructions on windows URL: https://github.com/apache/incubator-mxnet-site/pull/63#issuecomment-396889133 Thanks for the clarification :+1: I sent patch there.

[GitHub] yasserzamani closed pull request #63: update python gpu build from source instructions on windows

2018-06-13 Thread GitBox
yasserzamani closed pull request #63: update python gpu build from source instructions on windows URL: https://github.com/apache/incubator-mxnet-site/pull/63 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

[GitHub] yasserzamani opened a new pull request #11258: [MXNET-540] update python gpu build from source instructions on windows

2018-06-13 Thread GitBox
yasserzamani opened a new pull request #11258: [MXNET-540] update python gpu build from source instructions on windows URL: https://github.com/apache/incubator-mxnet/pull/11258 ## Description ## Recently I built MXNet from source on windows (gpu, python). During the process I

[GitHub] larroy commented on issue #9853: Flaky test_operator.test_binary_op

2018-06-13 Thread GitBox
larroy commented on issue #9853: Flaky test_operator.test_binary_op URL: https://github.com/apache/incubator-mxnet/issues/9853#issuecomment-396886126 I have likely found the root cause of this problem, just so we don't duplicate resources on this one.

[GitHub] lebeg commented on a change in pull request #10911: Fix engine stop/start

2018-06-13 Thread GitBox
lebeg commented on a change in pull request #10911: Fix engine stop/start URL: https://github.com/apache/incubator-mxnet/pull/10911#discussion_r195025583 ## File path: src/engine/threaded_engine_pooled.cc ## @@ -42,14 +43,38 @@ namespace engine { */ class

[GitHub] larroy commented on a change in pull request #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds.

2018-06-13 Thread GitBox
larroy commented on a change in pull request #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds. URL: https://github.com/apache/incubator-mxnet/pull/11245#discussion_r194947843 ## File path: tests/python/unittest/test_ndarray.py ## @@ -711,9 +711,8 @@

[GitHub] wenyangchu commented on issue #11247: add seed_aug parameter for ImageRecordItr to fix random seed for augm…

2018-06-13 Thread GitBox
wenyangchu commented on issue #11247: add seed_aug parameter for ImageRecordItr to fix random seed for augm… URL: https://github.com/apache/incubator-mxnet/pull/11247#issuecomment-396871695 Hi @marcoabreu , sure, I will add a test for it. Do you know where the existing tests for this

[GitHub] I3orn2FLY commented on issue #10778: Make android error

2018-06-13 Thread GitBox
I3orn2FLY commented on issue #10778: Make android error URL: https://github.com/apache/incubator-mxnet/issues/10778#issuecomment-396868304 Just to be clear, I use following configurations: NDK version => 15.2.4 TARGET => arm64/ARMV8 ``` export CC=aarch64-linux-android-gcc

[GitHub] lebeg commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests.

2018-06-13 Thread GitBox
lebeg commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests. URL: https://github.com/apache/incubator-mxnet/pull/10827#discussion_r194650363 ## File path: tests/nightly/Jenkinsfile ## @@ -0,0 +1,180 @@

[GitHub] lebeg commented on a change in pull request #11199: [MXNET-538] Add XUnit to python tests

2018-06-13 Thread GitBox
lebeg commented on a change in pull request #11199: [MXNET-538] Add XUnit to python tests URL: https://github.com/apache/incubator-mxnet/pull/11199#discussion_r195003127 ## File path: Jenkinsfile ## @@ -689,16 +764,21 @@ try { node('mxnetwindows-cpu') {

[GitHub] lebeg commented on issue #11257: Docker cache is not being used

2018-06-13 Thread GitBox
lebeg commented on issue #11257: Docker cache is not being used URL: https://github.com/apache/incubator-mxnet/issues/11257#issuecomment-396857874 I don't have permissions to add labels, but I believe that this should be tagged with the `CI` label.

[GitHub] lebeg opened a new issue #11257: Docker cache is not being used

2018-06-13 Thread GitBox
lebeg opened a new issue #11257: Docker cache is not being used URL: https://github.com/apache/incubator-mxnet/issues/11257 As can be seen for all [latest builds](http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/PR-11252/1/pipeline/70): docker cache

[GitHub] larroy commented on a change in pull request #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds.

2018-06-13 Thread GitBox
larroy commented on a change in pull request #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds. URL: https://github.com/apache/incubator-mxnet/pull/11245#discussion_r194980306 ## File path: tests/python/unittest/test_ndarray.py ## @@ -711,9 +711,8 @@

[GitHub] larroy commented on issue #11226: Fix build.py in docker for osx

2018-06-13 Thread GitBox
larroy commented on issue #11226: Fix build.py in docker for osx URL: https://github.com/apache/incubator-mxnet/pull/11226#issuecomment-396855520 @marcoabreu @lebeg ammended the PR. We will fix --cache-from issue in a different PR. Please merge this changes as it allows ci/build.py to

[GitHub] larroy commented on issue #11226: Fix build.py when CCACHE_DIR is set.

2018-06-13 Thread GitBox
larroy commented on issue #11226: Fix build.py when CCACHE_DIR is set. URL: https://github.com/apache/incubator-mxnet/pull/11226#issuecomment-396854754 Don't worry about this anymore, @lebeg has helped me to understand what's the issue. We will send a fix to master since this

[GitHub] larroy commented on a change in pull request #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds.

2018-06-13 Thread GitBox
larroy commented on a change in pull request #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds. URL: https://github.com/apache/incubator-mxnet/pull/11245#discussion_r194971684 ## File path: tests/python/unittest/test_ndarray.py ## @@ -711,9 +711,8 @@

[GitHub] larroy commented on a change in pull request #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds.

2018-06-13 Thread GitBox
larroy commented on a change in pull request #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds. URL: https://github.com/apache/incubator-mxnet/pull/11245#discussion_r194980306 ## File path: tests/python/unittest/test_ndarray.py ## @@ -711,9 +711,8 @@

[GitHub] larroy commented on issue #11226: Fix build.py when CCACHE_DIR is set.

2018-06-13 Thread GitBox
larroy commented on issue #11226: Fix build.py when CCACHE_DIR is set. URL: https://github.com/apache/incubator-mxnet/pull/11226#issuecomment-396835624 When I try local builds with ci/build.py -p armv7 . in commit b4eca7c223 from https://github.com/larroy/mxnet/tree/devel-arm-fixed it

[GitHub] larroy commented on issue #11226: Fix build.py when CCACHE_DIR is set.

2018-06-13 Thread GitBox
larroy commented on issue #11226: Fix build.py when CCACHE_DIR is set. URL: https://github.com/apache/incubator-mxnet/pull/11226#issuecomment-396835624 When I try local builds with ci/build.py -p armv7 . in commit b4eca7c223 it rebuilds the docker cache again an again. Is it because of

[GitHub] larroy commented on issue #11226: Fix build.py when CCACHE_DIR is set.

2018-06-13 Thread GitBox
larroy commented on issue #11226: Fix build.py when CCACHE_DIR is set. URL: https://github.com/apache/incubator-mxnet/pull/11226#issuecomment-396835624 When I try local builds with ci/build.py -p armv7 . in commit b4eca7c223 it rebuilds the docker cache again an again. Is it because of

[GitHub] yzhliu opened a new pull request #11256: [MXNET-539] Allow Scala users to specify data/label names for NDArrayIter

2018-06-13 Thread GitBox
yzhliu opened a new pull request #11256: [MXNET-539] Allow Scala users to specify data/label names for NDArrayIter URL: https://github.com/apache/incubator-mxnet/pull/11256 ## Description ## So far Scala users cannot specify the names for NDArrayIter - they are inferred

[GitHub] larroy commented on a change in pull request #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds.

2018-06-13 Thread GitBox
larroy commented on a change in pull request #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds. URL: https://github.com/apache/incubator-mxnet/pull/11245#discussion_r194971684 ## File path: tests/python/unittest/test_ndarray.py ## @@ -711,9 +711,8 @@

[GitHub] larroy commented on a change in pull request #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds.

2018-06-13 Thread GitBox
larroy commented on a change in pull request #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds. URL: https://github.com/apache/incubator-mxnet/pull/11245#discussion_r194971684 ## File path: tests/python/unittest/test_ndarray.py ## @@ -711,9 +711,8 @@

[GitHub] larroy commented on issue #11055: [MXNET-57] Added support android64

2018-06-13 Thread GitBox
larroy commented on issue #11055: [MXNET-57] Added support android64 URL: https://github.com/apache/incubator-mxnet/pull/11055#issuecomment-396831592 CI is giving problems when downloading CUDA?

[GitHub] larroy commented on issue #11226: Fix build.py when CCACHE_DIR is set.

2018-06-13 Thread GitBox
larroy commented on issue #11226: Fix build.py when CCACHE_DIR is set. URL: https://github.com/apache/incubator-mxnet/pull/11226#issuecomment-396831137 Thanks. I don't know much about docker, so probably you are right. I will keep observing and see if my observations match your

[GitHub] lupesko commented on issue #11023: No Windows Support for Scala

2018-06-13 Thread GitBox
lupesko commented on issue #11023: No Windows Support for Scala URL: https://github.com/apache/incubator-mxnet/issues/11023#issuecomment-396829527 @sandeep-krishnamurthy can you please label Scala and Windows? Thanks! This

[GitHub] liuzx32 commented on issue #11093: Some wrong with mxnet on spark: params.jars = jars.split(", |:")

2018-06-13 Thread GitBox
liuzx32 commented on issue #11093: Some wrong with mxnet on spark: params.jars = jars.split(",|:") URL: https://github.com/apache/incubator-mxnet/issues/11093#issuecomment-396827208 @lanking520 Perhaps the problem is here: `jars.map(jar => SparkFiles.get(new

[GitHub] liuzx32 commented on issue #11093: Some wrong with mxnet on spark: params.jars = jars.split(", |:")

2018-06-13 Thread GitBox
liuzx32 commented on issue #11093: Some wrong with mxnet on spark: params.jars = jars.split(",|:") URL: https://github.com/apache/incubator-mxnet/issues/11093#issuecomment-396825670 @yzhliu @lanking520 Mxnet on spark should be for yarn cluster mode not only yarn local mode. If every

[GitHub] marcoabreu commented on a change in pull request #11055: [MXNET-57] Added support android64

2018-06-13 Thread GitBox
marcoabreu commented on a change in pull request #11055: [MXNET-57] Added support android64 URL: https://github.com/apache/incubator-mxnet/pull/11055#discussion_r194963868 ## File path: ci/docker/runtime_functions.sh ## @@ -163,6 +163,30 @@ build_android_arm64() {

[GitHub] marcoabreu commented on a change in pull request #11055: [MXNET-57] Added support android64

2018-06-13 Thread GitBox
marcoabreu commented on a change in pull request #11055: [MXNET-57] Added support android64 URL: https://github.com/apache/incubator-mxnet/pull/11055#discussion_r194963868 ## File path: ci/docker/runtime_functions.sh ## @@ -163,6 +163,30 @@ build_android_arm64() {

<    1   2