[GitHub] azai91 opened a new pull request #11291: [MXNET-550] Write unit test for MKLDNNCopy

2018-06-14 Thread GitBox
azai91 opened a new pull request #11291: [MXNET-550] Write unit test for MKLDNNCopy URL: https://github.com/apache/incubator-mxnet/pull/11291 ## Description ## Add unit tests for MKLDNNCopy ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for

[GitHub] azai91 commented on issue #11291: [MXNET-550] Write unit test for MKLDNNCopy

2018-06-14 Thread GitBox
azai91 commented on issue #11291: [MXNET-550] Write unit test for MKLDNNCopy URL: https://github.com/apache/incubator-mxnet/pull/11291#issuecomment-397421836 this test is a little more robust than the prev CopyFrom test. That test reorders to inputs to default and before copying. This test

[GitHub] eric-haibin-lin closed issue #8425: MXNET AMI URL has been broken

2018-06-14 Thread GitBox
eric-haibin-lin closed issue #8425: MXNET AMI URL has been broken URL: https://github.com/apache/incubator-mxnet/issues/8425 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] eric-haibin-lin closed issue #7918: How to use the sample_weight parameter of SoftmaxCrossEntropyLoss?

2018-06-14 Thread GitBox
eric-haibin-lin closed issue #7918: How to use the sample_weight parameter of SoftmaxCrossEntropyLoss? URL: https://github.com/apache/incubator-mxnet/issues/7918 This is an automated message from the Apache Git Service. To

[GitHub] eric-haibin-lin closed issue #7918: How to use the sample_weight parameter of SoftmaxCrossEntropyLoss?

2018-06-14 Thread GitBox
eric-haibin-lin closed issue #7918: How to use the sample_weight parameter of SoftmaxCrossEntropyLoss? URL: https://github.com/apache/incubator-mxnet/issues/7918 This is an automated message from the Apache Git Service. To

[GitHub] eric-haibin-lin closed issue #7823: can anyone give clue how to install latest prebuilt package , where is setupenv.cmd? thanks

2018-06-14 Thread GitBox
eric-haibin-lin closed issue #7823: can anyone give clue how to install latest prebuilt package , where is setupenv.cmd? thanks URL: https://github.com/apache/incubator-mxnet/issues/7823 This is an automated message from

[GitHub] eric-haibin-lin closed issue #7897: What will happen when executing Model.update() and Model.forward() at same time in two threads?

2018-06-14 Thread GitBox
eric-haibin-lin closed issue #7897: What will happen when executing Model.update() and Model.forward() at same time in two threads? URL: https://github.com/apache/incubator-mxnet/issues/7897 This is an automated message

[GitHub] evahlis opened a new issue #7918: How to use the sample_weight parameter of SoftmaxCrossEntropyLoss?

2018-06-14 Thread GitBox
evahlis opened a new issue #7918: How to use the sample_weight parameter of SoftmaxCrossEntropyLoss? URL: https://github.com/apache/incubator-mxnet/issues/7918 Hi, The documentation of the loss function

[GitHub] eric-haibin-lin closed issue #8282: Error: dot‑gemm: matrix shape mismatch

2018-06-14 Thread GitBox
eric-haibin-lin closed issue #8282: Error: dot‑gemm: matrix shape mismatch URL: https://github.com/apache/incubator-mxnet/issues/8282 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] piiswrong closed pull request #11273: disable testcase test_gru_bidirectional temporarily

2018-06-14 Thread GitBox
piiswrong closed pull request #11273: disable testcase test_gru_bidirectional temporarily URL: https://github.com/apache/incubator-mxnet/pull/11273 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:

[GitHub] piiswrong closed pull request #11236: Improve hybridblock doc

2018-06-14 Thread GitBox
piiswrong closed pull request #11236: Improve hybridblock doc URL: https://github.com/apache/incubator-mxnet/pull/11236 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

[GitHub] Ishitori commented on a change in pull request #11274: [MXNET-547] Tutorial explaining how to use the profiler

2018-06-14 Thread GitBox
Ishitori commented on a change in pull request #11274: [MXNET-547] Tutorial explaining how to use the profiler URL: https://github.com/apache/incubator-mxnet/pull/11274#discussion_r195571404 ## File path: docs/tutorials/python/profiler.md ## @@ -0,0 +1,198 @@ +#

[GitHub] Ishitori commented on a change in pull request #11274: [MXNET-547] Tutorial explaining how to use the profiler

