[GitHub] BrianHanechak opened a new issue #12628: Version 1.3.0 doesn't work for me on Windows (cu92); 1.3.0b20180915 works fine

2018-09-20 Thread GitBox
BrianHanechak opened a new issue #12628: Version 1.3.0 doesn't work for me on Windows (cu92); 1.3.0b20180915 works fine URL: https://github.com/apache/incubator-mxnet/issues/12628 ## Description When I install version 1.3.0, the Python process exits whenever I do any GPU operation. 1.3.

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

2018-09-20 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 d0a99ac Bump the publish ti

[GitHub] lovickie commented on issue #12613: Report a bug of Engine::Get()

2018-09-20 Thread GitBox
lovickie commented on issue #12613: Report a bug of Engine::Get() URL: https://github.com/apache/incubator-mxnet/issues/12613#issuecomment-423430872 More information as follows. My project x, use mxnet to do sth, and also provides python api. When import mxnet before import x, the destr

[GitHub] szha commented on a change in pull request #12572: Make Gluon download function to be atomic

2018-09-20 Thread GitBox
szha commented on a change in pull request #12572: Make Gluon download function to be atomic URL: https://github.com/apache/incubator-mxnet/pull/12572#discussion_r219393275 ## File path: python/mxnet/gluon/utils.py ## @@ -195,6 +197,43 @@ def check_sha1(filename, sha1_hash

[GitHub] tqchen commented on issue #12047: [MXNET-779]Add DLPack Transformation API

2018-09-20 Thread GitBox
tqchen commented on issue #12047: [MXNET-779]Add DLPack Transformation API URL: https://github.com/apache/incubator-mxnet/pull/12047#issuecomment-423425707 @szha can you also take a round of review and if there is no problem, merge this in? -

[GitHub] tqchen commented on issue #12047: [MXNET-779]Add DLPack Transformation API

2018-09-20 Thread GitBox
tqchen commented on issue #12047: [MXNET-779]Add DLPack Transformation API URL: https://github.com/apache/incubator-mxnet/pull/12047#issuecomment-423425411 I think it is good to go from my end. thanks @wkcn . @piiswrong please take another look, if there is no issues I will merge this in in

[GitHub] andrewfayres commented on issue #12624: [MXNET-913] Java API -- NDArray Macros remove Option field

2018-09-20 Thread GitBox
andrewfayres commented on issue #12624: [MXNET-913] Java API -- NDArray Macros remove Option field URL: https://github.com/apache/incubator-mxnet/pull/12624#issuecomment-423421265 Looked through this and everything looks good. In the morning I'll clone this and build so that I can take a p

[GitHub] eric-haibin-lin commented on issue #9721: Feature Request gradcheck for Gluon and NDArray

2018-09-20 Thread GitBox
eric-haibin-lin commented on issue #9721: Feature Request gradcheck for Gluon and NDArray URL: https://github.com/apache/incubator-mxnet/issues/9721#issuecomment-423419738 TF has https://www.tensorflow.org/api_docs/python/tf/check_numerics

[GitHub] eric-haibin-lin commented on issue #12623: Feature Request NDArray.has_nan() method

2018-09-20 Thread GitBox
eric-haibin-lin commented on issue #12623: Feature Request NDArray.has_nan() method URL: https://github.com/apache/incubator-mxnet/issues/12623#issuecomment-423419594 Similarly NDArray.is_inf() https://www.tensorflow.org/api_docs/python/tf/is_inf --

[GitHub] paroda opened a new issue #12627: Interop bug - clojure - executor/forward - param coercion issue

2018-09-20 Thread GitBox
paroda opened a new issue #12627: Interop bug - clojure - executor/forward - param coercion issue URL: https://github.com/apache/incubator-mxnet/issues/12627 ## Description ClassCastException when using executor/forward in cojure-mxnet with 3 args ## Environment info (Required)

[GitHub] apeforest commented on issue #12594: [MXNET-867] Pooling1D with "same" padding

2018-09-20 Thread GitBox
apeforest commented on issue #12594: [MXNET-867] Pooling1D with "same" padding URL: https://github.com/apache/incubator-mxnet/pull/12594#issuecomment-423415586 Thanks for making this contribution. LGTM overall. You may also want to update the operator description in operator/nn/pooling.cc s

[GitHub] apeforest commented on a change in pull request #12594: [MXNET-867] Pooling1D with "same" padding

2018-09-20 Thread GitBox
apeforest commented on a change in pull request #12594: [MXNET-867] Pooling1D with "same" padding URL: https://github.com/apache/incubator-mxnet/pull/12594#discussion_r219385163 ## File path: tests/python/unittest/test_operator.py ## @@ -6957,6 +6957,21 @@ def test_invalid

