[GitHub] marcoabreu commented on a change in pull request #11630: [WIP] Fix flaky test test_deconvolution

2018-07-11 Thread GitBox
marcoabreu commented on a change in pull request #11630: [WIP] Fix flaky test test_deconvolution URL: https://github.com/apache/incubator-mxnet/pull/11630#discussion_r201789608 ## File path: tests/python/unittest/common.py ## @@ -94,6 +95,23 @@ def random_seed(seed=None):

[GitHub] nswamy commented on a change in pull request #11605: Updating MultiTask example to use new infer api and adding test for CI

2018-07-11 Thread GitBox
nswamy commented on a change in pull request #11605: Updating MultiTask example to use new infer api and adding test for CI URL: https://github.com/apache/incubator-mxnet/pull/11605#discussion_r201789116 ## File path:

[GitHub] aaronmarkham closed issue #11645: Cannot train SmileCNN with Keras-MXNet

2018-07-11 Thread GitBox
aaronmarkham closed issue #11645: Cannot train SmileCNN with Keras-MXNet URL: https://github.com/apache/incubator-mxnet/issues/11645 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] aaronmarkham commented on issue #11645: Cannot train SmileCNN with Keras-MXNet

2018-07-11 Thread GitBox
aaronmarkham commented on issue #11645: Cannot train SmileCNN with Keras-MXNet URL: https://github.com/apache/incubator-mxnet/issues/11645#issuecomment-404259350 Moving this to https://github.com/awslabs/keras-apache-mxnet/issues/134

[GitHub] haojin2 commented on issue #10755: 'Take' operator should support non-zero axis values

2018-07-11 Thread GitBox
haojin2 commented on issue #10755: 'Take' operator should support non-zero axis values URL: https://github.com/apache/incubator-mxnet/issues/10755#issuecomment-404259182 Supported in #11326, can close once that is merged.

[GitHub] haojin2 closed issue #11367: Segfault when running test_operator_gpu.test_sparse_dot many times

2018-07-11 Thread GitBox
haojin2 closed issue #11367: Segfault when running test_operator_gpu.test_sparse_dot many times URL: https://github.com/apache/incubator-mxnet/issues/11367 This is an automated message from the Apache Git Service. To

[GitHub] haojin2 commented on issue #11367: Segfault when running test_operator_gpu.test_sparse_dot many times

2018-07-11 Thread GitBox
haojin2 commented on issue #11367: Segfault when running test_operator_gpu.test_sparse_dot many times URL: https://github.com/apache/incubator-mxnet/issues/11367#issuecomment-404258534 Fix already submitted in #11389. No new reports, closing this issue.

[GitHub] haojin2 commented on issue #11498: Fix InferStorage for sparse fallback in FullyConnected

2018-07-11 Thread GitBox
haojin2 commented on issue #11498: Fix InferStorage for sparse fallback in FullyConnected URL: https://github.com/apache/incubator-mxnet/pull/11498#issuecomment-404257948 @eric-haibin-lin @zheng-da This is an automated

[GitHub] alireza202 edited a comment on issue #11635: Aborted python session when using nd.concat

2018-07-11 Thread GitBox
alireza202 edited a comment on issue #11635: Aborted python session when using nd.concat URL: https://github.com/apache/incubator-mxnet/issues/11635#issuecomment-404255759 The python crashing aside, how would I actually concatenate an empty tensor with another tensor? I tried both

[GitHub] szha commented on a change in pull request #11626: [MXNET-651] MXNet Model Backwards Compatibility Checker

2018-07-11 Thread GitBox
szha commented on a change in pull request #11626: [MXNET-651] MXNet Model Backwards Compatibility Checker URL: https://github.com/apache/incubator-mxnet/pull/11626#discussion_r201785017 ## File path: tests/nightly/model_backwards_compatibility_check/common.py ## @@

[GitHub] anirudh2290 commented on a change in pull request #11630: [WIP] Fix flaky test test_deconvolution

2018-07-11 Thread GitBox
anirudh2290 commented on a change in pull request #11630: [WIP] Fix flaky test test_deconvolution URL: https://github.com/apache/incubator-mxnet/pull/11630#discussion_r201784385 ## File path: Jenkinsfile ## @@ -667,6 +680,21 @@ try { } } }, +

[GitHub] alireza202 commented on issue #11635: Aborted python session when using nd.concat

2018-07-11 Thread GitBox
alireza202 commented on issue #11635: Aborted python session when using nd.concat URL: https://github.com/apache/incubator-mxnet/issues/11635#issuecomment-404255759 The python crashing aside, how would I actually concatenate an empty tensor with another tensor? I tried both dimensions,

[GitHub] zhreshold opened a new pull request #11650: remove windows note for dataloader, fix deprecated warning in python3.7

2018-07-11 Thread GitBox
zhreshold opened a new pull request #11650: remove windows note for dataloader, fix deprecated warning in python3.7 URL: https://github.com/apache/incubator-mxnet/pull/11650 ## Description ## - multi worker data loader is supported in windows, update the doc correspondingly. - fix a

