[GitHub] eric-haibin-lin commented on issue #11319: Static alloc for hybridblock

2018-06-15 Thread GitBox
eric-haibin-lin commented on issue #11319: Static alloc for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/11319#issuecomment-397789580 @marcoabreu My work also depends on this PR. https://github.com/apache/incubator-mxnet/pull/11306

[GitHub] piiswrong opened a new pull request #11320: Static alloc for hybridblock

2018-06-15 Thread GitBox
piiswrong opened a new pull request #11320: Static alloc for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/11320 This reverts commit 9b8eb567273d9521c639ea783e1117fe09cb3d64. ## Description ## (Brief description on what this PR is about) ## Checklist ##

[GitHub] chenbiaolong commented on issue #8474: How to train variable input shape CNN?

2018-06-15 Thread GitBox
chenbiaolong commented on issue #8474: How to train variable input shape CNN? URL: https://github.com/apache/incubator-mxnet/issues/8474#issuecomment-397787011 @lanking520 @marcoabreu We found MXNet is convenient in c++ deployment,especially when we want to improve our throughputs. The

[GitHub] tqchen commented on issue #11307: Memory Management

2018-06-15 Thread GitBox
tqchen commented on issue #11307: Memory Management URL: https://github.com/apache/incubator-mxnet/issues/11307#issuecomment-397786640 Just putting some comment about our existing effort here so that we don't waste effort one things already tried. If there is really a solution out of

[GitHub] TaoLv commented on issue #11262: [MXNET-542] Fix mkldnn performance regression + improve test logging

2018-06-15 Thread GitBox
TaoLv commented on issue #11262: [MXNET-542] Fix mkldnn performance regression + improve test logging URL: https://github.com/apache/incubator-mxnet/pull/11262#issuecomment-397785653 I verifed the performance of commit 83ce4f0 with inference of ResNet50, VGG16 and Inception-v3. It looks

[GitHub] haojin2 commented on issue #11319: Static alloc for hybridblock

2018-06-15 Thread GitBox
haojin2 commented on issue #11319: Static alloc for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/11319#issuecomment-397786323 @marcoabreu Can we bring this commit back? I've got some work (#11179) that depends of part of this PR.

[GitHub] haojin2 commented on issue #11319: Static alloc for hybridblock

2018-06-15 Thread GitBox
haojin2 commented on issue #11319: Static alloc for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/11319#issuecomment-397786323 @marcoabreu Can we bring this commit back? I've got some ongoing work (#11179) that depends of part of this PR.

[GitHub] chenbiaolong commented on issue #8474: How to train variable input shape CNN?

2018-06-15 Thread GitBox
chenbiaolong commented on issue #8474: How to train variable input shape CNN? URL: https://github.com/apache/incubator-mxnet/issues/8474#issuecomment-397514469 @lanking520 the main framework we use to train models is Tensorflow now.In last year we use tensorflow train variable shape

[GitHub] TaoLv commented on issue #11262: [MXNET-542] Fix mkldnn performance regression + improve test logging

2018-06-15 Thread GitBox
TaoLv commented on issue #11262: [MXNET-542] Fix mkldnn performance regression + improve test logging URL: https://github.com/apache/incubator-mxnet/pull/11262#issuecomment-397785653 I verifed the performance of commit 83ce4f0. It looks good to me, compared with commit 9514a1e.

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

2018-06-15 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-397784515 @kalyc this is python ssd 's log.

[GitHub] larroy commented on issue #11302: [MXNET-553] Restructure dockcross dockerfiles to fix caching

2018-06-15 Thread GitBox
larroy commented on issue #11302: [MXNET-553] Restructure dockcross dockerfiles to fix caching URL: https://github.com/apache/incubator-mxnet/pull/11302#issuecomment-397782922 LGTM I can try what I asked myself as I'm not sure my question was well understood.

[GitHub] tqchen commented on issue #11313: Static alloc for hybridblock

2018-06-15 Thread GitBox
tqchen commented on issue #11313: Static alloc for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/11313#issuecomment-397782646 just to be fair, I dismissed the your comment with specific comment that addresses your issue

[GitHub] TaoLv commented on a change in pull request #11262: [MXNET-542] Fix mkldnn performance regression + improve test logging