2018-06-14 Thread GitBox
Ishitori commented on a change in pull request #11274: [MXNET-547] Tutorial explaining how to use the profiler URL: https://github.com/apache/incubator-mxnet/pull/11274#discussion_r195572036 ## File path: docs/tutorials/python/profiler.md ## @@ -0,0 +1,198 @@ +#

[GitHub] Ishitori commented on a change in pull request #11274: [MXNET-547] Tutorial explaining how to use the profiler

2018-06-14 Thread GitBox
Ishitori commented on a change in pull request #11274: [MXNET-547] Tutorial explaining how to use the profiler URL: https://github.com/apache/incubator-mxnet/pull/11274#discussion_r195571081 ## File path: docs/tutorials/python/profiler.md ## @@ -0,0 +1,198 @@ +#

[GitHub] wenyangchu commented on issue #11247: Add seed_aug parameter for ImageRecordItr to fix random seed for default augmentation

2018-06-14 Thread GitBox
wenyangchu commented on issue #11247: Add seed_aug parameter for ImageRecordItr to fix random seed for default augmentation URL: https://github.com/apache/incubator-mxnet/pull/11247#issuecomment-397445042 Hi @marcoabreu or others, the c++ tests for iterators in mxnet are not there and

[GitHub] piiswrong closed pull request #11269: Enable shared ccache and fix test

2018-06-14 Thread GitBox
piiswrong closed pull request #11269: Enable shared ccache and fix test URL: https://github.com/apache/incubator-mxnet/pull/11269 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] kalyc commented on issue #11278: global name 'multiply' is not defined when using LBSGD

2018-06-14 Thread GitBox
kalyc commented on issue #11278: global name 'multiply' is not defined when using LBSGD URL: https://github.com/apache/incubator-mxnet/issues/11278#issuecomment-397448571 Thanks for submitting this issue @xmfbit @sandeep-krishnamurthy could you add label "Question" to this issue?