[GitHub] piyushghai commented on a change in pull request #11626: [MXNET-651] MXNet Model Backwards Compatibility Checker

2018-07-11 Thread GitBox
piyushghai commented on a change in pull request #11626: [MXNET-651] MXNet Model Backwards Compatibility Checker URL: https://github.com/apache/incubator-mxnet/pull/11626#discussion_r201782584 ## File path:

[GitHub] ThomasDelteil commented on a change in pull request #11626: [MXNET-651] MXNet Model Backwards Compatibility Checker

2018-07-11 Thread GitBox
ThomasDelteil commented on a change in pull request #11626: [MXNET-651] MXNet Model Backwards Compatibility Checker URL: https://github.com/apache/incubator-mxnet/pull/11626#discussion_r201781970 ## File path:

[GitHub] cclauss opened a new pull request #11648: Modernize Python 2 code to get ready for Python 3

2018-07-11 Thread GitBox
cclauss opened a new pull request #11648: Modernize Python 2 code to get ready for Python 3 URL: https://github.com/apache/incubator-mxnet/pull/11648 A more focused version of #11615 $ __futurize --stage1 -w .__ # http://python-future.org/futurize_cheatsheet.html Minimal,

[GitHub] piyushghai commented on a change in pull request #11626: [MXNET-651] MXNet Model Backwards Compatibility Checker

2018-07-11 Thread GitBox
piyushghai commented on a change in pull request #11626: [MXNET-651] MXNet Model Backwards Compatibility Checker URL: https://github.com/apache/incubator-mxnet/pull/11626#discussion_r201780568 ## File path:

[GitHub] mkolod commented on issue #11325: Added TensorRT runtime integration

2018-07-11 Thread GitBox
mkolod commented on issue #11325: Added TensorRT runtime integration URL: https://github.com/apache/incubator-mxnet/pull/11325#issuecomment-404252100 @reminisce Thanks, that's exactly what @Caenorst and I were thinking in terms of how to test the graph's topology after the rewrite, and

[GitHub] piyushghai commented on a change in pull request #11626: [MXNET-651] MXNet Model Backwards Compatibility Checker

2018-07-11 Thread GitBox
piyushghai commented on a change in pull request #11626: [MXNET-651] MXNet Model Backwards Compatibility Checker URL: https://github.com/apache/incubator-mxnet/pull/11626#discussion_r201780155 ## File path: tests/nightly/model_backwards_compatibility_check/common.py ## @@

[GitHub] anirudh2290 commented on issue #11635: Aborted python session when using nd.concat

2018-07-11 Thread GitBox
anirudh2290 commented on issue #11635: Aborted python session when using nd.concat URL: https://github.com/apache/incubator-mxnet/issues/11635#issuecomment-404251216 you should switch to 1.2.0 to prevent the python process from crashing

[GitHub] haojin2 commented on a change in pull request #11625: [MXNET-647] Add Diff Collator

2018-07-11 Thread GitBox
haojin2 commented on a change in pull request #11625: [MXNET-647] Add Diff Collator URL: https://github.com/apache/incubator-mxnet/pull/11625#discussion_r201779496 ## File path: tools/diff_collator.py ## @@ -0,0 +1,165 @@ +# Licensed to the Apache Software Foundation

[GitHub] haojin2 commented on a change in pull request #11625: [MXNET-647] Add Diff Collator

2018-07-11 Thread GitBox
haojin2 commented on a change in pull request #11625: [MXNET-647] Add Diff Collator URL: https://github.com/apache/incubator-mxnet/pull/11625#discussion_r201778366 ## File path: tools/diff_collator.py ## @@ -0,0 +1,165 @@ +# Licensed to the Apache Software Foundation

[GitHub] mkolod commented on a change in pull request #11325: Added TensorRT runtime integration

2018-07-11 Thread GitBox
mkolod commented on a change in pull request #11325: Added TensorRT runtime integration URL: https://github.com/apache/incubator-mxnet/pull/11325#discussion_r201778476 ## File path: src/operator/contrib/tensorrt.cc ## @@ -0,0 +1,196 @@ +/* + * Licensed to the Apache

[GitHub] cclauss closed pull request #11615: Modernize Python 2 code to get ready for Python 3

2018-07-11 Thread GitBox
cclauss closed pull request #11615: Modernize Python 2 code to get ready for Python 3 URL: https://github.com/apache/incubator-mxnet/pull/11615 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] haojin2 commented on issue #11635: Aborted python session when using nd.concat

2018-07-11 Thread GitBox
haojin2 commented on issue #11635: Aborted python session when using nd.concat URL: https://github.com/apache/incubator-mxnet/issues/11635#issuecomment-404247607 The issue could possibly be fixed by changing the InferShape of concat operator to adopt mutual shape inference.

[GitHub] anirudh2290 commented on a change in pull request #11573: Add stable nrm2 Reducer

