[GitHub] szha commented on issue #14053: in-place reshape ops

2019-02-03 Thread GitBox
szha commented on issue #14053: in-place reshape ops URL: https://github.com/apache/incubator-mxnet/pull/14053#issuecomment-460086429 @stephenrawls > This fix only effects the Python version when operating on ndarrays, not users of the expand_dims operator. Right. The reshape

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

2019-02-03 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 2073618 Bump the publish

[GitHub] sergeykolychev commented on a change in pull request #13697: [MKLDNN] Enable signed int8 support for convolution.

2019-02-03 Thread GitBox
sergeykolychev commented on a change in pull request #13697: [MKLDNN] Enable signed int8 support for convolution. URL: https://github.com/apache/incubator-mxnet/pull/13697#discussion_r243388983 ## File path: include/mxnet/c_api.h ## @@ -1833,10 +1832,12 @@ MXNET_DLL int

[GitHub] stephenrawls commented on issue #14053: in-place reshape ops

2019-02-03 Thread GitBox
stephenrawls commented on issue #14053: in-place reshape ops URL: https://github.com/apache/incubator-mxnet/pull/14053#issuecomment-460090968 @szha -- Thanks for the detailed explanation, that helps a lot. So just for clarity, I see that in the operator definition for expand_dims,

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

2019-02-03 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 73f4d7b Bump the publish

[GitHub] szha commented on a change in pull request #13896: Cudnn dropout

2019-02-03 Thread GitBox
szha commented on a change in pull request #13896: Cudnn dropout URL: https://github.com/apache/incubator-mxnet/pull/13896#discussion_r253315601 ## File path: src/operator/nn/dropout.cc ## @@ -119,25 +119,43 @@ Example:: for (size_t i = 0; i < nout; ++i)

[GitHub] KellenSunderland commented on issue #13265: MXNet C++Interface reasoning leads to CPU memory leak

2019-02-03 Thread GitBox
KellenSunderland commented on issue #13265: MXNet C++Interface reasoning leads to CPU memory leak URL: https://github.com/apache/incubator-mxnet/issues/13265#issuecomment-460084470 I tried to reproduce the memory leak with an ASAN build and don't see anything that jumps out as a leak.

[GitHub] KellenSunderland edited a comment on issue #13265: MXNet C++Interface reasoning leads to CPU memory leak

2019-02-03 Thread GitBox
KellenSunderland edited a comment on issue #13265: MXNet C++Interface reasoning leads to CPU memory leak URL: https://github.com/apache/incubator-mxnet/issues/13265#issuecomment-460084470 I tried to reproduce the memory leak with an ASAN build and don't see anything that jumps out as a

[GitHub] stephenrawls opened a new issue #14062: Bug in Hybridize w/ Concat

2019-02-03 Thread GitBox
stephenrawls opened a new issue #14062: Bug in Hybridize w/ Concat URL: https://github.com/apache/incubator-mxnet/issues/14062 Hi, I think I found a bug in the Concat operator after calling hybridize() on my network. It is possible it is a mis-understanding on my part about how

[GitHub] mxnet-label-bot commented on issue #14062: Bug in Hybridize w/ Concat

2019-02-03 Thread GitBox
mxnet-label-bot commented on issue #14062: Bug in Hybridize w/ Concat URL: https://github.com/apache/incubator-mxnet/issues/14062#issuecomment-460078378 Hey, this is the MXNet Label Bot. Thank you for submitting the issue! I will try and suggest some labels so that the appropriate

[GitHub] szha commented on a change in pull request #13896: Cudnn dropout

2019-02-03 Thread GitBox
szha commented on a change in pull request #13896: Cudnn dropout URL: https://github.com/apache/incubator-mxnet/pull/13896#discussion_r253315569 ## File path: src/operator/nn/dropout-inl.h ## @@ -39,12 +39,15 @@ #include "../random/sampler.h" #include