2018-06-15 Thread GitBox
TaoLv commented on a change in pull request #11262: [MXNET-542] Fix mkldnn performance regression + improve test logging URL: https://github.com/apache/incubator-mxnet/pull/11262#discussion_r195892008 ## File path: src/operator/nn/mkldnn/mkldnn_base-inl.h ## @@ -327,7

[GitHub] TaoLv commented on a change in pull request #11262: [MXNET-542] Fix mkldnn performance regression + improve test logging

2018-06-15 Thread GitBox
TaoLv commented on a change in pull request #11262: [MXNET-542] Fix mkldnn performance regression + improve test logging URL: https://github.com/apache/incubator-mxnet/pull/11262#discussion_r195891771 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -77,26 +77,35

[GitHub] TaoLv commented on a change in pull request #11262: [MXNET-542] Fix mkldnn performance regression + improve test logging

2018-06-15 Thread GitBox
TaoLv commented on a change in pull request #11262: [MXNET-542] Fix mkldnn performance regression + improve test logging URL: https://github.com/apache/incubator-mxnet/pull/11262#discussion_r195891794 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -77,26 +77,35

[GitHub] TaoLv commented on a change in pull request #11262: [MXNET-542] Fix mkldnn performance regression + improve test logging

2018-06-15 Thread GitBox
TaoLv commented on a change in pull request #11262: [MXNET-542] Fix mkldnn performance regression + improve test logging URL: https://github.com/apache/incubator-mxnet/pull/11262#discussion_r195891904 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -77,26 +77,35

[GitHub] TaoLv commented on a change in pull request #11262: [MXNET-542] Fix mkldnn performance regression + improve test logging

2018-06-15 Thread GitBox
TaoLv commented on a change in pull request #11262: [MXNET-542] Fix mkldnn performance regression + improve test logging URL: https://github.com/apache/incubator-mxnet/pull/11262#discussion_r195891782 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -77,26 +77,35

[GitHub] TaoLv commented on a change in pull request #11262: [MXNET-542] Fix mkldnn performance regression + improve test logging

2018-06-15 Thread GitBox
TaoLv commented on a change in pull request #11262: [MXNET-542] Fix mkldnn performance regression + improve test logging URL: https://github.com/apache/incubator-mxnet/pull/11262#discussion_r195891956 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -77,26 +77,35

[GitHub] TaoLv commented on a change in pull request #11262: [MXNET-542] Fix mkldnn performance regression + improve test logging

2018-06-15 Thread GitBox
TaoLv commented on a change in pull request #11262: [MXNET-542] Fix mkldnn performance regression + improve test logging URL: https://github.com/apache/incubator-mxnet/pull/11262#discussion_r195891865 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -77,26 +77,35

[GitHub] marcoabreu commented on issue #11313: Static alloc for hybridblock

2018-06-15 Thread GitBox
marcoabreu commented on issue #11313: Static alloc for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/11313#issuecomment-397780181 I was not the one who closed this PR, it was @tqchen. He was the one who dismissed my review and blocked other people from reviewing it. So

[GitHub] TaoLv commented on issue #11262: [MXNET-542] Fix mkldnn performance regression + improve test logging

2018-06-15 Thread GitBox
TaoLv commented on issue #11262: [MXNET-542] Fix mkldnn performance regression + improve test logging URL: https://github.com/apache/incubator-mxnet/pull/11262#issuecomment-397780096 @azai91 Yes, please see the comments and description in

[GitHub] zheng-da commented on issue #11313: Static alloc for hybridblock

2018-06-15 Thread GitBox
zheng-da commented on issue #11313: Static alloc for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/11313#issuecomment-397779855 I don't understand what the problem is. Because you veto, the PR should never be merged? We should open the PR and get proper reviews and

[GitHub] marcoabreu commented on issue #11318: Revert "Static alloc for hybridblock"

2018-06-15 Thread GitBox
marcoabreu commented on issue #11318: Revert "Static alloc for hybridblock" URL: https://github.com/apache/incubator-mxnet/pull/11318#issuecomment-397779766 Exactly, that's all I'm asking for. Give the community enough time to review the PR and address their concerns before merging it.

[GitHub] zheng-da commented on issue #11318: Revert "Static alloc for hybridblock"