[GitHub] apeforest commented on a change in pull request #12594: [MXNET-867] Pooling1D with "same" padding

2018-09-20 Thread GitBox
apeforest commented on a change in pull request #12594: [MXNET-867] Pooling1D with "same" padding URL: https://github.com/apache/incubator-mxnet/pull/12594#discussion_r219385163 ## File path: tests/python/unittest/test_operator.py ## @@ -6957,6 +6957,21 @@ def test_invalid

[GitHub] ChaiBapchya commented on a change in pull request #12594: [MXNET-867] Pooling1D with "same" padding

2018-09-20 Thread GitBox
ChaiBapchya commented on a change in pull request #12594: [MXNET-867] Pooling1D with "same" padding URL: https://github.com/apache/incubator-mxnet/pull/12594#discussion_r219384818 ## File path: src/operator/nn/pooling.cc ## @@ -96,6 +96,11 @@ static bool PoolingShape(const

[GitHub] apeforest commented on a change in pull request #12594: [MXNET-867] Pooling1D with "same" padding

2018-09-20 Thread GitBox
apeforest commented on a change in pull request #12594: [MXNET-867] Pooling1D with "same" padding URL: https://github.com/apache/incubator-mxnet/pull/12594#discussion_r219384478 ## File path: src/operator/nn/pooling.cc ## @@ -126,11 +131,15 @@ static bool PoolingShape(cons

[GitHub] apeforest commented on a change in pull request #12594: [MXNET-867] Pooling1D with "same" padding

2018-09-20 Thread GitBox
apeforest commented on a change in pull request #12594: [MXNET-867] Pooling1D with "same" padding URL: https://github.com/apache/incubator-mxnet/pull/12594#discussion_r219384312 ## File path: src/operator/nn/pooling.cc ## @@ -96,6 +96,11 @@ static bool PoolingShape(const n

[GitHub] jeremiedb commented on issue #12431: [R] use of mx.io.arrayiter completely crashes R environment

2018-09-20 Thread GitBox
jeremiedb commented on issue #12431: [R] use of mx.io.arrayiter completely crashes R environment URL: https://github.com/apache/incubator-mxnet/issues/12431#issuecomment-423410481 I think the issue comes from the evaluation metric. The following code include a modified rmse metric that fl

[GitHub] sandeep-krishnamurthy commented on issue #10697: [MXNET-360]auto convert str to bytes in img.imdecode when py3

2018-09-20 Thread GitBox
sandeep-krishnamurthy commented on issue #10697: [MXNET-360]auto convert str to bytes in img.imdecode when py3 URL: https://github.com/apache/incubator-mxnet/pull/10697#issuecomment-423410027 @zhreshold - This is good to go, if it looks fine for you. ---

[GitHub] sandeep-krishnamurthy commented on issue #12315: Enable gluon multi worker data loader test

2018-09-20 Thread GitBox
sandeep-krishnamurthy commented on issue #12315: Enable gluon multi worker data loader test URL: https://github.com/apache/incubator-mxnet/pull/12315#issuecomment-423409328 @haojin2 - Can you please take a look? Thanks. This

[GitHub] sandeep-krishnamurthy commented on issue #12298: Remove model parallelism references from FAQ

2018-09-20 Thread GitBox
sandeep-krishnamurthy commented on issue #12298: Remove model parallelism references from FAQ URL: https://github.com/apache/incubator-mxnet/pull/12298#issuecomment-423409401 @ThomasDelteil - Ping :-) Let me know next steps here. Thanks.

[GitHub] sandeep-krishnamurthy commented on issue #12399: ONNX export: Add Crop, Deconvolution and fix the default stride of Pooling to 1

2018-09-20 Thread GitBox
sandeep-krishnamurthy commented on issue #12399: ONNX export: Add Crop, Deconvolution and fix the default stride of Pooling to 1 URL: https://github.com/apache/incubator-mxnet/pull/12399#issuecomment-423409093 @ptrendx - Thanks for your contributions! Can you please address few review comm

[GitHub] sandeep-krishnamurthy commented on issue #12542: [MXNET-949] Module API to Gluon API tutorial

2018-09-20 Thread GitBox
sandeep-krishnamurthy commented on issue #12542: [MXNET-949] Module API to Gluon API tutorial URL: https://github.com/apache/incubator-mxnet/pull/12542#issuecomment-423408852 @ThomasDelteil - Can you please rebase and resolve conflict? --

[GitHub] sandeep-krishnamurthy commented on issue #12572: Make Gluon download function to be atomic

2018-09-20 Thread GitBox
sandeep-krishnamurthy commented on issue #12572: Make Gluon download function to be atomic URL: https://github.com/apache/incubator-mxnet/pull/12572#issuecomment-423408773 @anirudhacharya / @szha - This is good to go, if your review is addressed. ---