2018-07-11 Thread GitBox
anirudh2290 commented on a change in pull request #11573: Add stable nrm2 Reducer URL: https://github.com/apache/incubator-mxnet/pull/11573#discussion_r201775313 ## File path: src/operator/mshadow_op.h ## @@ -815,6 +868,76 @@ struct nanprod { } }; +/*! \brief

[GitHub] ThomasDelteil commented on a change in pull request #11626: [MXNET-651] MXNet Model Backwards Compatibility Checker

2018-07-11 Thread GitBox
ThomasDelteil commented on a change in pull request #11626: [MXNET-651] MXNet Model Backwards Compatibility Checker URL: https://github.com/apache/incubator-mxnet/pull/11626#discussion_r201773901 ## File path:

[GitHub] ThomasDelteil commented on a change in pull request #11626: [MXNET-651] MXNet Model Backwards Compatibility Checker

2018-07-11 Thread GitBox
ThomasDelteil commented on a change in pull request #11626: [MXNET-651] MXNet Model Backwards Compatibility Checker URL: https://github.com/apache/incubator-mxnet/pull/11626#discussion_r201773565 ## File path: tests/nightly/model_backwards_compatibility_check/common.py ##

[GitHub] ThomasDelteil commented on a change in pull request #11626: [MXNET-651] MXNet Model Backwards Compatibility Checker

2018-07-11 Thread GitBox
ThomasDelteil commented on a change in pull request #11626: [MXNET-651] MXNet Model Backwards Compatibility Checker URL: https://github.com/apache/incubator-mxnet/pull/11626#discussion_r201774694 ## File path:

[GitHub] mkolod commented on a change in pull request #11325: Added TensorRT runtime integration

2018-07-11 Thread GitBox
mkolod commented on a change in pull request #11325: Added TensorRT runtime integration URL: https://github.com/apache/incubator-mxnet/pull/11325#discussion_r201774632 ## File path: Makefile ## @@ -94,6 +94,14 @@ else endif CFLAGS += -I$(TPARTYDIR)/mshadow/

[GitHub] aaronmarkham commented on issue #11562: [MXNET-244] Update RaspberryPI instructions

2018-07-11 Thread GitBox
aaronmarkham commented on issue #11562: [MXNET-244] Update RaspberryPI instructions URL: https://github.com/apache/incubator-mxnet/pull/11562#issuecomment-404245844 @larroy - I'd remove the `##` and just `**` bold stuff for section headers. You could also use html styles and nested ``

[GitHub] aaronmarkham commented on a change in pull request #11049: Add linux and macos MKLDNN Building Instruction

2018-07-11 Thread GitBox
aaronmarkham commented on a change in pull request #11049: Add linux and macos MKLDNN Building Instruction URL: https://github.com/apache/incubator-mxnet/pull/11049#discussion_r201771261 ## File path: MKLDNN_README.md ## @@ -0,0 +1,287 @@ +# Build/Install MXNet with

[GitHub] aaronmarkham commented on a change in pull request #11049: Add linux and macos MKLDNN Building Instruction

2018-07-11 Thread GitBox
aaronmarkham commented on a change in pull request #11049: Add linux and macos MKLDNN Building Instruction URL: https://github.com/apache/incubator-mxnet/pull/11049#discussion_r201768297 ## File path: MKLDNN_README.md ## @@ -0,0 +1,287 @@ +# Build/Install MXNet with

[GitHub] aaronmarkham commented on a change in pull request #11049: Add linux and macos MKLDNN Building Instruction

2018-07-11 Thread GitBox
aaronmarkham commented on a change in pull request #11049: Add linux and macos MKLDNN Building Instruction URL: https://github.com/apache/incubator-mxnet/pull/11049#discussion_r201766017 ## File path: MKLDNN_README.md ## @@ -0,0 +1,302 @@ +# Build/Install MXNet with

[GitHub] aaronmarkham commented on a change in pull request #11049: Add linux and macos MKLDNN Building Instruction

2018-07-11 Thread GitBox
aaronmarkham commented on a change in pull request #11049: Add linux and macos MKLDNN Building Instruction URL: https://github.com/apache/incubator-mxnet/pull/11049#discussion_r201769386 ## File path: MKLDNN_README.md ## @@ -0,0 +1,287 @@ +# Build/Install MXNet with

[GitHub] aaronmarkham commented on a change in pull request #11049: Add linux and macos MKLDNN Building Instruction

2018-07-11 Thread GitBox
aaronmarkham commented on a change in pull request #11049: Add linux and macos MKLDNN Building Instruction URL: https://github.com/apache/incubator-mxnet/pull/11049#discussion_r201766968 ## File path: MKLDNN_README.md ## @@ -0,0 +1,229 @@ +# Build/Install MXNet with

[GitHub] aaronmarkham commented on a change in pull request #11049: Add linux and macos MKLDNN Building Instruction