2018-06-15 Thread GitBox
zheng-da commented on issue #11318: Revert "Static alloc for hybridblock" URL: https://github.com/apache/incubator-mxnet/pull/11318#issuecomment-397779690 @marcoabreu I think you should at least give people who understand the PR the opportunity to review it.

[GitHub] zheng-da commented on issue #11318: Revert "Static alloc for hybridblock"

2018-06-15 Thread GitBox
zheng-da commented on issue #11318: Revert "Static alloc for hybridblock" URL: https://github.com/apache/incubator-mxnet/pull/11318#issuecomment-397779615 @marcoabreu I have been intensively using CachedOp with static memory allocation this week. I also read the code this week. Other than

[GitHub] marcoabreu commented on issue #11313: Static alloc for hybridblock

2018-06-15 Thread GitBox
marcoabreu commented on issue #11313: Static alloc for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/11313#issuecomment-397779609 Hi Da, thanks for your comments. I'm sure that CachedOp is behaving as expected. The problems are described at

[GitHub] zheng-da commented on issue #11313: Static alloc for hybridblock

2018-06-15 Thread GitBox
zheng-da commented on issue #11313: Static alloc for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/11313#issuecomment-397779383 @marcoabreu I have been intensively using CachedOp with static memory allocation this week. I also read the code in the past week. Other than

[incubator-mxnet] branch piiswrong-patch-3 deleted (was 29faa9f)

2018-06-15 Thread tqchen
This is an automated email from the ASF dual-hosted git repository. tqchen pushed a change to branch piiswrong-patch-3 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. was 29faa9f fix The revisions that were on this branch are still contained in other references;

[incubator-mxnet] branch piiswrong-patch-2 deleted (was fcdca35)

2018-06-15 Thread tqchen
This is an automated email from the ASF dual-hosted git repository. tqchen pushed a change to branch piiswrong-patch-2 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. was fcdca35 Update cached_op.cc The revisions that were on this branch are still contained in

[incubator-mxnet] branch revert-11318-revert-11313-static deleted (was c8a9374)