[GitHub] sandeep-krishnamurthy commented on a change in pull request #12622: Gluon DataLoader: avoid recursionlimit error

2018-09-20 Thread GitBox
sandeep-krishnamurthy commented on a change in pull request #12622: Gluon DataLoader: avoid recursionlimit error URL: https://github.com/apache/incubator-mxnet/pull/12622#discussion_r219379367 ## File path: tests/python/unittest/test_gluon_data.py ## @@ -75,24 +75,35 @@ de

[GitHub] ZhennanQin commented on a change in pull request #12530: Implement mkldnn convolution fusion and quantization.

2018-09-20 Thread GitBox
ZhennanQin commented on a change in pull request #12530: Implement mkldnn convolution fusion and quantization. URL: https://github.com/apache/incubator-mxnet/pull/12530#discussion_r219376896 ## File path: src/operator/nn/mkldnn/mkldnn_convolution-inl.h ## @@ -35,19 +36,79

[GitHub] ZhennanQin commented on a change in pull request #12530: Implement mkldnn convolution fusion and quantization.

2018-09-20 Thread GitBox
ZhennanQin commented on a change in pull request #12530: Implement mkldnn convolution fusion and quantization. URL: https://github.com/apache/incubator-mxnet/pull/12530#discussion_r219376787 ## File path: src/operator/subgraph/mkldnn/mkldnn_conv.cc ## @@ -0,0 +1,670 @@ +/*

[GitHub] ZhennanQin commented on a change in pull request #12530: Implement mkldnn convolution fusion and quantization.

2018-09-20 Thread GitBox
ZhennanQin commented on a change in pull request #12530: Implement mkldnn convolution fusion and quantization. URL: https://github.com/apache/incubator-mxnet/pull/12530#discussion_r219376419 ## File path: src/operator/nn/mkldnn/mkldnn_convolution.cc ## @@ -257,32 +300,59 @

[GitHub] stu1130 opened a new pull request #12626: Fix distributed training doc

2018-09-20 Thread GitBox
stu1130 opened a new pull request #12626: Fix distributed training doc URL: https://github.com/apache/incubator-mxnet/pull/12626 ## Description ## * fix several errors in distributed training document * add the link for EFS ## Checklist ## ### Essentials ### Please feel free t

[GitHub] ChaiBapchya commented on a change in pull request #12594: [MXNET-867] Pooling1D with "same" padding

2018-09-20 Thread GitBox
ChaiBapchya commented on a change in pull request #12594: [MXNET-867] Pooling1D with "same" padding URL: https://github.com/apache/incubator-mxnet/pull/12594#discussion_r219361960 ## File path: src/operator/nn/pooling.cc ## @@ -126,8 +126,8 @@ static bool PoolingShape(cons

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

2018-09-20 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 da8db12 Bump the publish ti

[GitHub] stu1130 commented on issue #12624: [MXNET-913] Java API -- NDArray Macros remove Option field

2018-09-20 Thread GitBox
stu1130 commented on issue #12624: [MXNET-913] Java API -- NDArray Macros remove Option field URL: https://github.com/apache/incubator-mxnet/pull/12624#issuecomment-423372116 Thanks for your contribution @lanking520 @mxnet-label-bot[pr-awaiting-review] -

[GitHub] marcoabreu opened a new pull request #12615: Correct PR branch detection in code coverage

2018-09-20 Thread GitBox
marcoabreu opened a new pull request #12615: Correct PR branch detection in code coverage URL: https://github.com/apache/incubator-mxnet/pull/12615 The branch detection in the code coverage script was broken because a non-existing environment variable in groovy is considered null opposed t

[incubator-mxnet] branch fix-testcoverage-report updated (d724c8c -> 4320d32)

2018-09-20 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu pushed a change to branch fix-testcoverage-report in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from d724c8c Always use latest commit add 4320d32 Add merge commit detection No new

[GitHub] marcoabreu closed pull request #12615: Correct PR branch detection in code coverage

2018-09-20 Thread GitBox
marcoabreu closed pull request #12615: Correct PR branch detection in code coverage URL: https://github.com/apache/incubator-mxnet/pull/12615 This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] sandeep-krishnamurthy opened a new pull request #12625: [MXNET-979] [WIP] Add fix_beta support in BatchNorm

2018-09-20 Thread GitBox
sandeep-krishnamurthy opened a new pull request #12625: [MXNET-979] [WIP] Add fix_beta support in BatchNorm URL: https://github.com/apache/incubator-mxnet/pull/12625 ## Description ## Current implementation of Batchnorm does not support ignoring the beta parameter. However, it does supp

[GitHub] lanking520 opened a new pull request #12624: [MXNET-913] Java API -- NDArray Macros remove Option field