[GitHub] reminisce commented on issue #13697: [MKLDNN] Enable signed int8 support for convolution.

2019-02-03 Thread GitBox
reminisce commented on issue #13697: [MKLDNN] Enable signed int8 support for convolution. URL: https://github.com/apache/incubator-mxnet/pull/13697#issuecomment-460092314 @ZhennanQin This PR has touched the code shared by GPU and CPU quantization. It's better to verify that no bugs have

[GitHub] reminisce commented on a change in pull request #13697: [MKLDNN] Enable signed int8 support for convolution.

2019-02-03 Thread GitBox
reminisce commented on a change in pull request #13697: [MKLDNN] Enable signed int8 support for convolution. URL: https://github.com/apache/incubator-mxnet/pull/13697#discussion_r253321241 ## File path: python/mxnet/contrib/quantization.py ## @@ -286,12 +283,12 @@ def

[GitHub] mseth10 commented on issue #14016: fix test_depthwise_convoltuion for occasional CI failures

2019-02-03 Thread GitBox
mseth10 commented on issue #14016: fix test_depthwise_convoltuion for occasional CI failures URL: https://github.com/apache/incubator-mxnet/pull/14016#issuecomment-460038319 Filed another issue #14052 for MKLDNN bug that causes CI failure when test is enabled. Keeping the test disabled as

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

2019-02-03 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 b701907 Bump the publish

[GitHub] TaoLv commented on a change in pull request #13896: Cudnn dropout

2019-02-03 Thread GitBox
TaoLv commented on a change in pull request #13896: Cudnn dropout URL: https://github.com/apache/incubator-mxnet/pull/13896#discussion_r253305336 ## File path: src/operator/nn/dropout-inl.h ## @@ -39,12 +39,15 @@ #include "../random/sampler.h" #include

[GitHub] TaoLv commented on a change in pull request #13896: Cudnn dropout

2019-02-03 Thread GitBox
TaoLv commented on a change in pull request #13896: Cudnn dropout URL: https://github.com/apache/incubator-mxnet/pull/13896#discussion_r253305706 ## File path: src/operator/nn/dropout.cc ## @@ -119,25 +119,43 @@ Example:: for (size_t i = 0; i < nout; ++i)

[GitHub] TaoLv commented on a change in pull request #13896: Cudnn dropout

2019-02-03 Thread GitBox
TaoLv commented on a change in pull request #13896: Cudnn dropout URL: https://github.com/apache/incubator-mxnet/pull/13896#discussion_r253305372 ## File path: src/resource.cc ## @@ -92,11 +93,14 @@ class ResourceManagerImpl : public ResourceManager {

[GitHub] TaoLv commented on a change in pull request #13896: Cudnn dropout

2019-02-03 Thread GitBox
TaoLv commented on a change in pull request #13896: Cudnn dropout URL: https://github.com/apache/incubator-mxnet/pull/13896#discussion_r253305315 ## File path: src/operator/nn/dropout-inl.h ## @@ -39,12 +39,15 @@ #include "../random/sampler.h" #include

[GitHub] jasperDD opened a new issue #14061: error when try resize on pixel video to perform image-recognition analysis in R

2019-02-03 Thread GitBox
jasperDD opened a new issue #14061: error when try resize on pixel video to perform image-recognition analysis in R URL: https://github.com/apache/incubator-mxnet/issues/14061 I want to create the predicative model in line with the image processing, I have a lot of video

[GitHub] szha opened a new pull request #14063: fix test_stn

2019-02-03 Thread GitBox
szha opened a new pull request #14063: fix test_stn URL: https://github.com/apache/incubator-mxnet/pull/14063 ## Description ## fix test_stn display setting which likely have been broken by dependency change. This is an

[GitHub] szha commented on issue #14063: fix test_stn