[incubator-mxnet] branch master updated: Enable shared ccache and fix test (#11269)

2018-06-14 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie 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 2eadfa2 Enable shared ccache and fix

[GitHub] kalyc commented on issue #11172: Flaky test: test_io.test_LibSVMIter

2018-06-14 Thread GitBox
kalyc commented on issue #11172: Flaky test: test_io.test_LibSVMIter URL: https://github.com/apache/incubator-mxnet/issues/11172#issuecomment-397450500 Thanks for your contributions @ThomasDelteil - could you add a link to the PR here?

[GitHub] hcho3 commented on issue #11209: [MXNET-536] implement var/std operators

2018-06-14 Thread GitBox
hcho3 commented on issue #11209: [MXNET-536] implement var/std operators URL: https://github.com/apache/incubator-mxnet/pull/11209#issuecomment-397399618 Looks like two of the CI tests failed for reasons unrelated to the var/std operators. I'll trigger CI again to make sure tests pass.

[GitHub] hcho3 opened a new pull request #11209: [MXNET-536] implement var/std operators

2018-06-14 Thread GitBox
hcho3 opened a new pull request #11209: [MXNET-536] implement var/std operators URL: https://github.com/apache/incubator-mxnet/pull/11209 ## Description ## This is a pull request to implement variance and standard deviation operators. ## Checklist ## ### Essentials ###

[GitHub] hcho3 closed pull request #11209: [MXNET-536] implement var/std operators

2018-06-14 Thread GitBox
hcho3 closed pull request #11209: [MXNET-536] implement var/std operators URL: https://github.com/apache/incubator-mxnet/pull/11209 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] mbaijal opened a new issue #11288: [Nightly Tests] Installation Guide: Enable docker install test

2018-06-14 Thread GitBox
mbaijal opened a new issue #11288: [Nightly Tests] Installation Guide: Enable docker install test URL: https://github.com/apache/incubator-mxnet/issues/11288 All nightly tests now run in the official CI inside the docker containers. However, in order to run the docker install steps in CI

[GitHub] wenyangchu closed pull request #11247: [WIP] add seed_aug parameter for ImageRecordItr to fix random seed for augm…

2018-06-14 Thread GitBox
wenyangchu closed pull request #11247: [WIP] add seed_aug parameter for ImageRecordItr to fix random seed for augm… URL: https://github.com/apache/incubator-mxnet/pull/11247 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below

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

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

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

2018-06-14 Thread GitBox
marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests. URL: https://github.com/apache/incubator-mxnet/pull/10827#discussion_r195541860 ## File path: docs/install/index.md ## @@ -84,7 +84,7

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

2018-06-14 Thread GitBox
marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests. URL: https://github.com/apache/incubator-mxnet/pull/10827#discussion_r195541505 ## File path: ci/docker/runtime_functions.sh ## @@

[GitHub] anirudh2290 closed pull request #11270: Enable CUDNN for conv1D (#11194)

2018-06-14 Thread GitBox
anirudh2290 closed pull request #11270: Enable CUDNN for conv1D (#11194) URL: https://github.com/apache/incubator-mxnet/pull/11270 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] anirudh2290 closed pull request #11210: [MXNET-532] Clarify documentation of save_parameters(), load_parameters()

2018-06-14 Thread GitBox
anirudh2290 closed pull request #11210: [MXNET-532] Clarify documentation of save_parameters(), load_parameters() URL: https://github.com/apache/incubator-mxnet/pull/11210 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[incubator-mxnet] branch master updated: [MXNET-532] Clarify documentation of save_parameters(), load_parameters() (#11210)

2018-06-14 Thread anirudh2290
This is an automated email from the ASF dual-hosted git repository. anirudh2290 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 496b0f2 [MXNET-532] Clarify

[GitHub] kalyc commented on issue #11206: Embedding(sparse_grad=True) + hybridization fallback issues

2018-06-14 Thread GitBox
kalyc commented on issue #11206: Embedding(sparse_grad=True) + hybridization fallback issues URL: https://github.com/apache/incubator-mxnet/issues/11206#issuecomment-397439572 Hi @leezu thanks for submitting this issue. @sandeep-krishnamurthy could you add label "Sparse" & "Python" to

[GitHub] kalyc commented on issue #11192: BatchNorm operator on GPUs are slow with channels_last

2018-06-14 Thread GitBox
kalyc commented on issue #11192: BatchNorm operator on GPUs are slow with channels_last URL: https://github.com/apache/incubator-mxnet/issues/11192#issuecomment-397441601 Thanks for submitting this tracking issue @sandeep-krishnamurthy

[GitHub] kalyc commented on issue #11218: Feature request: randint operator

2018-06-14 Thread GitBox
kalyc commented on issue #11218: Feature request: randint operator URL: https://github.com/apache/incubator-mxnet/issues/11218#issuecomment-397445998 Thanks for submitting the issue @eric-haibin-lin This is an automated

[GitHub] kalyc commented on issue #11173: Flaky test: test_deformable_convolution_options

2018-06-14 Thread GitBox
kalyc commented on issue #11173: Flaky test: test_deformable_convolution_options URL: https://github.com/apache/incubator-mxnet/issues/11173#issuecomment-397446923 Thanks for submitting this issue @ThomasDelteil This is the related PR -

[GitHub] nswamy commented on a change in pull request #11256: [MXNET-539] Allow Scala users to specify data/label names for NDArrayIter

2018-06-14 Thread GitBox
nswamy commented on a change in pull request #11256: [MXNET-539] Allow Scala users to specify data/label names for NDArrayIter URL: https://github.com/apache/incubator-mxnet/pull/11256#discussion_r195578736 ## File path:

[GitHub] nswamy commented on a change in pull request #11256: [MXNET-539] Allow Scala users to specify data/label names for NDArrayIter

2018-06-14 Thread GitBox
nswamy commented on a change in pull request #11256: [MXNET-539] Allow Scala users to specify data/label names for NDArrayIter URL: https://github.com/apache/incubator-mxnet/pull/11256#discussion_r195579473 ## File path:

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

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

[GitHub] ThomasDelteil commented on issue #11219: Flaky test: test_gru_bidirectional

2018-06-14 Thread GitBox
ThomasDelteil commented on issue #11219: Flaky test: test_gru_bidirectional URL: https://github.com/apache/incubator-mxnet/issues/11219#issuecomment-397449610 might have been just merged and you got unlucky, got merged 1h ago as well

[GitHub] kalyc commented on issue #11290: Flaky test test_operator_gpu.test_scatter_gather_nd

2018-06-14 Thread GitBox
kalyc commented on issue #11290: Flaky test test_operator_gpu.test_scatter_gather_nd URL: https://github.com/apache/incubator-mxnet/issues/11290#issuecomment-397449764 Thanks for submitting this issue @marcoabreu This is

[GitHub] sandeep-krishnamurthy commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-14 Thread GitBox
sandeep-krishnamurthy commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r195551176 ## File path: python/mxnet/contrib/onnx/_export/export_model.py ## @@ -0,0 +1,89 @@ +#

[incubator-mxnet] branch v1.2.0 updated: Enable CUDNN for conv1D (#11194) (#11270)

2018-06-14 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 73ebc76 Enable CUDNN for conv1D

[GitHub] kalyc commented on issue #11284: Issues with Gluon example/gluon/image_classification.py example

2018-06-14 Thread GitBox
kalyc commented on issue #11284: Issues with Gluon example/gluon/image_classification.py example URL: https://github.com/apache/incubator-mxnet/issues/11284#issuecomment-397438625 Hi @ZaidQureshi thanks for submitting this issue. @sandeep-krishnamurthy could you add label "Gluon",

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

2018-06-14 Thread GitBox
marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests. URL: https://github.com/apache/incubator-mxnet/pull/10827#discussion_r195571306 ## File path: ci/docker/Dockerfile.build.ubuntu_emscripten

[GitHub] nswamy commented on issue #11193: Fix OpenCV 3.4.1 issue for Scala package for OSX

2018-06-14 Thread GitBox
nswamy commented on issue #11193: Fix OpenCV 3.4.1 issue for Scala package for OSX URL: https://github.com/apache/incubator-mxnet/issues/11193#issuecomment-397442124 Yes that is because we have already published the package and cannot undo without bumping up the version which will happen

[GitHub] kalyc commented on issue #11279: 2d fft operation with mxnet

2018-06-14 Thread GitBox
kalyc commented on issue #11279: 2d fft operation with mxnet URL: https://github.com/apache/incubator-mxnet/issues/11279#issuecomment-397444127 Hi @C2H5OHlife thanks for submitting this issue. There's an open forum to discuss about mxnet - you could use discuss.mxnet.io to learn more.

[GitHub] apeforest commented on issue #11222: Bug in gluon.block.Block.LoadParams

2018-06-14 Thread GitBox
apeforest commented on issue #11222: Bug in gluon.block.Block.LoadParams URL: https://github.com/apache/incubator-mxnet/issues/11222#issuecomment-397444626 I am looking at this issue now. I will create a JIRA ticket for it.

[GitHub] lanking520 opened a new pull request #11292: [MXNET-531] CNN Examples for Scala new APi

2018-06-14 Thread GitBox
lanking520 opened a new pull request #11292: [MXNET-531] CNN Examples for Scala new APi URL: https://github.com/apache/incubator-mxnet/pull/11292 ## Description ## This PR contains examples written from the new API. @nswamy @yzhliu @andrewfayres All examples pushed here are tested

[GitHub] ThomasDelteil commented on issue #11293: Update cached_op.cc

2018-06-14 Thread GitBox
ThomasDelteil commented on issue #11293: Update cached_op.cc URL: https://github.com/apache/incubator-mxnet/pull/11293#issuecomment-397449059 Thanks @piiswrong for updating the code this is related to this PR https://github.com/apache/incubator-mxnet/pull/10817 Using the

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

2018-06-14 Thread GitBox
kalyc commented on issue #11271: nd.topk regression with nan values URL: https://github.com/apache/incubator-mxnet/issues/11271#issuecomment-397449171 @leezu thanks for submitting this issue. @sandeep-krishnamurthy could you add label "Bug" to this?

[GitHub] marcoabreu commented on a change in pull request #11293: Update cached_op.cc

2018-06-14 Thread GitBox
marcoabreu commented on a change in pull request #11293: Update cached_op.cc URL: https://github.com/apache/incubator-mxnet/pull/11293#discussion_r195582140 ## File path: src/imperative/cached_op.cc ## @@ -674,7 +674,7 @@ OpStatePtr CachedOp::StaticForward(

[GitHub] ThomasDelteil closed issue #11172: Flaky test: test_io.test_LibSVMIter

2018-06-14 Thread GitBox
ThomasDelteil closed issue #11172: Flaky test: test_io.test_LibSVMIter URL: https://github.com/apache/incubator-mxnet/issues/11172 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] ThomasDelteil commented on issue #11172: Flaky test: test_io.test_LibSVMIter

2018-06-14 Thread GitBox
ThomasDelteil commented on issue #11172: Flaky test: test_io.test_LibSVMIter URL: https://github.com/apache/incubator-mxnet/issues/11172#issuecomment-397450017 I implemented the retry logic in `test_utils.download` and `gluon.utils.download`, hopefully this will not happen again 

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

2018-06-14 Thread GitBox
marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests. URL: https://github.com/apache/incubator-mxnet/pull/10827#discussion_r195543406 ## File path: ci/docker/Dockerfile.build.ubuntu_base_cpu

[GitHub] sandeep-krishnamurthy commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-14 Thread GitBox
sandeep-krishnamurthy commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r195551597 ## File path: python/mxnet/contrib/onnx/_export/export_model.py ## @@ -0,0 +1,89 @@ +#

[GitHub] eric-haibin-lin closed issue #4845: Has anyone been able to get MXNet to work with Spark on Mesos?

2018-06-14 Thread GitBox
eric-haibin-lin closed issue #4845: Has anyone been able to get MXNet to work with Spark on Mesos? URL: https://github.com/apache/incubator-mxnet/issues/4845 This is an automated message from the Apache Git Service. To

[GitHub] kalyc commented on issue #11283: Group Norm

2018-06-14 Thread GitBox
kalyc commented on issue #11283: Group Norm URL: https://github.com/apache/incubator-mxnet/issues/11283#issuecomment-397442423 Thanks for submitting this issue @smorrel1 @sandeep-krishnamurthy could you add label "Feature Request" to this issue?

[GitHub] wenyangchu commented on issue #11247: Add seed_aug parameter for ImageRecordItr to fix random seed for default augmentation

2018-06-14 Thread GitBox
wenyangchu commented on issue #11247: Add seed_aug parameter for ImageRecordItr to fix random seed for default augmentation URL: https://github.com/apache/incubator-mxnet/pull/11247#issuecomment-397445042 Hi @marcoabreu or others, the c++ tests in mxnet are not there and code is entirely

[GitHub] kalyc commented on issue #11172: Flaky test: test_io.test_LibSVMIter

2018-06-14 Thread GitBox
kalyc commented on issue #11172: Flaky test: test_io.test_LibSVMIter URL: https://github.com/apache/incubator-mxnet/issues/11172#issuecomment-397447588 Thanks for submitting this issue @ThomasDelteil This is an automated

[GitHub] kalyc commented on issue #11171: Flaky test: test_gluon.test_hybrid_static_memory_switching

2018-06-14 Thread GitBox
kalyc commented on issue #11171: Flaky test: test_gluon.test_hybrid_static_memory_switching URL: https://github.com/apache/incubator-mxnet/issues/11171#issuecomment-397448054 Thanks for submitting this issue @ThomasDelteil

[GitHub] aaronmarkham commented on issue #11219: Flaky test: test_gru_bidirectional

2018-06-14 Thread GitBox
aaronmarkham commented on issue #11219: Flaky test: test_gru_bidirectional URL: https://github.com/apache/incubator-mxnet/issues/11219#issuecomment-397448129 Doesn't seem disabled to me:

[GitHub] aaronmarkham commented on issue #11219: Flaky test: test_gru_bidirectional

2018-06-14 Thread GitBox
aaronmarkham commented on issue #11219: Flaky test: test_gru_bidirectional URL: https://github.com/apache/incubator-mxnet/issues/11219#issuecomment-397450310 Ya, you're right. I saw the 16 hours ago and "merged" nearby so I thought that's when it happened. I definitely missed the window.

[GitHub] piiswrong commented on a change in pull request #11027: Add standard ResNet data augmentation for ImageRecordIter

2018-06-14 Thread GitBox
piiswrong commented on a change in pull request #11027: Add standard ResNet data augmentation for ImageRecordIter URL: https://github.com/apache/incubator-mxnet/pull/11027#discussion_r195585197 ## File path: src/io/image_aug_default.cc ## @@ -277,7 +413,8 @@ class

[GitHub] spidyDev commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-14 Thread GitBox
spidyDev commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r195585421 ## File path: python/mxnet/contrib/onnx/__init__.py ## @@ -18,3 +18,4 @@ from ._import.import_model

[GitHub] spidyDev commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-14 Thread GitBox
spidyDev commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r195585681 ## File path: python/mxnet/contrib/onnx/_export/export_onnx.py ## @@ -0,0 +1,270 @@ +# Licensed to the

[GitHub] spidyDev commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-14 Thread GitBox
spidyDev commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r195585456 ## File path: python/mxnet/contrib/onnx/_export/export_model.py ## @@ -0,0 +1,89 @@ +# Licensed to the

[GitHub] piiswrong commented on issue #11267: Add NEWS and README

2018-06-14 Thread GitBox
piiswrong commented on issue #11267: Add NEWS and README URL: https://github.com/apache/incubator-mxnet/pull/11267#issuecomment-397459201 I think https://github.com/apache/incubator-mxnet/pull/11267/commits/7632100d8f3ced6375df6d3907c4688fa0979899 is too verbose. It would only confuse

[GitHub] apeforest commented on issue #11016: [Feature request] temperature parameter in Softmax and SoftmaxOutput

2018-06-14 Thread GitBox
apeforest commented on issue #11016: [Feature request] temperature parameter in Softmax and SoftmaxOutput URL: https://github.com/apache/incubator-mxnet/issues/11016#issuecomment-397460067 Hi @slitsey I will start working on this Feature Request. There are multiple operators related to

[GitHub] marcoabreu commented on a change in pull request #11293: Update cached_op.cc

2018-06-14 Thread GitBox
marcoabreu commented on a change in pull request #11293: Update cached_op.cc URL: https://github.com/apache/incubator-mxnet/pull/11293#discussion_r195593254 ## File path: src/imperative/cached_op.cc ## @@ -674,7 +674,7 @@ OpStatePtr CachedOp::StaticForward(

[GitHub] kalyc commented on issue #10553: Build error for OSX with CUDA support

2018-06-14 Thread GitBox
kalyc commented on issue #10553: Build error for OSX with CUDA support URL: https://github.com/apache/incubator-mxnet/issues/10553#issuecomment-397464494 @sbodenstein could you let us know if your issue has been fixed? This

[GitHub] apeforest commented on issue #11224: ‘make lint’ is broken under python2

2018-06-14 Thread GitBox
apeforest commented on issue #11224: ‘make lint’ is broken under python2 URL: https://github.com/apache/incubator-mxnet/issues/11224#issuecomment-397464272 @TaoLv I am looking into this issue. Could you please update the "Steps to reproduce". The error I got when running "make lint" is

[GitHub] kalyc commented on issue #11029: The example of train_mnist.py can't exit after completes.

2018-06-14 Thread GitBox
kalyc commented on issue #11029: The example of train_mnist.py can't exit after completes. URL: https://github.com/apache/incubator-mxnet/issues/11029#issuecomment-397464295 @liuzx32 thanks for submitting this issue It looks like @rahul003 opened a PR for this and it has been merged.

[incubator-mxnet] branch v1.2.0 updated (a48480b -> 62a47a7)

2018-06-14 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 a48480b [maven-release-plugin] prepare release mxnet-parent_2.11-1.2.0 add c2bfcf0 Fix

[GitHub] kovasb commented on issue #11205: Clojure Contrib Package

2018-06-14 Thread GitBox
kovasb commented on issue #11205: Clojure Contrib Package URL: https://github.com/apache/incubator-mxnet/pull/11205#issuecomment-397467009 Thanks for this contribution! In terms of code quality this looks very good, getting a lot done in a straightforward manner. People should feel

[GitHub] anirudh2290 commented on issue #11267: Add NEWS and README

2018-06-14 Thread GitBox
anirudh2290 commented on issue #11267: Add NEWS and README URL: https://github.com/apache/incubator-mxnet/pull/11267#issuecomment-397471169 I agree that we need the rest of the content to inform users about the `save_parameters` being the new API to use instead of `save_params` and to use

[GitHub] reminisce commented on a change in pull request #10889: [MXNET-382] Shape and Size Operator

2018-06-14 Thread GitBox
reminisce commented on a change in pull request #10889: [MXNET-382] Shape and Size Operator URL: https://github.com/apache/incubator-mxnet/pull/10889#discussion_r195603907 ## File path: src/operator/tensor/elemwise_unary_op_basic.cc ## @@ -398,6 +398,106 @@

[GitHub] tqchen commented on issue #10451: [MXNET-432] Add Foreach

2018-06-14 Thread GitBox
tqchen commented on issue #10451: [MXNET-432] Add Foreach URL: https://github.com/apache/incubator-mxnet/pull/10451#issuecomment-397473068 One suggestion, since this ops is common known as scan, why now just use the common name instead of inventing a new API name?

[GitHub] msharmavikram opened a new issue #11286: Similar performance between AlexNet and ResNet-50 for training during benchmarking with train_imagenet.py

2018-06-14 Thread GitBox
msharmavikram opened a new issue #11286: Similar performance between AlexNet and ResNet-50 for training during benchmarking with train_imagenet.py URL: https://github.com/apache/incubator-mxnet/issues/11286 Note: Providing complete information in the most concise form is the best way to

[GitHub] anirudh2290 commented on issue #11298: Add NEWS and README

2018-06-14 Thread GitBox
anirudh2290 commented on issue #11298: Add NEWS and README URL: https://github.com/apache/incubator-mxnet/pull/11298#issuecomment-397479900 I have addressed the comments in https://github.com/apache/incubator-mxnet/pull/11267

[GitHub] anirudh2290 opened a new pull request #11298: Add NEWS and README

2018-06-14 Thread GitBox
anirudh2290 opened a new pull request #11298: Add NEWS and README URL: https://github.com/apache/incubator-mxnet/pull/11298 ## Description ## Adding NEWS and README ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your PR. - [ ] The PR

[GitHub] azai91 commented on issue #11291: [MXNET-550] Create MKLDNNCopy helper for copying mkldnn memory

2018-06-14 Thread GitBox
azai91 commented on issue #11291: [MXNET-550] Create MKLDNNCopy helper for copying mkldnn memory URL: https://github.com/apache/incubator-mxnet/pull/11291#issuecomment-397482037 there is already a copyFrom test in the file so I know this change does not cause regression there.

[GitHub] marcoabreu commented on issue #11291: [MXNET-550] Create MKLDNNCopy helper for copying mkldnn memory

2018-06-14 Thread GitBox
marcoabreu commented on issue #11291: [MXNET-550] Create MKLDNNCopy helper for copying mkldnn memory URL: https://github.com/apache/incubator-mxnet/pull/11291#issuecomment-397483020 Ah, your previous comment sounded like you wrote a new test. Gotcha

[GitHub] marcoabreu commented on issue #11247: Add seed_aug parameter for ImageRecordItr to fix random seed for default augmentation

2018-06-14 Thread GitBox
marcoabreu commented on issue #11247: Add seed_aug parameter for ImageRecordItr to fix random seed for default augmentation URL: https://github.com/apache/incubator-mxnet/pull/11247#issuecomment-397482824 Sorry, it's flaky https://github.com/apache/incubator-mxnet/issues/9853

[GitHub] hetong007 commented on a change in pull request #11027: Add standard ResNet data augmentation for ImageRecordIter

2018-06-14 Thread GitBox
hetong007 commented on a change in pull request #11027: Add standard ResNet data augmentation for ImageRecordIter URL: https://github.com/apache/incubator-mxnet/pull/11027#discussion_r195590097 ## File path: src/io/image_aug_default.cc ## @@ -218,10 +265,93 @@ class

[GitHub] piiswrong commented on a change in pull request #11293: Update cached_op.cc

2018-06-14 Thread GitBox
piiswrong commented on a change in pull request #11293: Update cached_op.cc URL: https://github.com/apache/incubator-mxnet/pull/11293#discussion_r195591137 ## File path: src/imperative/cached_op.cc ## @@ -674,7 +674,7 @@ OpStatePtr CachedOp::StaticForward(

[GitHub] anirudh2290 commented on issue #11267: Add NEWS and README

2018-06-14 Thread GitBox
anirudh2290 commented on issue #11267: Add NEWS and README URL: https://github.com/apache/incubator-mxnet/pull/11267#issuecomment-397458409 @piiswrong @ThomasDelteil cherry picked your changes. Please help review as i had to resolve some conflicts.

[GitHub] ThomasDelteil commented on a change in pull request #11267: Add NEWS and README

2018-06-14 Thread GitBox
ThomasDelteil commented on a change in pull request #11267: Add NEWS and README URL: https://github.com/apache/incubator-mxnet/pull/11267#discussion_r195592275 ## File path: docs/tutorials/gluon/hybrid.md ## @@ -105,23 +105,35 @@ Hybridize will speed up execution and save

[GitHub] lanking520 opened a new pull request #11294: [MXNET-543][WIP]Scala Spark test

2018-06-14 Thread GitBox
lanking520 opened a new pull request #11294: [MXNET-543][WIP]Scala Spark test URL: https://github.com/apache/incubator-mxnet/pull/11294 ## Description ## This PR is a bug fixes on the Spark Example for Scala. We need to ensure there are enough amount of tests. Here is the plan for the

[GitHub] lanking520 commented on a change in pull request #11239: [MXNET-319] Javadoc fix

2018-06-14 Thread GitBox
lanking520 commented on a change in pull request #11239: [MXNET-319] Javadoc fix URL: https://github.com/apache/incubator-mxnet/pull/11239#discussion_r195594436 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/APIDocGenerator.scala ## @@ -61,8 +63,35 @@

[GitHub] kalyc commented on issue #11060: wget https://s3-us-west-2.amazonaws.com/mxnet.liuyz/data/mnist/train.txt throw 404

2018-06-14 Thread GitBox
kalyc commented on issue #11060: wget https://s3-us-west-2.amazonaws.com/mxnet.liuyz/data/mnist/train.txt throw 404 URL: https://github.com/apache/incubator-mxnet/issues/11060#issuecomment-397461546 @liuzx32 thanks for submitting this issue. It appears that you were able to obtain the

[GitHub] ThomasDelteil commented on issue #11267: Add NEWS and README

2018-06-14 Thread GitBox
ThomasDelteil commented on issue #11267: Add NEWS and README URL: https://github.com/apache/incubator-mxnet/pull/11267#issuecomment-397461590 @piiswrong I think we can remove this sentence: ``` Such model can then be loaded back in any language binding or even in Gluon using a

[GitHub] kovasb commented on issue #11205: Clojure Contrib Package

2018-06-14 Thread GitBox
kovasb commented on issue #11205: Clojure Contrib Package URL: https://github.com/apache/incubator-mxnet/pull/11205#issuecomment-397467009 Thanks for this contribution! In terms of code quality this looks very good, getting a lot done in a straightforward manner. People should feel

[GitHub] reminisce commented on a change in pull request #10889: [MXNET-382] Shape and Size Operator

2018-06-14 Thread GitBox
reminisce commented on a change in pull request #10889: [MXNET-382] Shape and Size Operator URL: https://github.com/apache/incubator-mxnet/pull/10889#discussion_r195603650 ## File path: src/operator/tensor/elemwise_unary_op_basic.cc ## @@ -398,6 +398,106 @@

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

2018-06-14 Thread GitBox
marcoabreu commented on a change in pull request #11199: [MXNET-538] Add XUnit to python tests URL: https://github.com/apache/incubator-mxnet/pull/11199#discussion_r195606253 ## File path: Jenkinsfile ## @@ -92,6 +92,20 @@ echo ${libs} | sed -e 's/,/ /g' | xargs md5sum

[GitHub] marcoabreu closed pull request #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds.

2018-06-14 Thread GitBox
marcoabreu closed pull request #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds. URL: https://github.com/apache/incubator-mxnet/pull/11245 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] marcoabreu opened a new issue #11297: Test top-k with big shape

2018-06-14 Thread GitBox
marcoabreu opened a new issue #11297: Test top-k with big shape URL: https://github.com/apache/incubator-mxnet/issues/11297 https://github.com/apache/incubator-mxnet/pull/11245/files#diff-69757562d07268150de8b369ff5b6b61L714 We had to remove a test because it was failing on arm.

[GitHub] marcoabreu commented on issue #11291: [MXNET-550] Create MKLDNNCopy helper for copying mkldnn memory

2018-06-14 Thread GitBox
marcoabreu commented on issue #11291: [MXNET-550] Create MKLDNNCopy helper for copying mkldnn memory URL: https://github.com/apache/incubator-mxnet/pull/11291#issuecomment-397481702 I might be missing something, but did you forget to add the test by any chance? :X

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

2018-06-14 Thread GitBox
azai91 commented on issue #11129: [MXNET-497]Test kAddTo request for mkldnn operators URL: https://github.com/apache/incubator-mxnet/pull/11129#issuecomment-397456719 working on breaking this diff into smaller diffs. will postpone trying to merge this for a couple of days.

[GitHub] nswamy commented on a change in pull request #11239: [MXNET-319] Javadoc fix

2018-06-14 Thread GitBox
nswamy commented on a change in pull request #11239: [MXNET-319] Javadoc fix URL: https://github.com/apache/incubator-mxnet/pull/11239#discussion_r195590681 ## File path: scala-package/core/pom.xml ## @@ -65,6 +65,26 @@ org.scalastyle

[GitHub] nswamy commented on a change in pull request #11239: [MXNET-319] Javadoc fix

2018-06-14 Thread GitBox
nswamy commented on a change in pull request #11239: [MXNET-319] Javadoc fix URL: https://github.com/apache/incubator-mxnet/pull/11239#discussion_r195583829 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/APIDocGenerator.scala ## @@ -61,8 +63,35 @@

[GitHub] nswamy commented on a change in pull request #11239: [MXNET-319] Javadoc fix

2018-06-14 Thread GitBox
nswamy commented on a change in pull request #11239: [MXNET-319] Javadoc fix URL: https://github.com/apache/incubator-mxnet/pull/11239#discussion_r195583486 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/APIDocGenerator.scala ## @@ -37,6 +37,8 @@

<    1   2   3   >