2018-07-11 Thread GitBox
aaronmarkham commented on a change in pull request #11049: Add linux and macos MKLDNN Building Instruction URL: https://github.com/apache/incubator-mxnet/pull/11049#discussion_r201767766 ## File path: MKLDNN_README.md ## @@ -0,0 +1,287 @@ +# Build/Install MXNet with

[GitHub] aaronmarkham commented on a change in pull request #11049: Add linux and macos MKLDNN Building Instruction

2018-07-11 Thread GitBox
aaronmarkham commented on a change in pull request #11049: Add linux and macos MKLDNN Building Instruction URL: https://github.com/apache/incubator-mxnet/pull/11049#discussion_r201767466 ## File path: MKLDNN_README.md ## @@ -0,0 +1,229 @@ +# Build/Install MXNet with

[GitHub] szha commented on issue #11616: Flaky test test_gluon.test_export

2018-07-11 Thread GitBox
szha commented on issue #11616: Flaky test test_gluon.test_export URL: https://github.com/apache/incubator-mxnet/issues/11616#issuecomment-404243226 `export MXNET_GPU_MEM_POOL_RESERVE=100` should effectively disable the pool.

[GitHub] haojin2 commented on a change in pull request #11631: Fix batchnorm problem with sparse matrices when fix_gamma=True

2018-07-11 Thread GitBox
haojin2 commented on a change in pull request #11631: Fix batchnorm problem with sparse matrices when fix_gamma=True URL: https://github.com/apache/incubator-mxnet/pull/11631#discussion_r201770974 ## File path: src/operator/nn/batch_norm.cc ## @@ -452,19 +453,56 @@ static

[GitHub] haojin2 commented on issue #11647: Sparse input matrices are not supported in batchnorm operator when fix_gamma=True

2018-07-11 Thread GitBox
haojin2 commented on issue #11647: Sparse input matrices are not supported in batchnorm operator when fix_gamma=True URL: https://github.com/apache/incubator-mxnet/issues/11647#issuecomment-404242662 #11631 This is an

[GitHub] haojin2 commented on issue #11647: Sparse input matrices are not supported in batchnorm operator when fix_gamma=True

2018-07-11 Thread GitBox
haojin2 commented on issue #11647: Sparse input matrices are not supported in batchnorm operator when fix_gamma=True URL: https://github.com/apache/incubator-mxnet/issues/11647#issuecomment-404242519 Suggested label: Feature Request

[GitHub] haojin2 opened a new issue #11647: Sparse input matrices are not supported in batchnorm operator when fix_gamma=True

2018-07-11 Thread GitBox
haojin2 opened a new issue #11647: Sparse input matrices are not supported in batchnorm operator when fix_gamma=True URL: https://github.com/apache/incubator-mxnet/issues/11647 As title. This is an automated message from the

[GitHub] golo314 commented on issue #11543: Seg. Fault installing cpp-package

2018-07-11 Thread GitBox
golo314 commented on issue #11543: Seg. Fault installing cpp-package URL: https://github.com/apache/incubator-mxnet/issues/11543#issuecomment-404240855 The problem was with OpenBLAS. I installed ATLAS and was able to build.

[GitHub] golo314 closed issue #11543: Seg. Fault installing cpp-package

2018-07-11 Thread GitBox
golo314 closed issue #11543: Seg. Fault installing cpp-package URL: https://github.com/apache/incubator-mxnet/issues/11543 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] szha opened a new issue #11646: download function should avoid race-condition

2018-07-11 Thread GitBox
szha opened a new issue #11646: download function should avoid race-condition URL: https://github.com/apache/incubator-mxnet/issues/11646 Currently the download function would directly write to target file path, which causes file integrity problem when multiple processes use the same logic

[GitHub] marcoabreu edited a comment on issue #11616: Flaky test test_gluon.test_export

2018-07-11 Thread GitBox
marcoabreu edited a comment on issue #11616: Flaky test test_gluon.test_export URL: https://github.com/apache/incubator-mxnet/issues/11616#issuecomment-404238313 But yeah, we are regularly testing MXNet CPU on windows and don't seem to encounter many problems. I also wouldn't know much

[GitHub] apeforest commented on issue #11620: LBSGD does not have momentum

2018-07-11 Thread GitBox
apeforest commented on issue #11620: LBSGD does not have momentum URL: https://github.com/apache/incubator-mxnet/issues/11620#issuecomment-404238513 @sandeep-krishnamurthy Please label this issue _Example_, _Good for first time_

[GitHub] marcoabreu commented on issue #11616: Flaky test test_gluon.test_export

2018-07-11 Thread GitBox
marcoabreu commented on issue #11616: Flaky test test_gluon.test_export URL: https://github.com/apache/incubator-mxnet/issues/11616#issuecomment-404238313 But yeah, we are regularly testing MXNet CPU on windows and don't seem to encounter many problems. I also wouldn't know much reason

[GitHub] eric-haibin-lin closed pull request #11577: Fix flaky test test_gluon.test_hybrid_static_memory_switching