2018-09-20 Thread GitBox
lanking520 opened a new pull request #12624: [MXNET-913] Java API -- NDArray Macros remove Option field URL: https://github.com/apache/incubator-mxnet/pull/12624 ## Description ## The Plan 7 discussed in the mxnet Scala Group @nswamy @andrewfayres @yzhliu @zachgk ## Checklist

[GitHub] hetong007 commented on issue #12319: Feature Request: support several statistical operators in NDArray

2018-09-20 Thread GitBox
hetong007 commented on issue #12319: Feature Request: support several statistical operators in NDArray URL: https://github.com/apache/incubator-mxnet/issues/12319#issuecomment-423354023 @lupesko , thanks for your response. My use case was in network quantization, i.e. train the net

[GitHub] zhanghang1989 commented on issue #12623: Feature Request NDArray.has_nan() method

2018-09-20 Thread GitBox
zhanghang1989 commented on issue #12623: Feature Request NDArray.has_nan() method URL: https://github.com/apache/incubator-mxnet/issues/12623#issuecomment-423346538 @mxnet-label-bot [Feature Request, Operator] This is an aut

[GitHub] zhanghang1989 opened a new issue #12623: Feature Request NDArray.has_nan() method

2018-09-20 Thread GitBox
zhanghang1989 opened a new issue #12623: Feature Request NDArray.has_nan() method URL: https://github.com/apache/incubator-mxnet/issues/12623 `ndarray.has_nan(NDArray)` and its convenient form `NDArray.has_nan()`, which return `True` if there is `nan` in the data else return `False`. This

[incubator-mxnet] branch master updated (2becd76 -> 33876c2)