2019-02-03 Thread GitBox
szha commented on issue #14063: fix test_stn URL: https://github.com/apache/incubator-mxnet/pull/14063#issuecomment-460120521 @apache/mxnet-committers your attention is appreciated as the problem fixed in this PR is blocking the CI builds for other PRs.

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

2019-02-03 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 d64a672 Bump the publish

[GitHub] pengzhao-intel commented on issue #13697: [MKLDNN] Enable signed int8 support for convolution.

2019-02-03 Thread GitBox
pengzhao-intel commented on issue #13697: [MKLDNN] Enable signed int8 support for convolution. URL: https://github.com/apache/incubator-mxnet/pull/13697#issuecomment-460113921 Thanks for the suggestions and comments @szha @reminisce @sergeykolychev We are on the Chinese holiday now and

[GitHub] ZhennanQin commented on a change in pull request #13697: [MKLDNN] Enable signed int8 support for convolution.

2019-02-03 Thread GitBox
ZhennanQin commented on a change in pull request #13697: [MKLDNN] Enable signed int8 support for convolution. URL: https://github.com/apache/incubator-mxnet/pull/13697#discussion_r253335753 ## File path: include/mxnet/c_api.h ## @@ -1833,10 +1832,12 @@ MXNET_DLL int

[GitHub] KellenSunderland commented on issue #14063: fix test_stn

2019-02-03 Thread GitBox
KellenSunderland commented on issue #14063: fix test_stn URL: https://github.com/apache/incubator-mxnet/pull/14063#issuecomment-460122746 LGTM, thanks for fixing this @szha. This is an automated message from the Apache Git

[incubator-mxnet] branch master updated: fix test_stn (#14063)

2019-02-03 Thread kellen
This is an automated email from the ASF dual-hosted git repository. kellen pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new d02beeb fix test_stn (#14063) d02beeb

[GitHub] KellenSunderland merged pull request #14063: fix test_stn

2019-02-03 Thread GitBox
KellenSunderland merged pull request #14063: fix test_stn URL: https://github.com/apache/incubator-mxnet/pull/14063 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] ZhennanQin commented on a change in pull request #13697: [MKLDNN] Enable signed int8 support for convolution.

2019-02-03 Thread GitBox
ZhennanQin commented on a change in pull request #13697: [MKLDNN] Enable signed int8 support for convolution. URL: https://github.com/apache/incubator-mxnet/pull/13697#discussion_r253343906 ## File path: include/mxnet/c_api.h ## @@ -1833,10 +1832,12 @@ MXNET_DLL int

[incubator-mxnet] branch szha-patch-2 deleted (was e1fac7e)

2019-02-03 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a change to branch szha-patch-2 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. was e1fac7e fix test_stn The revisions that were on this branch are still contained in other

[incubator-mxnet] branch szha-patch-1 updated (32c7f63 -> edde677)

2019-02-03 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a change to branch szha-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. discard 32c7f63 Update NOTICE new d02beeb fix test_stn (#14063) new edde677 Update NOTICE

[GitHub] ZhennanQin commented on issue #13697: [MKLDNN] Enable signed int8 support for convolution.

2019-02-03 Thread GitBox
ZhennanQin commented on issue #13697: [MKLDNN] Enable signed int8 support for convolution. URL: https://github.com/apache/incubator-mxnet/pull/13697#issuecomment-460126456 @reminisce As you have concern about entropy change, I removed the parts that for uint8 fix, as it's not the major

[GitHub] ZhennanQin commented on a change in pull request #13697: [MKLDNN] Enable signed int8 support for convolution.

2019-02-03 Thread GitBox
ZhennanQin commented on a change in pull request #13697: [MKLDNN] Enable signed int8 support for convolution. URL: https://github.com/apache/incubator-mxnet/pull/13697#discussion_r253345880 ## File path: python/mxnet/contrib/quantization.py ## @@ -286,12 +283,12 @@ def

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

2019-02-03 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 330ec4f Bump the publish