2018-07-11 Thread GitBox
eric-haibin-lin closed pull request #11577: Fix flaky test test_gluon.test_hybrid_static_memory_switching URL: https://github.com/apache/incubator-mxnet/pull/11577 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] apeforest commented on issue #11634: website install page query string needed for devices

2018-07-11 Thread GitBox
apeforest commented on issue #11634: website install page query string needed for devices URL: https://github.com/apache/incubator-mxnet/issues/11634#issuecomment-404237681 @sandeep-krishnamurthy Please label this _Feature Request_

[incubator-mxnet] branch master updated: Fix flaky test test_gluon.test_hybrid_static_memory_switching (#11577)

2018-07-11 Thread haibin
This is an automated email from the ASF dual-hosted git repository. haibin 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 b264f6f Fix flaky test

[GitHub] marcoabreu commented on issue #11616: Flaky test test_gluon.test_export

2018-07-11 Thread GitBox
marcoabreu commented on issue #11616: Flaky test test_gluon.test_export URL: https://github.com/apache/incubator-mxnet/issues/11616#issuecomment-404237611 I tried, what you described was exactly my observation: you run out of cuda memory if you run MXNet in parallel. I didn't know we had

[GitHub] apeforest commented on issue #11635: Aborted python session when using nd.concat

2018-07-11 Thread GitBox
apeforest commented on issue #11635: Aborted python session when using nd.concat URL: https://github.com/apache/incubator-mxnet/issues/11635#issuecomment-404237389 @alireza202 I think the reason is because you are concatenating on the wrong axis. Numpy would issue an error if you do the

[GitHub] szha commented on issue #11616: Flaky test test_gluon.test_export

2018-07-11 Thread GitBox
szha commented on issue #11616: Flaky test test_gluon.test_export URL: https://github.com/apache/incubator-mxnet/issues/11616#issuecomment-404235872 Users don't directly deal with CUDA. They interact with CUDA through MXNet. MXNet has internal memory pool to hold previously allocated

[GitHub] sandeep-krishnamurthy commented on issue #11645: Cannot train SmileCNN with Keras-MXNet

2018-07-11 Thread GitBox
sandeep-krishnamurthy commented on issue #11645: Cannot train SmileCNN with Keras-MXNet URL: https://github.com/apache/incubator-mxnet/issues/11645#issuecomment-404235388 @aaronmarkham - Can you please open issue with keras-mxnet repo? Looks like issue in MXNet backend of Keras.

[GitHub] apeforest edited a comment on issue #11645: Cannot train SmileCNN with Keras-MXNet

2018-07-11 Thread GitBox
apeforest edited a comment on issue #11645: Cannot train SmileCNN with Keras-MXNet URL: https://github.com/apache/incubator-mxnet/issues/11645#issuecomment-404233406 @kalyc Can you please help to investigate this issue? Thx

[GitHub] apeforest commented on issue #11645: Cannot train SmileCNN with Keras-MXNet

2018-07-11 Thread GitBox
apeforest commented on issue #11645: Cannot train SmileCNN with Keras-MXNet URL: https://github.com/apache/incubator-mxnet/issues/11645#issuecomment-404233642 @sandeep-krishnamurthy @nswamy Please label this issue _Keras_

[GitHub] apeforest commented on issue #11645: Cannot train SmileCNN with Keras-MXNet

2018-07-11 Thread GitBox
apeforest commented on issue #11645: Cannot train SmileCNN with Keras-MXNet URL: https://github.com/apache/incubator-mxnet/issues/11645#issuecomment-404233406 @kalyc Can you please help to check this example? Thx This is an

[GitHub] marcoabreu commented on issue #11616: Flaky test test_gluon.test_export

2018-07-11 Thread GitBox
marcoabreu commented on issue #11616: Flaky test test_gluon.test_export URL: https://github.com/apache/incubator-mxnet/issues/11616#issuecomment-404230989 Every process can manage it's own GPU memory - cuda supports multi tenancy out of the box. You're probably only going to run into out

[GitHub] marcoabreu commented on issue #11640: Write a new operator by myself, error says Backward is not implemented.but i write it indeed

2018-07-11 Thread GitBox
marcoabreu commented on issue #11640: Write a new operator by myself,error says Backward is not implemented.but i write it indeed URL: https://github.com/apache/incubator-mxnet/issues/11640#issuecomment-404228516 For these kind of things I'd like to to invite a thread at discuss.mxnet.io

[GitHub] szha commented on issue #11616: Flaky test test_gluon.test_export

2018-07-11 Thread GitBox
szha commented on issue #11616: Flaky test test_gluon.test_export URL: https://github.com/apache/incubator-mxnet/issues/11616#issuecomment-404227932 OK. Following this logic, we either have inter-process memory management for GPU or our CPU users can't run multi-process CPU inference.

[GitHub] marcoabreu edited a comment on issue #11616: Flaky test test_gluon.test_export

2018-07-11 Thread GitBox
marcoabreu edited a comment on issue #11616: Flaky test test_gluon.test_export URL: https://github.com/apache/incubator-mxnet/issues/11616#issuecomment-404226812 My stand on that would be that our tests should all be completely self contained and thus have no other side effects that

[GitHub] marcoabreu commented on issue #11616: Flaky test test_gluon.test_export

2018-07-11 Thread GitBox
marcoabreu commented on issue #11616: Flaky test test_gluon.test_export URL: https://github.com/apache/incubator-mxnet/issues/11616#issuecomment-404226812 My stand on that would be that our tests should all be completely self contained. What do you rgibk

[GitHub] KellenSunderland opened a new issue #11616: Flaky test test_gluon.test_export

2018-07-11 Thread GitBox
KellenSunderland opened a new issue #11616: Flaky test test_gluon.test_export URL: https://github.com/apache/incubator-mxnet/issues/11616 Failed build: http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/master/1148/pipeline

[GitHub] marcoabreu closed issue #11616: Flaky test test_gluon.test_export

2018-07-11 Thread GitBox
marcoabreu closed issue #11616: Flaky test test_gluon.test_export URL: https://github.com/apache/incubator-mxnet/issues/11616 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] marcoabreu commented on issue #11616: Flaky test test_gluon.test_export

2018-07-11 Thread GitBox
marcoabreu commented on issue #11616: Flaky test test_gluon.test_export URL: https://github.com/apache/incubator-mxnet/issues/11616#issuecomment-404226482 Yes, because everything else is undefined behaviour, right? If we design a system to be used only in a specific way, we don't

[GitHub] zheng-da commented on issue #11641: [WIP] make CachedOp a normal operator

2018-07-11 Thread GitBox
zheng-da commented on issue #11641: [WIP] make CachedOp a normal operator URL: https://github.com/apache/incubator-mxnet/pull/11641#issuecomment-404226450 @larroy thanks for the review. The code, especially the API design, is still experimental. I'll let you know when the code is ready for

[GitHub] szha commented on issue #11616: Flaky test test_gluon.test_export

2018-07-11 Thread GitBox
szha commented on issue #11616: Flaky test test_gluon.test_export URL: https://github.com/apache/incubator-mxnet/issues/11616#issuecomment-404226152 We can also ask whether one test should affect another in any way, and if so, why, and why not managed explicitly.

[GitHub] thomelane commented on issue #11629: deconvolution result different with caffe result

2018-07-11 Thread GitBox
thomelane commented on issue #11629: deconvolution result different with caffe result URL: https://github.com/apache/incubator-mxnet/issues/11629#issuecomment-404225649 See https://github.com/apache/incubator-mxnet/issues/11203. Seems like a case of the same bug, found with an even

[GitHub] szha commented on issue #11616: Flaky test test_gluon.test_export

2018-07-11 Thread GitBox
szha commented on issue #11616: Flaky test test_gluon.test_export URL: https://github.com/apache/incubator-mxnet/issues/11616#issuecomment-404225226 @marcoabreu and to you, that statement equals "you are only allowed to run a single instance of MXNet"?

[GitHub] aaronmarkham opened a new issue #11645: Cannot train SmileCNN with Keras-MXNet

2018-07-11 Thread GitBox
aaronmarkham opened a new issue #11645: Cannot train SmileCNN with Keras-MXNet URL: https://github.com/apache/incubator-mxnet/issues/11645 ## Description Error during training of the SmileCNN demo using Keras-MXNet. (Error in operator conv2d_1/conv2d1) ## Environment info

[GitHub] szha opened a new issue #11644: Jenkins problem

2018-07-11 Thread GitBox
szha opened a new issue #11644: Jenkins problem URL: https://github.com/apache/incubator-mxnet/issues/11644 ``` Remote call on JNLP4-connect connection from ip-172-31-25-206.us-west-2.compute.internal/172.31.25.206:52920 failed. The channel is closing down or has closed down ```

[GitHub] ashkanaev edited a comment on issue #6204: [Fast-RCNN] Proposal operator failed: too many resources requested for launch

2018-07-11 Thread GitBox
ashkanaev edited a comment on issue #6204: [Fast-RCNN] Proposal operator failed: too many resources requested for launch URL: https://github.com/apache/incubator-mxnet/issues/6204#issuecomment-404207841 Is there any update? Maybe someone developers who responsible for pip build could

[GitHub] ashkanaev commented on issue #6204: [Fast-RCNN] Proposal operator failed: too many resources requested for launch

2018-07-11 Thread GitBox
ashkanaev commented on issue #6204: [Fast-RCNN] Proposal operator failed: too many resources requested for launch URL: https://github.com/apache/incubator-mxnet/issues/6204#issuecomment-404207841 Is there any update? Maybe someone from developers who responsible for pip build could

[GitHub] Ericargus commented on issue #470: More informative debug info for errors in engine

2018-07-11 Thread GitBox
Ericargus commented on issue #470: More informative debug info for errors in engine URL: https://github.com/apache/incubator-mxnet/issues/470#issuecomment-404196747 i @piiswrong hello, i met this error: mxnet.base.MXNetError: [22:01:36] include/mxnet/././tensor_blob.h:235: Check

[GitHub] Ericargus commented on issue #11640: Write a new operator by myself, error says Backward is not implemented.but i write it indeed

2018-07-11 Thread GitBox
Ericargus commented on issue #11640: Write a new operator by myself,error says Backward is not implemented.but i write it indeed URL: https://github.com/apache/incubator-mxnet/issues/11640#issuecomment-404186994 @marcoabreu I'm careless.I solved it by copy similar offical operator,and

[GitHub] Ericargus commented on issue #11640: Write a new operator by myself, error says Backward is not implemented.but i write it indeed

2018-07-11 Thread GitBox
Ericargus commented on issue #11640: Write a new operator by myself,error says Backward is not implemented.but i write it indeed URL: https://github.com/apache/incubator-mxnet/issues/11640#issuecomment-404184833 @marcoabreu but i have a new problem.(error reports "TBlob.get: device type

[GitHub] marcoabreu commented on issue #11640: Write a new operator by myself, error says Backward is not implemented.but i write it indeed

2018-07-11 Thread GitBox
marcoabreu commented on issue #11640: Write a new operator by myself,error says Backward is not implemented.but i write it indeed URL: https://github.com/apache/incubator-mxnet/issues/11640#issuecomment-404184787 Awesome! Do you mind sharing what you did to solve the problem?

[GitHub] marcoabreu closed issue #11640: Write a new operator by myself, error says Backward is not implemented.but i write it indeed

2018-07-11 Thread GitBox
marcoabreu closed issue #11640: Write a new operator by myself,error says Backward is not implemented.but i write it indeed URL: https://github.com/apache/incubator-mxnet/issues/11640 This is an automated message from the

[GitHub] Ericargus commented on issue #11640: Write a new operator by myself, error says Backward is not implemented.but i write it indeed

2018-07-11 Thread GitBox
Ericargus commented on issue #11640: Write a new operator by myself,error says Backward is not implemented.but i write it indeed URL: https://github.com/apache/incubator-mxnet/issues/11640#issuecomment-404184247 @marcoabreu i have solved it

[GitHub] Ericargus commented on issue #11640: Write a new operator by myself, error says Backward is not implemented.but i write it indeed

2018-07-11 Thread GitBox
Ericargus commented on issue #11640: Write a new operator by myself,error says Backward is not implemented.but i write it indeed URL: https://github.com/apache/incubator-mxnet/issues/11640#issuecomment-404184093 I have solved it.

[GitHub] ifeherva edited a comment on issue #9253: Numpy Diag in MXNet

2018-07-11 Thread GitBox
ifeherva edited a comment on issue #9253: Numpy Diag in MXNet URL: https://github.com/apache/incubator-mxnet/issues/9253#issuecomment-403290348 I am working on replicating np.diag(). Once done will do the full diagonal operator. This is my first operator so I am learning a lot at the

[GitHub] ifeherva opened a new pull request #11643: Added the diag() operator

2018-07-11 Thread GitBox
ifeherva opened a new pull request #11643: Added the diag() operator URL: https://github.com/apache/incubator-mxnet/pull/11643 ## Description ## Added a new tensor operator called diag() replicating numpy.diag() ## Checklist ## ### Essentials ### Please feel free to remove

[GitHub] xinyu-intel commented on issue #11049: Add linux and macos MKLDNN Building Instruction

2018-07-11 Thread GitBox
xinyu-intel commented on issue #11049: Add linux and macos MKLDNN Building Instruction URL: https://github.com/apache/incubator-mxnet/pull/11049#issuecomment-404165835 Somebody said that installation of your software in Mac OS is really hard

[GitHub] xinyu-intel commented on issue #11049: Add linux and macos MKLDNN Building Instruction

2018-07-11 Thread GitBox
xinyu-intel commented on issue #11049: Add linux and macos MKLDNN Building Instruction URL: https://github.com/apache/incubator-mxnet/pull/11049#issuecomment-404163601 @larroy Thanks for your suggestions. Doc for windows is based on cmake, and cmake instructions for linux and mac will be

[GitHub] larroy commented on a change in pull request #11636: [WIP] set MXNET_DATA_DIR as base for downloaded models through base.data_dir()

2018-07-11 Thread GitBox
larroy commented on a change in pull request #11636: [WIP] set MXNET_DATA_DIR as base for downloaded models through base.data_dir() URL: https://github.com/apache/incubator-mxnet/pull/11636#discussion_r201677321 ## File path: docs/faq/env_var.md ## @@ -126,6 +126,11 @@

[GitHub] larroy commented on a change in pull request #11636: [WIP] set MXNET_DATA_DIR as base for downloaded models through base.data_dir()

2018-07-11 Thread GitBox
larroy commented on a change in pull request #11636: [WIP] set MXNET_DATA_DIR as base for downloaded models through base.data_dir() URL: https://github.com/apache/incubator-mxnet/pull/11636#discussion_r201677265 ## File path: docs/faq/env_var.md ## @@ -126,6 +126,11 @@

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

2018-07-11 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 1c3f920 Bump the publish

[GitHub] larroy commented on issue #11049: Add linux and macos MKLDNN Building Instruction

2018-07-11 Thread GitBox
larroy commented on issue #11049: Add linux and macos MKLDNN Building Instruction URL: https://github.com/apache/incubator-mxnet/pull/11049#issuecomment-404144051 Can we document with CMake? we are trying to migrate to CMake, so would be great to have instructions that are future proof.

[GitHub] yajiedesign commented on issue #11610: import error

2018-07-11 Thread GitBox
yajiedesign commented on issue #11610: import error URL: https://github.com/apache/incubator-mxnet/issues/11610#issuecomment-404133908 @adaa libmxnet.dll This is an automated message from the Apache Git Service. To

[GitHub] larroy commented on a change in pull request #11641: [WIP] make CachedOp a normal operator

2018-07-11 Thread GitBox
larroy commented on a change in pull request #11641: [WIP] make CachedOp a normal operator URL: https://github.com/apache/incubator-mxnet/pull/11641#discussion_r201625843 ## File path: tests/python/unittest/test_operator.py ## @@ -6968,6 +6968,39 @@ def

[GitHub] larroy commented on a change in pull request #11641: [WIP] make CachedOp a normal operator

2018-07-11 Thread GitBox
larroy commented on a change in pull request #11641: [WIP] make CachedOp a normal operator URL: https://github.com/apache/incubator-mxnet/pull/11641#discussion_r201625451 ## File path: src/imperative/cached_op.h ## @@ -116,6 +116,24 @@ class CachedOp {

[GitHub] larroy commented on a change in pull request #11641: [WIP] make CachedOp a normal operator

2018-07-11 Thread GitBox
larroy commented on a change in pull request #11641: [WIP] make CachedOp a normal operator URL: https://github.com/apache/incubator-mxnet/pull/11641#discussion_r201624918 ## File path: src/imperative/cached_op.cc ## @@ -1047,6 +1047,105 @@ void CachedOp::Backward(

[GitHub] larroy commented on a change in pull request #11641: [WIP] make CachedOp a normal operator

2018-07-11 Thread GitBox
larroy commented on a change in pull request #11641: [WIP] make CachedOp a normal operator URL: https://github.com/apache/incubator-mxnet/pull/11641#discussion_r201624481 ## File path: src/imperative/cached_op.cc ## @@ -1047,6 +1047,105 @@ void CachedOp::Backward(

[GitHub] larroy commented on a change in pull request #11641: [WIP] make CachedOp a normal operator

2018-07-11 Thread GitBox
larroy commented on a change in pull request #11641: [WIP] make CachedOp a normal operator URL: https://github.com/apache/incubator-mxnet/pull/11641#discussion_r201623478 ## File path: src/imperative/cached_op.cc ## @@ -1047,6 +1047,105 @@ void CachedOp::Backward(

[GitHub] larroy commented on a change in pull request #11641: [WIP] make CachedOp a normal operator

2018-07-11 Thread GitBox
larroy commented on a change in pull request #11641: [WIP] make CachedOp a normal operator URL: https://github.com/apache/incubator-mxnet/pull/11641#discussion_r201623330 ## File path: src/imperative/cached_op.cc ## @@ -1047,6 +1047,105 @@ void CachedOp::Backward(

[GitHub] larroy commented on a change in pull request #11641: [WIP] make CachedOp a normal operator

2018-07-11 Thread GitBox
larroy commented on a change in pull request #11641: [WIP] make CachedOp a normal operator URL: https://github.com/apache/incubator-mxnet/pull/11641#discussion_r201623080 ## File path: src/c_api/c_api_symbolic.cc ## @@ -366,6 +367,36 @@ int

[GitHub] larroy commented on a change in pull request #11641: [WIP] make CachedOp a normal operator

2018-07-11 Thread GitBox
larroy commented on a change in pull request #11641: [WIP] make CachedOp a normal operator URL: https://github.com/apache/incubator-mxnet/pull/11641#discussion_r201622787 ## File path: src/c_api/c_api_symbolic.cc ## @@ -366,6 +367,36 @@ int

[GitHub] larroy commented on a change in pull request #11641: [WIP] make CachedOp a normal operator

2018-07-11 Thread GitBox
larroy commented on a change in pull request #11641: [WIP] make CachedOp a normal operator URL: https://github.com/apache/incubator-mxnet/pull/11641#discussion_r201622379 ## File path: src/c_api/c_api_symbolic.cc ## @@ -366,6 +367,36 @@ int

<    1   2   3   4   >