2018-09-20 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 2becd76 [MXNET-860] Update to modern nullptr usage (#12352) add 33876c2 [MXNET-922] Fix memleak i

[GitHub] marcoabreu closed pull request #12499: [MXNET-922] Fix memleak in profiler

2018-09-20 Thread GitBox
marcoabreu closed pull request #12499: [MXNET-922] Fix memleak in profiler URL: https://github.com/apache/incubator-mxnet/pull/12499 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 for

[incubator-mxnet] branch master updated: [MXNET-860] Update to modern nullptr usage (#12352)

2018-09-20 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu 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 2becd76 [MXNET-860] Update to mode

[GitHub] zhreshold opened a new pull request #12622: Gluon DataLoader: avoid recursionlimit error

2018-09-20 Thread GitBox
zhreshold opened a new pull request #12622: Gluon DataLoader: avoid recursionlimit error URL: https://github.com/apache/incubator-mxnet/pull/12622 ## Description ## Avoid RecursionError: maximum recursion depth exceeded in comparison if in some hardware the default value is smaller than

[GitHub] marcoabreu closed pull request #12352: [MXNET-860] Update to modern nullptr usage

2018-09-20 Thread GitBox
marcoabreu closed pull request #12352: [MXNET-860] Update to modern nullptr usage URL: https://github.com/apache/incubator-mxnet/pull/12352 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[GitHub] stu1130 commented on issue #12619: Too large max depth value in _recursive_fork_recordio

2018-09-20 Thread GitBox
stu1130 commented on issue #12619: Too large max depth value in _recursive_fork_recordio URL: https://github.com/apache/incubator-mxnet/issues/12619#issuecomment-423344613 Thanks for submitting the issue @caiqi @mxnet-label-bot [data-loading] --

[GitHub] stu1130 commented on issue #12620: Update and modify Windows docs

2018-09-20 Thread GitBox
stu1130 commented on issue #12620: Update and modify Windows docs URL: https://github.com/apache/incubator-mxnet/pull/12620#issuecomment-423343230 Thanks for your contribution @vrakesh @mxnet-label-bot[pr-awaiting-review]

[GitHub] stu1130 opened a new pull request #12621: Add im2rec docstring

2018-09-20 Thread GitBox
stu1130 opened a new pull request #12621: Add im2rec docstring URL: https://github.com/apache/incubator-mxnet/pull/12621 ## Description ## * update the docstring in [im2rec.py](https://github.com/apache/incubator-mxnet/blob/master/tools/im2rec.py). please refer to #11884 * address th

[GitHub] KellenSunderland commented on a change in pull request #12603: [MXNET-969] Fix buffer overflow in RNNOp

2018-09-20 Thread GitBox
KellenSunderland commented on a change in pull request #12603: [MXNET-969] Fix buffer overflow in RNNOp URL: https://github.com/apache/incubator-mxnet/pull/12603#discussion_r219314629 ## File path: src/operator/rnn-inl.h ## @@ -576,7 +583,7 @@ class RNNOp : public Operator

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

2018-09-20 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 274db1a Bump the publish ti

[GitHub] lupesko commented on issue #12319: Feature Request: support several statistical operators in NDArray

2018-09-20 Thread GitBox
lupesko commented on issue #12319: Feature Request: support several statistical operators in NDArray URL: https://github.com/apache/incubator-mxnet/issues/12319#issuecomment-423316301 @hetong007 - are there examples of models or use cases where these are useful? would help me to see if I

[GitHub] stu1130 commented on issue #12608: [MXNET-913] Java API --- Scala NDArray Macros

2018-09-20 Thread GitBox
stu1130 commented on issue #12608: [MXNET-913] Java API --- Scala NDArray Macros URL: https://github.com/apache/incubator-mxnet/pull/12608#issuecomment-423310770 Thanks for the contribution @lanking520 @mxnet-label-bot[pr-awaiting-review]

[GitHub] apeforest commented on a change in pull request #12603: [MXNET-969] Fix buffer overflow in RNNOp

2018-09-20 Thread GitBox
apeforest commented on a change in pull request #12603: [MXNET-969] Fix buffer overflow in RNNOp URL: https://github.com/apache/incubator-mxnet/pull/12603#discussion_r219286106 ## File path: src/operator/rnn-inl.h ## @@ -576,7 +583,7 @@ class RNNOp : public Operator{

[GitHub] apeforest commented on a change in pull request #12603: [MXNET-969] Fix buffer overflow in RNNOp

2018-09-20 Thread GitBox
apeforest commented on a change in pull request #12603: [MXNET-969] Fix buffer overflow in RNNOp URL: https://github.com/apache/incubator-mxnet/pull/12603#discussion_r219286106 ## File path: src/operator/rnn-inl.h ## @@ -576,7 +583,7 @@ class RNNOp : public Operator{

[incubator-mxnet] branch master updated: update download links to apache distros (#12617)

2018-09-20 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new c479664 update download links to apa

[GitHub] szha closed pull request #12617: update download links to apache distros

2018-09-20 Thread GitBox
szha closed pull request #12617: update download links to apache distros URL: https://github.com/apache/incubator-mxnet/pull/12617 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 forei

[incubator-mxnet] branch master updated: Fixed NoSuchMethodError for Jenkins Job for MBCC (#12618)

2018-09-20 Thread nswamy
This is an automated email from the ASF dual-hosted git repository. nswamy 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 251e11d Fixed NoSuchMethodError for Je

[GitHub] nswamy closed pull request #12618: Fixed NoSuchMethodError for Jenkins Job for MBCC

2018-09-20 Thread GitBox
nswamy closed pull request #12618: Fixed NoSuchMethodError for Jenkins Job for MBCC URL: https://github.com/apache/incubator-mxnet/pull/12618 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 thi

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

2018-09-20 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 0702e02 Bump the publish ti

[GitHub] vrakesh opened a new pull request #12620: Update and modify Windows docs

2018-09-20 Thread GitBox
vrakesh opened a new pull request #12620: Update and modify Windows docs URL: https://github.com/apache/incubator-mxnet/pull/12620 1. Removed sudo from not necessary changes. 2. Added CUDNN setup information link(s). ## Description ## (Brief description on what this PR is about)

[GitHub] caiqi opened a new issue #12619: Too large max depth value in _recursive_fork_recordio

2018-09-20 Thread GitBox
caiqi opened a new issue #12619: Too large max depth value in _recursive_fork_recordio URL: https://github.com/apache/incubator-mxnet/issues/12619 It seems that 1000 is too large for _recursive_fork_recordio in https://github.com/apache/incubator-mxnet/blob/master/python/mxnet/gluon/da

[GitHub] ChaiBapchya commented on a change in pull request #12594: [MXNET-867] Pooling1D with "same" padding

2018-09-20 Thread GitBox
ChaiBapchya commented on a change in pull request #12594: [MXNET-867] Pooling1D with "same" padding URL: https://github.com/apache/incubator-mxnet/pull/12594#discussion_r219262176 ## File path: tests/python/unittest/test_operator.py ## @@ -6957,6 +6957,17 @@ def test_inval

[GitHub] haojin2 commented on a change in pull request #12594: [MXNET-867] Pooling1D with "same" padding

2018-09-20 Thread GitBox
haojin2 commented on a change in pull request #12594: [MXNET-867] Pooling1D with "same" padding URL: https://github.com/apache/incubator-mxnet/pull/12594#discussion_r219257516 ## File path: tests/python/unittest/test_operator.py ## @@ -6957,6 +6957,17 @@ def test_invalid_d

[GitHub] haojin2 commented on a change in pull request #12594: [MXNET-867] Pooling1D with "same" padding

2018-09-20 Thread GitBox
haojin2 commented on a change in pull request #12594: [MXNET-867] Pooling1D with "same" padding URL: https://github.com/apache/incubator-mxnet/pull/12594#discussion_r219257516 ## File path: tests/python/unittest/test_operator.py ## @@ -6957,6 +6957,17 @@ def test_invalid_d

[incubator-mxnet] branch master updated: Remove pip overwrites (#12604)

2018-09-20 Thread indhub
This is an automated email from the ASF dual-hosted git repository. indhub 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 0f8d2d2 Remove pip overwrites (#12604)

[GitHub] indhub closed pull request #12604: Remove pip overwrites

2018-09-20 Thread GitBox
indhub closed pull request #12604: Remove pip overwrites URL: https://github.com/apache/incubator-mxnet/pull/12604 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 pull request

[GitHub] aaronmarkham commented on a change in pull request #12606: Refine the documentation of im2rec

2018-09-20 Thread GitBox
aaronmarkham commented on a change in pull request #12606: Refine the documentation of im2rec URL: https://github.com/apache/incubator-mxnet/pull/12606#discussion_r219236514 ## File path: docs/faq/recordio.md ## @@ -6,35 +6,39 @@ RecordIO implements a file format for a seq

[GitHub] aaronmarkham commented on a change in pull request #12606: Refine the documentation of im2rec

2018-09-20 Thread GitBox
aaronmarkham commented on a change in pull request #12606: Refine the documentation of im2rec URL: https://github.com/apache/incubator-mxnet/pull/12606#discussion_r219236865 ## File path: docs/tutorials/basic/data.md ## @@ -315,6 +315,8 @@ print(mx.recordio.unpack_img(s))

[GitHub] aaronmarkham commented on a change in pull request #12606: Refine the documentation of im2rec

2018-09-20 Thread GitBox
aaronmarkham commented on a change in pull request #12606: Refine the documentation of im2rec URL: https://github.com/apache/incubator-mxnet/pull/12606#discussion_r219011040 ## File path: docs/faq/recordio.md ## @@ -6,35 +6,39 @@ RecordIO implements a file format for a seq

[GitHub] aaronmarkham commented on a change in pull request #12606: Refine the documentation of im2rec

2018-09-20 Thread GitBox
aaronmarkham commented on a change in pull request #12606: Refine the documentation of im2rec URL: https://github.com/apache/incubator-mxnet/pull/12606#discussion_r219236447 ## File path: docs/faq/recordio.md ## @@ -6,35 +6,39 @@ RecordIO implements a file format for a seq

[GitHub] aaronmarkham commented on a change in pull request #12606: Refine the documentation of im2rec

2018-09-20 Thread GitBox
aaronmarkham commented on a change in pull request #12606: Refine the documentation of im2rec URL: https://github.com/apache/incubator-mxnet/pull/12606#discussion_r219237598 ## File path: docs/tutorials/basic/data.md ## @@ -315,6 +315,8 @@ print(mx.recordio.unpack_img(s))

[GitHub] marcoabreu edited a comment on issue #12615: Correct PR branch detection in code coverage

2018-09-20 Thread GitBox
marcoabreu edited a comment on issue #12615: Correct PR branch detection in code coverage URL: https://github.com/apache/incubator-mxnet/pull/12615#issuecomment-423259538 Ah! I found something: These two runs reference the same commit https://github.com/apache/incubator-mxnet/pull/1

[GitHub] marcoabreu commented on issue #12615: Correct PR branch detection in code coverage

2018-09-20 Thread GitBox
marcoabreu commented on issue #12615: Correct PR branch detection in code coverage URL: https://github.com/apache/incubator-mxnet/pull/12615#issuecomment-423259538 Ah! I found something: These two runs reference the same commit https://github.com/apache/incubator-mxnet/pull/12615/co

[GitHub] piyushghai commented on a change in pull request #12618: Fixed NoSuchMethodError for Jenkins Job for MBCC

2018-09-20 Thread GitBox
piyushghai commented on a change in pull request #12618: Fixed NoSuchMethodError for Jenkins Job for MBCC URL: https://github.com/apache/incubator-mxnet/pull/12618#discussion_r219243691 ## File path: tests/nightly/model_backwards_compatibility_check/JenkinsfileForMBCC ## @

[GitHub] marcoabreu commented on issue #12615: Correct PR branch detection in code coverage

2018-09-20 Thread GitBox
marcoabreu commented on issue #12615: Correct PR branch detection in code coverage URL: https://github.com/apache/incubator-mxnet/pull/12615#issuecomment-423258724 It seems like it just broke again. Please don't merge This is

[GitHub] marcoabreu commented on a change in pull request #12618: Fixed NoSuchMethodError for Jenkins Job for MBCC

2018-09-20 Thread GitBox
marcoabreu commented on a change in pull request #12618: Fixed NoSuchMethodError for Jenkins Job for MBCC URL: https://github.com/apache/incubator-mxnet/pull/12618#discussion_r219242928 ## File path: tests/nightly/model_backwards_compatibility_check/JenkinsfileForMBCC ## @

[GitHub] lanking520 edited a comment on issue #12568: [Scala][macOS] Trying to build from source

2018-09-20 Thread GitBox
lanking520 edited a comment on issue #12568: [Scala][macOS] Trying to build from source URL: https://github.com/apache/incubator-mxnet/issues/12568#issuecomment-423255990 I would like to refer to this [issue](https://github.com/tensorflow/tensorflow/issues/6729#issuecomment-272589138). T

[GitHub] lanking520 commented on issue #12568: [Scala][macOS] Trying to build from source

2018-09-20 Thread GitBox
lanking520 commented on issue #12568: [Scala][macOS] Trying to build from source URL: https://github.com/apache/incubator-mxnet/issues/12568#issuecomment-423255990 I would like to refer to this [issue](https://github.com/tensorflow/tensorflow/issues/6729#issuecomment-272589138). TF seemed

[GitHub] kalyc commented on issue #12565: Improve error messages on shape inference (scala)

2018-09-20 Thread GitBox
kalyc commented on issue #12565: Improve error messages on shape inference (scala) URL: https://github.com/apache/incubator-mxnet/issues/12565#issuecomment-423255952 @mdespriee thanks for your contribution. Looks like the PR has been merged. Can this issue be closed?

[GitHub] indhub closed pull request #12607: [MXNet-970] improve tutorial redirection

2018-09-20 Thread GitBox
indhub closed pull request #12607: [MXNet-970] improve tutorial redirection URL: https://github.com/apache/incubator-mxnet/pull/12607 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 fo

[incubator-mxnet] branch master updated: improve tutorial redirection (#12607)

2018-09-20 Thread indhub
This is an automated email from the ASF dual-hosted git repository. indhub 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 ba993d1 improve tutorial redirection (

[GitHub] piyushghai commented on issue #12618: Fixed NoSuchMethodError for Jenkins Job for MBCC

2018-09-20 Thread GitBox
piyushghai commented on issue #12618: Fixed NoSuchMethodError for Jenkins Job for MBCC URL: https://github.com/apache/incubator-mxnet/pull/12618#issuecomment-423251016 @KellenSunderland @marcoabreu This is an automated messa

[GitHub] piyushghai opened a new pull request #12618: Fixed NoSuchMethodError for Jenkins Job for MBCC

2018-09-20 Thread GitBox
piyushghai opened a new pull request #12618: Fixed NoSuchMethodError for Jenkins Job for MBCC URL: https://github.com/apache/incubator-mxnet/pull/12618 ## Description ## A recent change made in PR : #12370 had broken the [Jenkins job for MBCC](http://jenkins.mxnet-ci.amazon-ml.com/job/r

[GitHub] kjchalup commented on issue #11884: im2rec documentation is lacking / buggy

2018-09-20 Thread GitBox
kjchalup commented on issue #11884: im2rec documentation is lacking / buggy URL: https://github.com/apache/incubator-mxnet/issues/11884#issuecomment-423246008 Thank you for the response! 1) The page you linked has a nice example thanks! It'd be great if it was discoverable from [thi

[GitHub] stu1130 commented on issue #12607: [MXNet-970] improve tutorial redirection

2018-09-20 Thread GitBox
stu1130 commented on issue #12607: [MXNet-970] improve tutorial redirection URL: https://github.com/apache/incubator-mxnet/pull/12607#issuecomment-423245454 Thanks for your contribution @aaronmarkham @mxnet-label-bot[pr-awaiting-merge]

[GitHub] stu1130 edited a comment on issue #12614: fix bug, issue 12613

2018-09-20 Thread GitBox
stu1130 edited a comment on issue #12614: fix bug, issue 12613 URL: https://github.com/apache/incubator-mxnet/pull/12614#issuecomment-423245144 Thanks for your contribution @lovickie @mxnet-label-bot[pr-awaiting-review] Th

[GitHub] stu1130 commented on issue #12609: Include missing import in TensorRT tutorial

2018-09-20 Thread GitBox
stu1130 commented on issue #12609: Include missing import in TensorRT tutorial URL: https://github.com/apache/incubator-mxnet/pull/12609#issuecomment-423245317 Thanks for your contribution @KellenSunderland @mxnet-label-bot[pr-awaiting-review] ---

[GitHub] stu1130 commented on issue #12614: fix bug, issue 12613

2018-09-20 Thread GitBox
stu1130 commented on issue #12614: fix bug, issue 12613 URL: https://github.com/apache/incubator-mxnet/pull/12614#issuecomment-423245144 Thanks for your contribution @ @mxnet-label-bot[pr-awaiting-review] This is an automat

[GitHub] stu1130 commented on issue #12617: update download links to apache distros

2018-09-20 Thread GitBox
stu1130 commented on issue #12617: update download links to apache distros URL: https://github.com/apache/incubator-mxnet/pull/12617#issuecomment-423244966 Thanks for your contribution @aaronmarkham @mxnet-label-bot[pr-awaiting-merge]

[GitHub] stu1130 commented on issue #12613: Report a bug of Engine::Get()

2018-09-20 Thread GitBox
stu1130 commented on issue #12613: Report a bug of Engine::Get() URL: https://github.com/apache/incubator-mxnet/issues/12613#issuecomment-423242210 Thanks for submitting the issue @lovickie @mxnet-label-bot [Bug, Backend, C] -

[GitHub] marcoabreu closed pull request #12616: [Test] Fix testcoverage report

2018-09-20 Thread GitBox
marcoabreu closed pull request #12616: [Test] Fix testcoverage report URL: https://github.com/apache/incubator-mxnet/pull/12616 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] marcoabreu commented on issue #12615: [WIP] Correct PR branch detection in code coverage

2018-09-20 Thread GitBox
marcoabreu commented on issue #12615: [WIP] Correct PR branch detection in code coverage URL: https://github.com/apache/incubator-mxnet/pull/12615#issuecomment-423241519 Branch validation successful. Everything's working as expected now :) --

[GitHub] aaronmarkham opened a new pull request #12617: update download links to apache distros

2018-09-20 Thread GitBox
aaronmarkham opened a new pull request #12617: update download links to apache distros URL: https://github.com/apache/incubator-mxnet/pull/12617 ## Description ## This PR updates the download links to use the Apache distro servers. ---

[GitHub] stu1130 commented on issue #12612: cannot converge use the dcgan with mnist and cifar10 dataset?

2018-09-20 Thread GitBox
stu1130 commented on issue #12612: cannot converge use the dcgan with mnist and cifar10 dataset? URL: https://github.com/apache/incubator-mxnet/issues/12612#issuecomment-423238553 Thanks for submitting the issue @pengxin99 @mxnet-label-bot [question, example] You can also post the q

[GitHub] stu1130 commented on issue #12611: Create docker images with jupyter notebook along with sample notebook

2018-09-20 Thread GitBox
stu1130 commented on issue #12611: Create docker images with jupyter notebook along with sample notebook URL: https://github.com/apache/incubator-mxnet/issues/12611#issuecomment-423237245 Thanks for submitting the issue @gautamkmr @mxnet-label-bot [Feature request, Docker] ---

[GitHub] marcoabreu opened a new pull request #12615: [WIP] Correct PR branch detection in code coverage

2018-09-20 Thread GitBox
marcoabreu opened a new pull request #12615: [WIP] Correct PR branch detection in code coverage URL: https://github.com/apache/incubator-mxnet/pull/12615 The branch detection in the code coverage script was broken because a non-existing environment variable in groovy is considered null opp

[GitHub] marcoabreu commented on issue #12615: [WIP] Correct PR branch detection in code coverage

2018-09-20 Thread GitBox
marcoabreu commented on issue #12615: [WIP] Correct PR branch detection in code coverage URL: https://github.com/apache/incubator-mxnet/pull/12615#issuecomment-423231593 I have verified that this change https://github.com/apache/incubator-mxnet/pull/12615/commits/d724c8cbffb90bb4ba64397ead

[GitHub] marcoabreu closed pull request #12615: [WIP] Correct PR branch detection in code coverage

2018-09-20 Thread GitBox
marcoabreu closed pull request #12615: [WIP] Correct PR branch detection in code coverage URL: https://github.com/apache/incubator-mxnet/pull/12615 This is an automated message from the Apache Git Service. To respond to the

[GitHub] marcoabreu commented on issue #12615: [WIP] Correct PR branch detection in code coverage

2018-09-20 Thread GitBox
marcoabreu commented on issue #12615: [WIP] Correct PR branch detection in code coverage URL: https://github.com/apache/incubator-mxnet/pull/12615#issuecomment-423228794 Closing to trigger branch job This is an automated mess

[incubator-mxnet] branch master updated: review require() usages to add meaningful messages. (#12570)

2018-09-20 Thread nswamy
This is an automated email from the ASF dual-hosted git repository. nswamy 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 3401e6e review require() usages to add

[GitHub] nswamy closed pull request #12570: review require() usages to add meaningful messages.

2018-09-20 Thread GitBox
nswamy closed pull request #12570: review require() usages to add meaningful messages. URL: https://github.com/apache/incubator-mxnet/pull/12570 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

  1   2   >