2018-06-15 Thread tqchen
This is an automated email from the ASF dual-hosted git repository. tqchen pushed a change to branch revert-11318-revert-11313-static in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. was c8a9374 Revert "Revert "Static alloc for hybridblock (#11313)" (#11318)" The

[GitHub] tqchen closed pull request #11319: Static alloc for hybridblock

2018-06-15 Thread GitBox
tqchen closed pull request #11319: Static alloc for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/11319 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] tqchen opened a new pull request #11319: Static alloc for hybridblock

2018-06-15 Thread GitBox
tqchen opened a new pull request #11319: Static alloc for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/11319 Bring back the changes of hybrid-blocks. The test coverage issues has been addressed. This is an

[GitHub] tqchen commented on issue #11318: Revert "Static alloc for hybridblock"

2018-06-15 Thread GitBox
tqchen commented on issue #11318: Revert "Static alloc for hybridblock" URL: https://github.com/apache/incubator-mxnet/pull/11318#issuecomment-397778667 This PR is already fixing the problems and added test coverages. If we just block this, how can we move forward?

[GitHub] ThomasDelteil commented on issue #11318: Revert "Static alloc for hybridblock"

2018-06-15 Thread GitBox
ThomasDelteil commented on issue #11318: Revert "Static alloc for hybridblock" URL: https://github.com/apache/incubator-mxnet/pull/11318#issuecomment-397778642 I think I have raised specific concerns (twice) that have gone unanswered, just in case they got lost in the flow, let me write

[GitHub] tqchen commented on issue #11318: Revert "Static alloc for hybridblock"

2018-06-15 Thread GitBox
tqchen commented on issue #11318: Revert "Static alloc for hybridblock" URL: https://github.com/apache/incubator-mxnet/pull/11318#issuecomment-397778667 This PR is already fixing the problems and added test coverages. If we just block this, how can we move forward?

[GitHub] tqchen commented on issue #11318: Revert "Static alloc for hybridblock"

2018-06-15 Thread GitBox
tqchen commented on issue #11318: Revert "Static alloc for hybridblock" URL: https://github.com/apache/incubator-mxnet/pull/11318#issuecomment-397778646 This PR is already fixing the problems and added test coverages. If we just block this, how can we move forward?

[GitHub] marcoabreu closed pull request #11319: Static alloc for hybridblock

2018-06-15 Thread GitBox
marcoabreu closed pull request #11319: Static alloc for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/11319 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] marcoabreu commented on issue #11318: Revert "Static alloc for hybridblock"

2018-06-15 Thread GitBox
marcoabreu commented on issue #11318: Revert "Static alloc for hybridblock" URL: https://github.com/apache/incubator-mxnet/pull/11318#issuecomment-397778459 https://github.com/apache/incubator-mxnet/issues/11171 has not been addressed Pedros review in

[GitHub] tqchen opened a new pull request #11319: Static alloc for hybridblock

2018-06-15 Thread GitBox
tqchen opened a new pull request #11319: Static alloc for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/11319 Bring back the changes of hybrid-blocks. The test coverage issues has been addressed. This is an

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

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

[GitHub] tqchen commented on issue #11318: Revert "Static alloc for hybridblock"

2018-06-15 Thread GitBox
tqchen commented on issue #11318: Revert "Static alloc for hybridblock" URL: https://github.com/apache/incubator-mxnet/pull/11318#issuecomment-39458 If you have specific concerns, raise them. Your concern of lack of test coverage has been addressed, and you are abusing your right as

[GitHub] marcoabreu commented on issue #11318: Revert "Static alloc for hybridblock"

2018-06-15 Thread GitBox
marcoabreu commented on issue #11318: Revert "Static alloc for hybridblock" URL: https://github.com/apache/incubator-mxnet/pull/11318#issuecomment-39312 My as well as Thomas' concerns have not been addressed. This is an

[GitHub] tqchen commented on issue #11318: Revert "Static alloc for hybridblock"

2018-06-15 Thread GitBox
tqchen commented on issue #11318: Revert "Static alloc for hybridblock" URL: https://github.com/apache/incubator-mxnet/pull/11318#issuecomment-39148 it seems you have been using veto too often lately, please @marcoabreu we are not being unreasonable here, your concern about testcases

[GitHub] marcoabreu closed pull request #11318: Revert "Static alloc for hybridblock"

2018-06-15 Thread GitBox
marcoabreu closed pull request #11318: Revert "Static alloc for hybridblock" URL: https://github.com/apache/incubator-mxnet/pull/11318 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] tqchen closed pull request #11318: Revert "Static alloc for hybridblock"

2018-06-15 Thread GitBox
tqchen closed pull request #11318: Revert "Static alloc for hybridblock" URL: https://github.com/apache/incubator-mxnet/pull/11318 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] marcoabreu opened a new pull request #11318: Revert "Static alloc for hybridblock"

2018-06-15 Thread GitBox
marcoabreu opened a new pull request #11318: Revert "Static alloc for hybridblock" URL: https://github.com/apache/incubator-mxnet/pull/11318 Reverts apache/incubator-mxnet#11313 My veto has been ignored and thus I'm allowed to revert this PR. See

[GitHub] tqchen commented on issue #11318: Revert "Static alloc for hybridblock"

2018-06-15 Thread GitBox
tqchen commented on issue #11318: Revert "Static alloc for hybridblock" URL: https://github.com/apache/incubator-mxnet/pull/11318#issuecomment-39148 it seems p This is an automated message from the Apache Git Service. To

[GitHub] marcoabreu opened a new pull request #11318: Revert "Static alloc for hybridblock"

2018-06-15 Thread GitBox
marcoabreu opened a new pull request #11318: Revert "Static alloc for hybridblock" URL: https://github.com/apache/incubator-mxnet/pull/11318 Reverts apache/incubator-mxnet#11313 My veto has been ignored and thus I'm allowed to revert this PR. See

[incubator-mxnet] branch revert-11313-static created (now 24f14f7)

2018-06-15 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu pushed a change to branch revert-11313-static in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. at 24f14f7 Revert "Static alloc for hybridblock (#11313)" This branch includes the

[incubator-mxnet] 01/01: Revert "Static alloc for hybridblock (#11313)"

2018-06-15 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu pushed a commit to branch revert-11313-static in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit 24f14f7fdbef427aab25738608267c57de645987 Author: Marco de Abreu AuthorDate: Fri Jun 15

[incubator-mxnet] branch master updated: Static alloc for hybridblock (#11313)

2018-06-15 Thread tqchen
This is an automated email from the ASF dual-hosted git repository. tqchen 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 5431e12 Static alloc for hybridblock

[GitHub] tqchen commented on issue #11307: Memory Management

2018-06-15 Thread GitBox
tqchen commented on issue #11307: Memory Management URL: https://github.com/apache/incubator-mxnet/issues/11307#issuecomment-397774432 I think @yzhliu knows more details, the original design of scala was done in an implicit way by doing disposable at finalizer (like what you suggested).

[GitHub] tqchen commented on issue #11307: Memory Management

2018-06-15 Thread GitBox
tqchen commented on issue #11307: Memory Management URL: https://github.com/apache/incubator-mxnet/issues/11307#issuecomment-397774432 I think @yzhliu knows more details, the original design of scala was done in an implicit way by doing disposable at finalizer. The problem is

[GitHub] nswamy commented on issue #11307: Memory Management

2018-06-15 Thread GitBox
nswamy commented on issue #11307: Memory Management URL: https://github.com/apache/incubator-mxnet/issues/11307#issuecomment-397773700 We could explore using WeakReferences like how ND4J does or use ObjectPoo. Scala Implicits helps to remove the ugliness of boiler plate code. I'll write

[GitHub] marcoabreu commented on a change in pull request #11294: [MXNET-543][WIP]Scala Spark test

2018-06-15 Thread GitBox
marcoabreu commented on a change in pull request #11294: [MXNET-543][WIP]Scala Spark test URL: https://github.com/apache/incubator-mxnet/pull/11294#discussion_r195887039 ## File path: scala-package/spark/src/test/scala/org/apache/mxnet/spark/MXNetGeneralSuite.scala ## @@

[GitHub] tqchen commented on issue #11307: Memory Management

2018-06-15 Thread GitBox
tqchen commented on issue #11307: Memory Management URL: https://github.com/apache/incubator-mxnet/issues/11307#issuecomment-397770425 The fact is you can't. Unfortunately, we have to blame JVM in this case, which does not have ARC(accurate ref counting). The gc works well for the

[incubator-mxnet] branch v1.2.0 updated: Fix discard_stderr (#11309)

2018-06-15 Thread anirudh2290
This is an automated email from the ASF dual-hosted git repository. anirudh2290 pushed a commit to branch v1.2.0 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/v1.2.0 by this push: new daa957c Fix discard_stderr

[GitHub] anirudh2290 closed pull request #11309: Backport fix discard_stderr

2018-06-15 Thread GitBox
anirudh2290 closed pull request #11309: Backport fix discard_stderr URL: https://github.com/apache/incubator-mxnet/pull/11309 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] safrooze commented on issue #9881: Inconsistent weight decay logics in multiple optimizers

2018-06-15 Thread GitBox
safrooze commented on issue #9881: Inconsistent weight decay logics in multiple optimizers URL: https://github.com/apache/incubator-mxnet/issues/9881#issuecomment-397769492 Also there is this paper that discusses how weight_decay must be applied independently from optimizer:

[incubator-mxnet] branch master updated: [MXNET-539] Allow Scala users to specify data/label names for NDArrayIter (#11256)

2018-06-15 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 02e8a71 [MXNET-539] Allow Scala users

[GitHub] nswamy closed pull request #11256: [MXNET-539] Allow Scala users to specify data/label names for NDArrayIter

2018-06-15 Thread GitBox
nswamy closed pull request #11256: [MXNET-539] Allow Scala users to specify data/label names for NDArrayIter URL: https://github.com/apache/incubator-mxnet/pull/11256 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] lanking520 commented on a change in pull request #11294: [MXNET-543][WIP]Scala Spark test

2018-06-15 Thread GitBox
lanking520 commented on a change in pull request #11294: [MXNET-543][WIP]Scala Spark test URL: https://github.com/apache/incubator-mxnet/pull/11294#discussion_r195884144 ## File path: scala-package/spark/src/test/scala/org/apache/mxnet/spark/MXNetGeneralSuite.scala ## @@

[incubator-mxnet] 01/04: Add NEWS and README

2018-06-15 Thread anirudh2290
This is an automated email from the ASF dual-hosted git repository. anirudh2290 pushed a commit to branch v1.2.0 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit f6a8cb76b7e182e86c4c9a382421baba56bd6cb4 Author: Anirudh Subramanian AuthorDate: Fri Jun 15 00:15:06 2018

[incubator-mxnet] 04/04: [MXNET-532] Clarify documentation of save_parameters(), load_parameters() (#11210)

2018-06-15 Thread anirudh2290
This is an automated email from the ASF dual-hosted git repository. anirudh2290 pushed a commit to branch v1.2.0 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit 332056a3413d8065adf7ede5d6a789954b3a8326 Author: Thomas Delteil AuthorDate: Thu Jun 14 14:10:11 2018

[incubator-mxnet] branch v1.2.0 updated (00ec21c -> 332056a)

2018-06-15 Thread anirudh2290
This is an automated email from the ASF dual-hosted git repository. anirudh2290 pushed a change to branch v1.2.0 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 00ec21c Scala inference memory leak fix #11204 (#11216) new f6a8cb7 Add NEWS and README

[incubator-mxnet] 02/04: add import_ for SymbolBlock (#11127)

2018-06-15 Thread anirudh2290
This is an automated email from the ASF dual-hosted git repository. anirudh2290 pushed a commit to branch v1.2.0 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit 719beeaab96be2b282761ddbf479670226a77f83 Author: Eric Junyuan Xie AuthorDate: Wed Jun 13 22:01:40 2018

[incubator-mxnet] 03/04: Improve hybridblock doc (#11236)

2018-06-15 Thread anirudh2290
This is an automated email from the ASF dual-hosted git repository. anirudh2290 pushed a commit to branch v1.2.0 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit dfc17bb6ec2627e4ee7bb65c89cc16154c9fa2f9 Author: Eric Junyuan Xie AuthorDate: Thu Jun 14 14:02:48 2018

[GitHub] lanking520 commented on issue #11239: [MXNET-319] Javadoc fix

2018-06-15 Thread GitBox
lanking520 commented on issue #11239: [MXNET-319] Javadoc fix URL: https://github.com/apache/incubator-mxnet/pull/11239#issuecomment-397765994 Currently, I created a file called `FILEHASH` lives in the core used to check if there is a change on generated base files. Add a require as well

[GitHub] apeforest opened a new pull request #11317: Fix the inaccruate instruction on installing nose

2018-06-15 Thread GitBox
apeforest opened a new pull request #11317: Fix the inaccruate instruction on installing nose URL: https://github.com/apache/incubator-mxnet/pull/11317 ## Description ## The instruction for installing nose package should add the 'sudo' in the command. Otherwise, the Mac OS bash would

[GitHub] marcoabreu opened a new issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler

2018-06-15 Thread GitBox
marcoabreu opened a new issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler URL: https://github.com/apache/incubator-mxnet/issues/11249

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

2018-06-15 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-397764278 I meant this is not blocking the pipeline, i'll keep it reopen and edit the

[GitHub] marcoabreu commented on issue #11265: Flaky test: Python 3: CPU Win

2018-06-15 Thread GitBox
marcoabreu commented on issue #11265: Flaky test: Python 3: CPU Win URL: https://github.com/apache/incubator-mxnet/issues/11265#issuecomment-397763495 I don't think there is much we can do - especially considering our Windows slaves are effectively a black box. Closing for now

[GitHub] marcoabreu closed issue #11265: Flaky test: Python 3: CPU Win

2018-06-15 Thread GitBox
marcoabreu closed issue #11265: Flaky test: Python 3: CPU Win URL: https://github.com/apache/incubator-mxnet/issues/11265 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] lanking520 commented on issue #11292: [MXNET-531] CNN Examples for Scala new APi

2018-06-15 Thread GitBox
lanking520 commented on issue #11292: [MXNET-531] CNN Examples for Scala new APi URL: https://github.com/apache/incubator-mxnet/pull/11292#issuecomment-397763241 Currently facing the CI problem as shown below: ``` - Example CI - CNN Example *** FAILED ***

[GitHub] lanking520 commented on issue #11292: [MXNET-531] CNN Examples for Scala new APi

2018-06-15 Thread GitBox
lanking520 commented on issue #11292: [MXNET-531] CNN Examples for Scala new APi URL: https://github.com/apache/incubator-mxnet/pull/11292#issuecomment-397763241 Currently facing the CI problem as shown below: ``` - Example CI - CNN Example *** FAILED ***

[GitHub] asmushetzel commented on issue #11271: nd.topk regression with nan values

2018-06-15 Thread GitBox
asmushetzel commented on issue #11271: nd.topk regression with nan values URL: https://github.com/apache/incubator-mxnet/issues/11271#issuecomment-397763186 @piiswrong I searched a bit through the documentation and the only place where I could find something about NaN handling was in

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

2018-06-15 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-397762978 This is resolved lets open new if there is an issue

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

2018-06-15 Thread GitBox
nswamy closed issue #11249: Flaky Scala test: IllegalArgumentException: requirement failed: Failed to start ps scheduler URL: https://github.com/apache/incubator-mxnet/issues/11249 This is an automated message from the

[GitHub] marcoabreu commented on issue #11313: Static alloc for hybridblock

2018-06-15 Thread GitBox
marcoabreu commented on issue #11313: Static alloc for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/11313#issuecomment-397762309 Sorry if I mixed it up. I tried to read the description, but it was not very helpful.

[GitHub] tqchen commented on issue #11311: Revert "Static memory allocation for cached_op (#10817)"

2018-06-15 Thread GitBox
tqchen commented on issue #11311: Revert "Static memory allocation for cached_op (#10817)" URL: https://github.com/apache/incubator-mxnet/pull/11311#issuecomment-397759632 with great power comes great responsibility. I would never abused that power to any changes just to make a point and

[GitHub] piiswrong commented on issue #11313: Static alloc for hybridblock

2018-06-15 Thread GitBox
piiswrong commented on issue #11313: Static alloc for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/11313#issuecomment-397758785 This is a different PR. If you have any specific concerns on the code please utter them. If you don't understand the code please let

[GitHub] piiswrong commented on issue #11313: Static alloc for hybridblock

2018-06-15 Thread GitBox
piiswrong commented on issue #11313: Static alloc for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/11313#issuecomment-397758785 This is a different PR. If you have any legitimate concerns on the code please specify them. If you don't understand the code please let

[GitHub] tqchen commented on issue #11313: Static alloc for hybridblock

2018-06-15 Thread GitBox
tqchen commented on issue #11313: Static alloc for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/11313#issuecomment-397758362 regression tests are already introduced in https://github.com/apache/incubator-mxnet/pull/11313/files#diff-962bd5bb7248659d7eb3be37ee8a4c6bR1288

[GitHub] tqchen commented on issue #11313: Static alloc for hybridblock

2018-06-15 Thread GitBox
tqchen commented on issue #11313: Static alloc for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/11313#issuecomment-397758362 regression tests are already introduced in https://github.com/apache/incubator-mxnet/pull/11313/files#diff-962bd5bb7248659d7eb3be37ee8a4c6bR1288

[GitHub] eric-haibin-lin commented on issue #11316: MXNET_FORCE_ADDTAKEGRAD to disable AddTakeGradLargeBatchCaller

2018-06-15 Thread GitBox
eric-haibin-lin commented on issue #11316: MXNET_FORCE_ADDTAKEGRAD to disable AddTakeGradLargeBatchCaller URL: https://github.com/apache/incubator-mxnet/pull/11316#issuecomment-397758053 Excellent finding! Worse case we can modify the sparse kernel a bit (just remove the lookup table)

[GitHub] mklissa commented on issue #11296: Adding 2 tutorials on Learning Rate Schedules

2018-06-15 Thread GitBox
mklissa commented on issue #11296: Adding 2 tutorials on Learning Rate Schedules URL: https://github.com/apache/incubator-mxnet/pull/11296#issuecomment-397757892 Very informative and well-written! Good job! This is an

[GitHub] marcoabreu commented on issue #11311: Revert "Static memory allocation for cached_op (#10817)"

2018-06-15 Thread GitBox
marcoabreu commented on issue #11311: Revert "Static memory allocation for cached_op (#10817)" URL: https://github.com/apache/incubator-mxnet/pull/11311#issuecomment-397757505 I'd like to introduce you to the Apache guidelines: https://www.apache.org/foundation/voting.html "2.

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

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

[GitHub] leezu opened a new pull request #11316: MXNET_FORCE_ADDTAKEGRAD to disable AddTakeGradLargeBatchCaller

2018-06-15 Thread GitBox
leezu opened a new pull request #11316: MXNET_FORCE_ADDTAKEGRAD to disable AddTakeGradLargeBatchCaller URL: https://github.com/apache/incubator-mxnet/pull/11316 Adds an environment variable that allows working around #11314 until a proper fix is found.

[GitHub] nswamy opened a new issue #11315: Support multiple input types in Scala

2018-06-15 Thread GitBox
nswamy opened a new issue #11315: Support multiple input types in Scala URL: https://github.com/apache/incubator-mxnet/issues/11315 Currently MXNet-Scala APIs support only float32, this would cause accuracy issues when customers port models between developed on Python to Scala. This needs

[GitHub] leezu opened a new issue #11314: Embedding Backward (AddTakeGradLargeBatchCaller) non-deterministic nan values

2018-06-15 Thread GitBox
leezu opened a new issue #11314: Embedding Backward (AddTakeGradLargeBatchCaller) non-deterministic nan values URL: https://github.com/apache/incubator-mxnet/issues/11314 ## Description The `AddTakeGradLargeBatchCaller` operator called during backward of Embedding is broken and

[GitHub] tqchen commented on issue #11310: Update cached_op.cc

2018-06-15 Thread GitBox
tqchen commented on issue #11310: Update cached_op.cc URL: https://github.com/apache/incubator-mxnet/pull/11310#issuecomment-397755433 replace by #10817 This is an automated message from the Apache Git Service. To respond to

[GitHub] tqchen closed pull request #11310: Update cached_op.cc

2018-06-15 Thread GitBox
tqchen closed pull request #11310: Update cached_op.cc URL: https://github.com/apache/incubator-mxnet/pull/11310 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

[GitHub] lanking520 commented on a change in pull request #11294: [MXNET-543][WIP]Scala Spark test

2018-06-15 Thread GitBox
lanking520 commented on a change in pull request #11294: [MXNET-543][WIP]Scala Spark test URL: https://github.com/apache/incubator-mxnet/pull/11294#discussion_r195873690 ## File path: scala-package/spark/src/test/scala/org/apache/mxnet/spark/MXNetGeneralSuite.scala ## @@

[GitHub] lanking520 commented on a change in pull request #11294: [MXNET-543][WIP]Scala Spark test

2018-06-15 Thread GitBox
lanking520 commented on a change in pull request #11294: [MXNET-543][WIP]Scala Spark test URL: https://github.com/apache/incubator-mxnet/pull/11294#discussion_r195873656 ## File path: scala-package/spark/src/test/scala/org/apache/mxnet/spark/MXNetGeneralSuite.scala ## @@

[GitHub] piiswrong opened a new pull request #11313: Static alloc for hybridblock

2018-06-15 Thread GitBox
piiswrong opened a new pull request #11313: Static alloc for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/11313 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items

[GitHub] piiswrong opened a new pull request #11310: Update cached_op.cc

2018-06-15 Thread GitBox
piiswrong opened a new pull request #11310: Update cached_op.cc URL: https://github.com/apache/incubator-mxnet/pull/11310 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your

[GitHub] tqchen commented on issue #11310: Update cached_op.cc

2018-06-15 Thread GitBox
tqchen commented on issue #11310: Update cached_op.cc URL: https://github.com/apache/incubator-mxnet/pull/11310#issuecomment-397755065 please bring in the new changes, and rebase against the master This is an automated

[GitHub] CodingCat commented on a change in pull request #11294: [MXNET-543][WIP]Scala Spark test

2018-06-15 Thread GitBox
CodingCat commented on a change in pull request #11294: [MXNET-543][WIP]Scala Spark test URL: https://github.com/apache/incubator-mxnet/pull/11294#discussion_r195873461 ## File path: scala-package/spark/src/test/scala/org/apache/mxnet/spark/MXNetGeneralSuite.scala ## @@

[GitHub] piiswrong closed pull request #11310: Update cached_op.cc

2018-06-15 Thread GitBox
piiswrong closed pull request #11310: Update cached_op.cc URL: https://github.com/apache/incubator-mxnet/pull/11310 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

  1   2   3   >