[GitHub] [incubator-mxnet] TaoLv commented on issue #16735: Use single-bit for mask in dropout operator

2020-02-14 Thread GitBox
TaoLv commented on issue #16735: Use single-bit for mask in dropout operator URL: https://github.com/apache/incubator-mxnet/pull/16735#issuecomment-586564858 > Does the slow down come from more computation in the new algorithm or the sub-optimal implementation? The new

[incubator-mxnet] branch master updated (d352673 -> 149975c)

2020-02-14 Thread haoj
This is an automated email from the ASF dual-hosted git repository. haoj pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from d352673 Fix transformer.cu interleaved matmul for cuda arch < 5 (#17596) add 149975c [numpy][Do Not

[incubator-mxnet] branch master updated (d352673 -> 149975c)

2020-02-14 Thread haoj
This is an automated email from the ASF dual-hosted git repository. haoj pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from d352673 Fix transformer.cu interleaved matmul for cuda arch < 5 (#17596) add 149975c [numpy][Do Not

[GitHub] [incubator-mxnet] haojin2 merged pull request #16865: [numpy][Do Not Review]add op insert

2020-02-14 Thread GitBox
haojin2 merged pull request #16865: [numpy][Do Not Review]add op insert URL: https://github.com/apache/incubator-mxnet/pull/16865 This is an automated message from the Apache Git Service. To respond to the message, please

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

2020-02-14 Thread aaronmarkham
This is an automated email from the ASF dual-hosted git repository. aaronmarkham 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 b73b3ea Bump the

[GitHub] [incubator-mxnet] leezu opened a new pull request #17603: Backport #17596

2020-02-14 Thread GitBox
leezu opened a new pull request #17603: Backport #17596 URL: https://github.com/apache/incubator-mxnet/pull/17603 Backport #17596 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-mxnet] leezu commented on issue #17017: USE_BLAS=apple broken on OSX 10.15

2020-02-14 Thread GitBox
leezu commented on issue #17017: USE_BLAS=apple broken on OSX 10.15 URL: https://github.com/apache/incubator-mxnet/issues/17017#issuecomment-586560692 Fixed by https://github.com/apache/incubator-mxnet/pull/17602 for cmake build

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #17599: Fixed Embedding op for LT input

2020-02-14 Thread GitBox
apeforest commented on a change in pull request #17599: Fixed Embedding op for LT input URL: https://github.com/apache/incubator-mxnet/pull/17599#discussion_r379766442 ## File path: src/operator/tensor/indexing_op.h ## @@ -66,7 +66,7 @@ enum QuantizedEmbeddingOpResource

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #16735: Use single-bit for mask in dropout operator

2020-02-14 Thread GitBox
apeforest commented on a change in pull request #16735: Use single-bit for mask in dropout operator URL: https://github.com/apache/incubator-mxnet/pull/16735#discussion_r379763372 ## File path: src/operator/nn/dropout-inl.h ## @@ -152,15 +181,20 @@ class DropoutOp {

[GitHub] [incubator-mxnet] leezu opened a new pull request #17602: Fix OS X staticbuild and add tests

2020-02-14 Thread GitBox
leezu opened a new pull request #17602: Fix OS X staticbuild and add tests URL: https://github.com/apache/incubator-mxnet/pull/17602 ## Description ## Fix OS X staticbuild and add tests based on Github Actions. This is an

[incubator-mxnet] branch master updated (9ee4f04 -> d352673)

2020-02-14 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 9ee4f04 Support broadcast assign for `npi_boolean_mask_assign_tensor` (#17131) add d352673 Fix

[incubator-mxnet] branch master updated (9ee4f04 -> d352673)

2020-02-14 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 9ee4f04 Support broadcast assign for `npi_boolean_mask_assign_tensor` (#17131) add d352673 Fix

[GitHub] [incubator-mxnet] leezu merged pull request #17596: Fix transformer.cu interleaved matmul for cuda arch < 5

2020-02-14 Thread GitBox
leezu merged pull request #17596: Fix transformer.cu interleaved matmul for cuda arch < 5 URL: https://github.com/apache/incubator-mxnet/pull/17596 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-mxnet] reminisce opened a new pull request #17601: Port conv3d dilate > 1 fix to 1.5.x

2020-02-14 Thread GitBox
reminisce opened a new pull request #17601: Port conv3d dilate > 1 fix to 1.5.x URL: https://github.com/apache/incubator-mxnet/pull/17601 Port https://github.com/apache/incubator-mxnet/pull/17491 to 1.5.x. This is an

[GitHub] [incubator-mxnet] TaoLv commented on issue #17595: MKLDNN incompatibility with large tensor (dim >= 2^32) data

2020-02-14 Thread GitBox
TaoLv commented on issue #17595: MKLDNN incompatibility with large tensor (dim >= 2^32) data URL: https://github.com/apache/incubator-mxnet/issues/17595#issuecomment-586552974 I can reproduce the crash. This is an automated

[incubator-mxnet] branch master updated (f619c52 -> 9ee4f04)

2020-02-14 Thread reminisce
This is an automated email from the ASF dual-hosted git repository. reminisce pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from f619c52 Python 2 cleanup (#17583) add 9ee4f04 Support broadcast assign for

[GitHub] [incubator-mxnet] reminisce merged pull request #17131: Support broadcast assign for `npi_boolean_mask_assign_tensor`

2020-02-14 Thread GitBox
reminisce merged pull request #17131: Support broadcast assign for `npi_boolean_mask_assign_tensor` URL: https://github.com/apache/incubator-mxnet/pull/17131 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-mxnet] reminisce opened a new pull request #17600: Fix Non-ASCII character in docstring

2020-02-14 Thread GitBox
reminisce opened a new pull request #17600: Fix Non-ASCII character in docstring URL: https://github.com/apache/incubator-mxnet/pull/17600 ## Description ## As title. Expected to complement https://github.com/apache/incubator-mxnet/pull/17593 for fixing

[GitHub] [incubator-mxnet] TaoLv commented on issue #17595: MKLDNN incompatibility with large tensor (dim >= 2^32) data

2020-02-14 Thread GitBox
TaoLv commented on issue #17595: MKLDNN incompatibility with large tensor (dim >= 2^32) data URL: https://github.com/apache/incubator-mxnet/issues/17595#issuecomment-586548992 Thank you for reporting the issue. I will take a look at this. But my initial thought is that MKL-DNN itself

[GitHub] [incubator-mxnet] TaoLv commented on issue #17576: Fix storage type infer of softmax backward

2020-02-14 Thread GitBox
TaoLv commented on issue #17576: Fix storage type infer of softmax backward URL: https://github.com/apache/incubator-mxnet/pull/17576#issuecomment-586548359 @leeze, Got it. Thank you for the merging. This is an automated

[GitHub] [incubator-mxnet] TaoLv commented on a change in pull request #16735: Use single-bit for mask in dropout operator

2020-02-14 Thread GitBox
TaoLv commented on a change in pull request #16735: Use single-bit for mask in dropout operator URL: https://github.com/apache/incubator-mxnet/pull/16735#discussion_r379713840 ## File path: src/operator/nn/dropout-inl.h ## @@ -152,15 +181,20 @@ class DropoutOp {

[GitHub] [incubator-mxnet] rongzha1 commented on a change in pull request #17265: Add bfloat16 floating-point format support based on AMP

2020-02-14 Thread GitBox
rongzha1 commented on a change in pull request #17265: Add bfloat16 floating-point format support based on AMP URL: https://github.com/apache/incubator-mxnet/pull/17265#discussion_r379713720 ## File path: 3rdparty/mshadow/mshadow/bfloat.h ## @@ -0,0 +1,167 @@ +/*!

[GitHub] [incubator-mxnet] connorgoggins commented on issue #17599: Fixed Embedding op for LT input

2020-02-14 Thread GitBox
connorgoggins commented on issue #17599: Fixed Embedding op for LT input URL: https://github.com/apache/incubator-mxnet/pull/17599#issuecomment-586543671 @access2rohit great question. Here is an example of when sparseEmbedding params are used without my fix, and the resulting error (same

[GitHub] [incubator-mxnet] zhreshold commented on issue #17265: Add bfloat16 floating-point format support based on AMP

2020-02-14 Thread GitBox
zhreshold commented on issue #17265: Add bfloat16 floating-point format support based on AMP URL: https://github.com/apache/incubator-mxnet/pull/17265#issuecomment-586543482 @larroy Do you have idea how to display more logs for the edge tests? It consistently fail at this stage.

[GitHub] [incubator-mxnet] leezu commented on issue #17592: Add cmake build support for macOS static build.

2020-02-14 Thread GitBox
leezu commented on issue #17592: Add cmake build support for macOS static build. URL: https://github.com/apache/incubator-mxnet/pull/17592#issuecomment-586543059 As per https://github.com/leezu/mxnet/commit/0bbd44c45f34c37bf70c90635e29a5fcf2e6f0cc/checks/447232316/logs libtiff 4.1 does

[GitHub] [incubator-mxnet] access2rohit edited a comment on issue #17599: Fixed Embedding op for LT input

2020-02-14 Thread GitBox
access2rohit edited a comment on issue #17599: Fixed Embedding op for LT input URL: https://github.com/apache/incubator-mxnet/pull/17599#issuecomment-586540131 @connorgoggins can you paste output of a case where sparseEmbedding params are used ? Are both used in the same example use case

[GitHub] [incubator-mxnet] access2rohit commented on issue #17599: Fixed Embedding op for LT input

2020-02-14 Thread GitBox
access2rohit commented on issue #17599: Fixed Embedding op for LT input URL: https://github.com/apache/incubator-mxnet/pull/17599#issuecomment-586540131 @connorgoggins can you paste output of a case where sparseEmbedding params are used ?

[incubator-mxnet] branch master updated (40c0c54 -> f619c52)

2020-02-14 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 40c0c54 Fix storage type infer of softmax backward (#17576) add f619c52 Python 2 cleanup (#17583)

[incubator-mxnet] branch master updated (40c0c54 -> f619c52)

2020-02-14 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 40c0c54 Fix storage type infer of softmax backward (#17576) add f619c52 Python 2 cleanup (#17583)

[GitHub] [incubator-mxnet] leezu merged pull request #17583: Python 2 cleanup

2020-02-14 Thread GitBox
leezu merged pull request #17583: Python 2 cleanup URL: https://github.com/apache/incubator-mxnet/pull/17583 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [incubator-mxnet] leezu commented on issue #17596: Fix transformer.cu interleaved matmul for cuda arch < 5

2020-02-14 Thread GitBox
leezu commented on issue #17596: Fix transformer.cu interleaved matmul for cuda arch < 5 URL: https://github.com/apache/incubator-mxnet/pull/17596#issuecomment-586539281 Verified this patch by finetuning Bert on P2 instance. Verification was initially blocked / delayed by

[incubator-mxnet] branch master updated (39b158f -> 40c0c54)

2020-02-14 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 39b158f quantile_scalar (#17572) add 40c0c54 Fix storage type infer of softmax backward (#17576) No

[incubator-mxnet] branch master updated (39b158f -> 40c0c54)

2020-02-14 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 39b158f quantile_scalar (#17572) add 40c0c54 Fix storage type infer of softmax backward (#17576) No

[GitHub] [incubator-mxnet] leezu merged pull request #17576: Fix storage type infer of softmax backward

2020-02-14 Thread GitBox
leezu merged pull request #17576: Fix storage type infer of softmax backward URL: https://github.com/apache/incubator-mxnet/pull/17576 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-mxnet] D-Roberts commented on issue #17590: [numpy] Implement Weibull backward

2020-02-14 Thread GitBox
D-Roberts commented on issue #17590: [numpy] Implement Weibull backward URL: https://github.com/apache/incubator-mxnet/pull/17590#issuecomment-586538103 @haojin2 @xidulu Ready for review. This is an automated message from the

[GitHub] [incubator-mxnet] leezu commented on issue #17576: Fix storage type infer of softmax backward

2020-02-14 Thread GitBox
leezu commented on issue #17576: Fix storage type infer of softmax backward URL: https://github.com/apache/incubator-mxnet/pull/17576#issuecomment-586537272 @TaoLv could you next time revert the buggy commit? I spent some time today to track down the bug, which could have been avoided.

[GitHub] [incubator-mxnet] connorgoggins commented on issue #17599: Fixed Embedding op for LT input

2020-02-14 Thread GitBox
connorgoggins commented on issue #17599: Fixed Embedding op for LT input URL: https://github.com/apache/incubator-mxnet/pull/17599#issuecomment-586535371 @mxnet-label-bot add [pr-awaiting-review] This is an automated message

[GitHub] [incubator-mxnet] connorgoggins opened a new pull request #17599: Fixed Embedding op for LT input

2020-02-14 Thread GitBox
connorgoggins opened a new pull request #17599: Fixed Embedding op for LT input URL: https://github.com/apache/incubator-mxnet/pull/17599 ## Description ## The Embedding op was previously breaking on large tensor (dimension >= 2^32) data. With the following input: ```

[GitHub] [incubator-mxnet] leezu commented on a change in pull request #17592: Add cmake build support for macOS static build.

2020-02-14 Thread GitBox
leezu commented on a change in pull request #17592: Add cmake build support for macOS static build. URL: https://github.com/apache/incubator-mxnet/pull/17592#discussion_r379702140 ## File path: tools/dependencies/curl.sh ## @@ -35,6 +35,7 @@ if [[ ! -f

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

2020-02-14 Thread aaronmarkham
This is an automated email from the ASF dual-hosted git repository. aaronmarkham 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 fe0b694 Bump the

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #16735: Use single-bit for mask in dropout operator

2020-02-14 Thread GitBox
apeforest commented on a change in pull request #16735: Use single-bit for mask in dropout operator URL: https://github.com/apache/incubator-mxnet/pull/16735#discussion_r379688573 ## File path: src/operator/nn/dropout-inl.h ## @@ -152,15 +181,20 @@ class DropoutOp {

[GitHub] [incubator-mxnet] apeforest edited a comment on issue #16735: Use single-bit for mask in dropout operator

2020-02-14 Thread GitBox
apeforest edited a comment on issue #16735: Use single-bit for mask in dropout operator URL: https://github.com/apache/incubator-mxnet/pull/16735#issuecomment-586519348 > Does the slow down come from more computation in the new algorithm or the sub-optimal implementation? @PatricZhao

[GitHub] [incubator-mxnet] apeforest commented on issue #16735: Use single-bit for mask in dropout operator

2020-02-14 Thread GitBox
apeforest commented on issue #16735: Use single-bit for mask in dropout operator URL: https://github.com/apache/incubator-mxnet/pull/16735#issuecomment-586519348 > Does the slow down come from more computation in the new algorithm or the sub-optimal implementation? @PatricZhao The

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #16735: Use single-bit for mask in dropout operator

2020-02-14 Thread GitBox
apeforest commented on a change in pull request #16735: Use single-bit for mask in dropout operator URL: https://github.com/apache/incubator-mxnet/pull/16735#discussion_r379688573 ## File path: src/operator/nn/dropout-inl.h ## @@ -152,15 +181,20 @@ class DropoutOp {

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #16865: [numpy][Do Not Review]add op insert

2020-02-14 Thread GitBox
haojin2 commented on a change in pull request #16865: [numpy][Do Not Review]add op insert URL: https://github.com/apache/incubator-mxnet/pull/16865#discussion_r379682542 ## File path: src/operator/numpy/np_insert_op_tensor-inl.h ## @@ -0,0 +1,229 @@ +/* + * Licensed to

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #16865: [numpy][Do Not Review]add op insert

2020-02-14 Thread GitBox
haojin2 commented on a change in pull request #16865: [numpy][Do Not Review]add op insert URL: https://github.com/apache/incubator-mxnet/pull/16865#discussion_r379682473 ## File path: src/operator/numpy/np_insert_op_tensor-inl.h ## @@ -0,0 +1,229 @@ +/* + * Licensed to

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #16865: [numpy][Do Not Review]add op insert

2020-02-14 Thread GitBox
haojin2 commented on a change in pull request #16865: [numpy][Do Not Review]add op insert URL: https://github.com/apache/incubator-mxnet/pull/16865#discussion_r379682391 ## File path: src/operator/numpy/np_insert_op_slice-inl.h ## @@ -0,0 +1,200 @@ +/* + * Licensed to the

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #16865: [numpy][Do Not Review]add op insert

2020-02-14 Thread GitBox
haojin2 commented on a change in pull request #16865: [numpy][Do Not Review]add op insert URL: https://github.com/apache/incubator-mxnet/pull/16865#discussion_r379682345 ## File path: src/operator/numpy/np_insert_op_slice-inl.h ## @@ -0,0 +1,200 @@ +/* + * Licensed to the

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #16865: [numpy][Do Not Review]add op insert

2020-02-14 Thread GitBox
haojin2 commented on a change in pull request #16865: [numpy][Do Not Review]add op insert URL: https://github.com/apache/incubator-mxnet/pull/16865#discussion_r379682486 ## File path: src/operator/numpy/np_insert_op_tensor-inl.h ## @@ -0,0 +1,229 @@ +/* + * Licensed to

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #16865: [numpy][Do Not Review]add op insert

2020-02-14 Thread GitBox
haojin2 commented on a change in pull request #16865: [numpy][Do Not Review]add op insert URL: https://github.com/apache/incubator-mxnet/pull/16865#discussion_r379682089 ## File path: src/operator/numpy/np_insert_op-inl.h ## @@ -0,0 +1,371 @@ +/* + * Licensed to the

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #16865: [numpy][Do Not Review]add op insert

2020-02-14 Thread GitBox
haojin2 commented on a change in pull request #16865: [numpy][Do Not Review]add op insert URL: https://github.com/apache/incubator-mxnet/pull/16865#discussion_r379682109 ## File path: src/operator/numpy/np_insert_op-inl.h ## @@ -0,0 +1,371 @@ +/* + * Licensed to the

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #16865: [numpy][Do Not Review]add op insert

2020-02-14 Thread GitBox
haojin2 commented on a change in pull request #16865: [numpy][Do Not Review]add op insert URL: https://github.com/apache/incubator-mxnet/pull/16865#discussion_r379681197 ## File path: src/operator/numpy/np_insert_op_tensor.cc ## @@ -0,0 +1,152 @@ +/* + * Licensed to the

[GitHub] [incubator-mxnet] haojin2 commented on issue #17590: [WIP][numpy] Implement Weibull backward

2020-02-14 Thread GitBox
haojin2 commented on issue #17590: [WIP][numpy] Implement Weibull backward URL: https://github.com/apache/incubator-mxnet/pull/17590#issuecomment-586505702 @D-Roberts Thanks for your contribution! Please let @xidulu and/or me know when you think this PR is ready for a review.

[incubator-mxnet] branch master updated (8d887ca -> 39b158f)

2020-02-14 Thread haoj
This is an automated email from the ASF dual-hosted git repository. haoj pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 8d887ca add isposinf isneginf isfinite (#17563) add 39b158f quantile_scalar (#17572) No new revisions

[GitHub] [incubator-mxnet] haojin2 merged pull request #17572: [Numpy] Quantile/Percentile-scalar support

2020-02-14 Thread GitBox
haojin2 merged pull request #17572: [Numpy] Quantile/Percentile-scalar support URL: https://github.com/apache/incubator-mxnet/pull/17572 This is an automated message from the Apache Git Service. To respond to the message,

[incubator-mxnet] branch master updated (1c07771 -> 8d887ca)

2020-02-14 Thread haoj
This is an automated email from the ASF dual-hosted git repository. haoj pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 1c07771 fix CD and remove leftover from #15990 (#17551) add 8d887ca add isposinf isneginf isfinite

[GitHub] [incubator-mxnet] haojin2 merged pull request #17563: [Numpy] add isposinf isneginf isfinite

2020-02-14 Thread GitBox
haojin2 merged pull request #17563: [Numpy] add isposinf isneginf isfinite URL: https://github.com/apache/incubator-mxnet/pull/17563 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-mxnet] leezu closed pull request #17598: Add OS X Staticbuild CI based on Github Actions

2020-02-14 Thread GitBox
leezu closed pull request #17598: Add OS X Staticbuild CI based on Github Actions URL: https://github.com/apache/incubator-mxnet/pull/17598 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-mxnet] leezu commented on issue #17598: Add OS X Staticbuild CI based on Github Actions

2020-02-14 Thread GitBox
leezu commented on issue #17598: Add OS X Staticbuild CI based on Github Actions URL: https://github.com/apache/incubator-mxnet/pull/17598#issuecomment-586497521 Closing to avoid trigger Jenkins on every push. You can track status at above link

[GitHub] [incubator-mxnet] leezu commented on issue #17598: Add OS X Staticbuild CI based on Github Actions

2020-02-14 Thread GitBox
leezu commented on issue #17598: Add OS X Staticbuild CI based on Github Actions URL: https://github.com/apache/incubator-mxnet/pull/17598#issuecomment-586496421 WIP: See build status at https://github.com/leezu/mxnet/actions

[GitHub] [incubator-mxnet] leezu opened a new pull request #17598: Add OS X Staticbuild CI based on Github Actions

2020-02-14 Thread GitBox
leezu opened a new pull request #17598: Add OS X Staticbuild CI based on Github Actions URL: https://github.com/apache/incubator-mxnet/pull/17598 ## Description ## Add OS X Staticbuild CI based on Github Actions This is

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #17597: move stacktrance print to console rather than system.err

2020-02-14 Thread GitBox
ChaiBapchya commented on issue #17597: move stacktrance print to console rather than system.err URL: https://github.com/apache/incubator-mxnet/pull/17597#issuecomment-586492046 @anirudh2290 @access2rohit @marcoabreu Since you reviewed the previous PR, thought you could take a look at

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #17597: move stacktrance print to console rather than system.err

2020-02-14 Thread GitBox
ChaiBapchya commented on issue #17597: move stacktrance print to console rather than system.err URL: https://github.com/apache/incubator-mxnet/pull/17597#issuecomment-586491889 @mxnet-label-bot add [pr-awaiting-review] This

[GitHub] [incubator-mxnet] ChaiBapchya opened a new pull request #17597: move stacktrance print to console rather than system.err

2020-02-14 Thread GitBox
ChaiBapchya opened a new pull request #17597: move stacktrance print to console rather than system.err URL: https://github.com/apache/incubator-mxnet/pull/17597 ## Description ## Right now with #17465 if the build fails, we print the stacktrace using `error.printStackTrace()` function

[GitHub] [incubator-mxnet] access2rohit edited a comment on issue #17595: MKLDNN incompatibility with large tensor (dim >= 2^32) data

2020-02-14 Thread GitBox
access2rohit edited a comment on issue #17595: MKLDNN incompatibility with large tensor (dim >= 2^32) data URL: https://github.com/apache/incubator-mxnet/issues/17595#issuecomment-586451995 @connorgoggins thanks for bringing this up @PatricZhao @TaoLv looks like

[GitHub] [incubator-mxnet] leezu commented on issue #16899: Enable MKL-DNN in pip packages

2020-02-14 Thread GitBox
leezu commented on issue #16899: Enable MKL-DNN in pip packages URL: https://github.com/apache/incubator-mxnet/pull/16899#issuecomment-586477909 @TaoLv Sheng just merged fixes for the CD pipeline. Let's wait 24 hours to verify CD works on master branch. Then we can merge this PR.

[incubator-mxnet] branch master updated (0f35489 -> 1c07771)

2020-02-14 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 0f35489 use py3 for kvstore tests (#17593) add 1c07771 fix CD and remove leftover from #15990

[incubator-mxnet] branch master updated (0f35489 -> 1c07771)

2020-02-14 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 0f35489 use py3 for kvstore tests (#17593) add 1c07771 fix CD and remove leftover from #15990

[GitHub] [incubator-mxnet] szha merged pull request #17551: [CD] fix CD and remove leftover from #15990

2020-02-14 Thread GitBox
szha merged pull request #17551: [CD] fix CD and remove leftover from #15990 URL: https://github.com/apache/incubator-mxnet/pull/17551 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-mxnet] leezu opened a new pull request #17596: Fix transformer.cu interleaved matmul for cuda arch < 5

2020-02-14 Thread GitBox
leezu opened a new pull request #17596: Fix transformer.cu interleaved matmul for cuda arch < 5 URL: https://github.com/apache/incubator-mxnet/pull/17596 ## Description ## `cublasGemmBatchedEx` is only supported for GPU with architecture capabilities equal or greater than 5.0.

[GitHub] [incubator-mxnet] eric-haibin-lin commented on issue #1137: cublas error when using like mx.nd.***

2020-02-14 Thread GitBox
eric-haibin-lin commented on issue #1137: cublas error when using like mx.nd.*** URL: https://github.com/apache/incubator-mxnet/issues/1137#issuecomment-586459620 or just put a `mx.nd.waitall()` This is an automated message

[GitHub] [incubator-mxnet] access2rohit edited a comment on issue #17595: MKLDNN incompatibility with large tensor (dim >= 2^32) data

2020-02-14 Thread GitBox
access2rohit edited a comment on issue #17595: MKLDNN incompatibility with large tensor (dim >= 2^32) data URL: https://github.com/apache/incubator-mxnet/issues/17595#issuecomment-586451995 @connorgoggins thanks for bringing this up @PatricZhao @TaoLv looks like

[GitHub] [incubator-mxnet] access2rohit commented on issue #17595: MKLDNN incompatibility with large tensor (dim >= 2^32) data

2020-02-14 Thread GitBox
access2rohit commented on issue #17595: MKLDNN incompatibility with large tensor (dim >= 2^32) data URL: https://github.com/apache/incubator-mxnet/issues/17595#issuecomment-586451995 @PatricZhao @TaoLv looks like blas=MKL/openblas/none(mnative mxnet) and MKLDNN=OFF is supporting gemm on

[GitHub] [incubator-mxnet] apeforest commented on issue #17595: MKLDNN incompatibility with large tensor (dim >= 2^32) data

2020-02-14 Thread GitBox
apeforest commented on issue #17595: MKLDNN incompatibility with large tensor (dim >= 2^32) data URL: https://github.com/apache/incubator-mxnet/issues/17595#issuecomment-586443098 @PatricZhao Could your team please take a look at this? Thanks!

[GitHub] [incubator-mxnet] access2rohit commented on issue #17595: MKLDNN incompatibility with large tensor (dim >= 2^32) data

2020-02-14 Thread GitBox
access2rohit commented on issue #17595: MKLDNN incompatibility with large tensor (dim >= 2^32) data URL: https://github.com/apache/incubator-mxnet/issues/17595#issuecomment-586439922 @mxnet-label-bot add [MKLDNN] This is an

[GitHub] [incubator-mxnet] connorgoggins opened a new issue #17595: MKLDNN incompatibility with large tensor (dim >= 2^32) data

2020-02-14 Thread GitBox
connorgoggins opened a new issue #17595: MKLDNN incompatibility with large tensor (dim >= 2^32) data URL: https://github.com/apache/incubator-mxnet/issues/17595 ## Description While testing individual ops for large tensor (dimension >= 2^32) input functionality, I found an error in

[GitHub] [incubator-mxnet] zhreshold commented on a change in pull request #17566: [WIP] Discussion on merging BMXNet 2 contributions

2020-02-14 Thread GitBox
zhreshold commented on a change in pull request #17566: [WIP] Discussion on merging BMXNet 2 contributions URL: https://github.com/apache/incubator-mxnet/pull/17566#discussion_r379594847 ## File path: .gitlab-ci.yml ## @@ -0,0 +1,71 @@ +stages: Review comment: this

[GitHub] [incubator-mxnet] zhreshold commented on a change in pull request #17566: [WIP] Discussion on merging BMXNet 2 contributions

2020-02-14 Thread GitBox
zhreshold commented on a change in pull request #17566: [WIP] Discussion on merging BMXNet 2 contributions URL: https://github.com/apache/incubator-mxnet/pull/17566#discussion_r379595075 ## File path: .gitmodules ## @@ -29,3 +29,6 @@ [submodule "3rdparty/nvidia_cub"]

[GitHub] [incubator-mxnet] zhreshold commented on a change in pull request #17566: [WIP] Discussion on merging BMXNet 2 contributions

2020-02-14 Thread GitBox
zhreshold commented on a change in pull request #17566: [WIP] Discussion on merging BMXNet 2 contributions URL: https://github.com/apache/incubator-mxnet/pull/17566#discussion_r379595336 ## File path: README.md ## @@ -1,116 +1,136 @@ - - - - - - - - - - - - - - - - - - -

[GitHub] [incubator-mxnet] aaronmarkham opened a new pull request #17594: upgrade sphinx and autodocsumm

2020-02-14 Thread GitBox
aaronmarkham opened a new pull request #17594: upgrade sphinx and autodocsumm URL: https://github.com/apache/incubator-mxnet/pull/17594 ## Description ## Rollback of #17561 A new version of autodocsumm came out. It supports the most recent version of Sphinx. This reverts

[GitHub] [incubator-mxnet] mjdenkowski commented on issue #17559: [MXNET-1446] Quantization: intgemm matrix multiply wrappers

2020-02-14 Thread GitBox
mjdenkowski commented on issue #17559: [MXNET-1446] Quantization: intgemm matrix multiply wrappers URL: https://github.com/apache/incubator-mxnet/pull/17559#issuecomment-586425627 Hi @TaoLv, thanks for taking a look at this! We understand your comments about MXNet already having

[GitHub] [incubator-mxnet] apeforest edited a comment on issue #16735: Use single-bit for mask in dropout operator

2020-02-14 Thread GitBox
apeforest edited a comment on issue #16735: Use single-bit for mask in dropout operator URL: https://github.com/apache/incubator-mxnet/pull/16735#issuecomment-586062120 @roywei Using the test script in https://github.com/apache/incubator-mxnet/pull/13896 Build | runtime (before) |

[GitHub] [incubator-mxnet] leezu edited a comment on issue #16408: Add MXNet Ops for fast multihead attention

2020-02-14 Thread GitBox
leezu edited a comment on issue #16408: Add MXNet Ops for fast multihead attention URL: https://github.com/apache/incubator-mxnet/pull/16408#issuecomment-586418793 There are two separate problems. `cublasGemmStridedBatchedEx` is buggy and has fixes in 10.2. But

[GitHub] [incubator-mxnet] leezu commented on issue #16408: Add MXNet Ops for fast multihead attention

2020-02-14 Thread GitBox
leezu commented on issue #16408: Add MXNet Ops for fast multihead attention URL: https://github.com/apache/incubator-mxnet/pull/16408#issuecomment-586418793 No. My comment was wrong This is an automated message from the

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

2020-02-14 Thread aaronmarkham
This is an automated email from the ASF dual-hosted git repository. aaronmarkham 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 978b5ea Bump the

[GitHub] [incubator-mxnet] larroy commented on issue #16408: Add MXNet Ops for fast multihead attention

2020-02-14 Thread GitBox
larroy commented on issue #16408: Add MXNet Ops for fast multihead attention URL: https://github.com/apache/incubator-mxnet/pull/16408#issuecomment-586415593 So it's related to cuda and not the arch? This is an automated

[incubator-mxnet] branch master updated (8438d98 -> 0f35489)

2020-02-14 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 8438d98 Implement all miscellaneous ops (#17511) add 0f35489 use py3 for kvstore tests (#17593)

[incubator-mxnet] branch master updated (8438d98 -> 0f35489)

2020-02-14 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 8438d98 Implement all miscellaneous ops (#17511) add 0f35489 use py3 for kvstore tests (#17593)

[GitHub] [incubator-mxnet] szha merged pull request #17593: [CI] use py3 for kvstore tests

2020-02-14 Thread GitBox
szha merged pull request #17593: [CI] use py3 for kvstore tests URL: https://github.com/apache/incubator-mxnet/pull/17593 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-mxnet] szha commented on a change in pull request #17265: Add bfloat16 floating-point format support based on AMP

2020-02-14 Thread GitBox
szha commented on a change in pull request #17265: Add bfloat16 floating-point format support based on AMP URL: https://github.com/apache/incubator-mxnet/pull/17265#discussion_r379517986 ## File path: 3rdparty/mshadow/mshadow/bfloat.h ## @@ -0,0 +1,167 @@ +/*! Review

[GitHub] [incubator-mxnet] TaoLv commented on a change in pull request #17265: Add bfloat16 floating-point format support based on AMP

2020-02-14 Thread GitBox
TaoLv commented on a change in pull request #17265: Add bfloat16 floating-point format support based on AMP URL: https://github.com/apache/incubator-mxnet/pull/17265#discussion_r379484516 ## File path: 3rdparty/mshadow/mshadow/bfloat.h ## @@ -0,0 +1,167 @@ +/*! Review

[GitHub] [incubator-mxnet] TaoLv commented on issue #17265: Add bfloat16 floating-point format support based on AMP

2020-02-14 Thread GitBox
TaoLv commented on issue #17265: Add bfloat16 floating-point format support based on AMP URL: https://github.com/apache/incubator-mxnet/pull/17265#issuecomment-586329605 Hi @leezu, @larroy, this PR can pass the builds for ARM but always hit the time out of the test.

[GitHub] [incubator-mxnet] TaoLv commented on a change in pull request #17318: Enable MKL-DNN FullyConnected backward

2020-02-14 Thread GitBox
TaoLv commented on a change in pull request #17318: Enable MKL-DNN FullyConnected backward URL: https://github.com/apache/incubator-mxnet/pull/17318#discussion_r379465641 ## File path: tests/cpp/include/test_mkldnn.h ## @@ -330,7 +330,7 @@ inline void PrintVerifyMsg(const

[GitHub] [incubator-mxnet] TaoLv commented on a change in pull request #17318: Enable MKL-DNN FullyConnected backward

2020-02-14 Thread GitBox
TaoLv commented on a change in pull request #17318: Enable MKL-DNN FullyConnected backward URL: https://github.com/apache/incubator-mxnet/pull/17318#discussion_r379465354 ## File path: tests/cpp/include/test_mkldnn.h ## @@ -63,24 +63,24 @@ struct TestArrayShapes { };

[GitHub] [incubator-mxnet] sl1pkn07 commented on issue #17514: C++-Package examples broken

2020-02-14 Thread GitBox
sl1pkn07 commented on issue #17514: C++-Package examples broken URL: https://github.com/apache/incubator-mxnet/issues/17514#issuecomment-586274460 try with ~~~ -DCUDA_HOST_COMPILER=/usr/bin/cc-8 \ -DCMAKE_C_COMPILER=/usr/bin/cc-8 \

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

2020-02-14 Thread aaronmarkham
This is an automated email from the ASF dual-hosted git repository. aaronmarkham 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 3fca7d8 Bump the

[GitHub] [incubator-mxnet] ciyongch commented on a change in pull request #17318: Enable MKL-DNN FullyConnected backward

2020-02-14 Thread GitBox
ciyongch commented on a change in pull request #17318: Enable MKL-DNN FullyConnected backward URL: https://github.com/apache/incubator-mxnet/pull/17318#discussion_r379387996 ## File path: tests/cpp/include/test_mkldnn.h ## @@ -63,24 +63,24 @@ struct TestArrayShapes { };

[GitHub] [incubator-mxnet] ciyongch commented on a change in pull request #17318: Enable MKL-DNN FullyConnected backward

2020-02-14 Thread GitBox
ciyongch commented on a change in pull request #17318: Enable MKL-DNN FullyConnected backward URL: https://github.com/apache/incubator-mxnet/pull/17318#discussion_r379389375 ## File path: tests/cpp/include/test_mkldnn.h ## @@ -63,24 +63,24 @@ struct TestArrayShapes { };

[GitHub] [incubator-mxnet] ChiaraXian commented on issue #8148: How do I interpret the JSON compute graph?

2020-02-14 Thread GitBox
ChiaraXian commented on issue #8148: How do I interpret the JSON compute graph? URL: https://github.com/apache/incubator-mxnet/issues/8148#issuecomment-586195655 @aidan-plenert-macdonald Thanks a lot!!! Thank you for your help!!!

[GitHub] [incubator-mxnet] pengzhao-intel commented on a change in pull request #17318: Enable MKL-DNN FullyConnected backward

2020-02-14 Thread GitBox
pengzhao-intel commented on a change in pull request #17318: Enable MKL-DNN FullyConnected backward URL: https://github.com/apache/incubator-mxnet/pull/17318#discussion_r379349568 ## File path: tests/cpp/include/test_mkldnn.h ## @@ -330,7 +330,7 @@ inline void

  1   2   >