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

2019-08-23 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu 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 d3744e6 Bump the publish

[incubator-mxnet] branch marcoabreu-patch-codeowners updated (a797805 -> 55427e3)

2019-08-23 Thread wkcn
This is an automated email from the ASF dual-hosted git repository. wkcn pushed a change to branch marcoabreu-patch-codeowners in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from a797805 Update CODEOWNERS add 55427e3 retrigger ci No new revisions were added by

[GitHub] [incubator-mxnet] zoeygxy commented on issue #15942: Refines NDArray indexing and adds numpy ndarray indexing [DO NOT MERGE YET]

2019-08-23 Thread GitBox
zoeygxy commented on issue #15942: Refines NDArray indexing and adds numpy ndarray indexing [DO NOT MERGE YET] URL: https://github.com/apache/incubator-mxnet/pull/15942#issuecomment-524221129 Fixed SliceAxisGrad_ and now supports subscription on zero-size shapes (eg. x = np.ones((3, 0)))

[GitHub] [incubator-mxnet] hgt312 commented on a change in pull request #15973: Numpy . implement numpy op exp2 with tvm

2019-08-23 Thread GitBox
hgt312 commented on a change in pull request #15973: Numpy . implement numpy op exp2 with tvm URL: https://github.com/apache/incubator-mxnet/pull/15973#discussion_r317022543 ## File path: contrib/tvmop/basic/ufunc.py ## @@ -98,3 +99,71 @@ def backward_vadd_gpu(dtype, ndim

[GitHub] [incubator-mxnet] leezu opened a new pull request #15985: [Python] Remove requests_failed_to_import handling

2019-08-23 Thread GitBox
leezu opened a new pull request #15985: [Python] Remove requests_failed_to_import handling URL: https://github.com/apache/incubator-mxnet/pull/15985 We already required requests package. If a user is missing it nevertheless, fail with a standard error message instead of some hard to und

[GitHub] [incubator-mxnet] mseth10 commented on a change in pull request #15886: [WIP] Graph Partition API

2019-08-23 Thread GitBox
mseth10 commented on a change in pull request #15886: [WIP] Graph Partition API URL: https://github.com/apache/incubator-mxnet/pull/15886#discussion_r317061250 ## File path: tests/python/unittest/test_subgraph_op.py ## @@ -146,11 +146,135 @@ def get_executor(sym, subgraph_b

[GitHub] [incubator-mxnet] HaoLiuHust closed issue #12806: when use python3 to write recodeio, "can't concat str to bytes" will happen

2019-08-23 Thread GitBox
HaoLiuHust closed issue #12806: when use python3 to write recodeio, "can't concat str to bytes" will happen URL: https://github.com/apache/incubator-mxnet/issues/12806 This is an automated message from the Apache Git Service

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

2019-08-23 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu 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 8fd9325 Bump the publish

[GitHub] [incubator-mxnet] leezu opened a new pull request #15986: [DOC] Link tensor_inspector_tutorial.md from FAQ index; Delete develop_and_hack.md

2019-08-23 Thread GitBox
leezu opened a new pull request #15986: [DOC] Link tensor_inspector_tutorial.md from FAQ index; Delete develop_and_hack.md URL: https://github.com/apache/incubator-mxnet/pull/15986 ## Description ## Currently only linked from develop_and_hack.md and add_op_in_backend.md "Develop a

[GitHub] [incubator-mxnet] leezu commented on issue #15986: [DOC] Link tensor_inspector_tutorial.md from FAQ index; Delete develop_and_hack.md

2019-08-23 Thread GitBox
leezu commented on issue #15986: [DOC] Link tensor_inspector_tutorial.md from FAQ index; Delete develop_and_hack.md URL: https://github.com/apache/incubator-mxnet/pull/15986#issuecomment-524325169 FYI @Zha0q1 This is an a

[GitHub] [incubator-mxnet] leezu commented on issue #14983: Bug with arange inside a foreach

2019-08-23 Thread GitBox
leezu commented on issue #14983: Bug with arange inside a foreach URL: https://github.com/apache/incubator-mxnet/issues/14983#issuecomment-524326612 You can use the following workaround to make forward shape inference work for the use-case (thus working around the lack of reverse shape inf

[GitHub] [incubator-mxnet] leezu edited a comment on issue #14983: Bug with arange inside a foreach

2019-08-23 Thread GitBox
leezu edited a comment on issue #14983: Bug with arange inside a foreach URL: https://github.com/apache/incubator-mxnet/issues/14983#issuecomment-524326612 You may be able to use the following workaround to work around the lack of reverse shape inference ``` python ze

[GitHub] [incubator-mxnet] TaoLv commented on issue #15948: Fix a memory misalignment in topk operator

2019-08-23 Thread GitBox
TaoLv commented on issue #15948: Fix a memory misalignment in topk operator URL: https://github.com/apache/incubator-mxnet/pull/15948#issuecomment-524328959 Thank you for the fix, @apeforest . Will this be picked to the v1.5.x branch?

[GitHub] [incubator-mxnet] TaoLv commented on issue #15964: remove Julia cat image for license issue

2019-08-23 Thread GitBox
TaoLv commented on issue #15964: remove Julia cat image for license issue URL: https://github.com/apache/incubator-mxnet/pull/15964#issuecomment-524329500 @roywei Could you please re-trigger the CI? Seems the status was not reflected to this page. --

[GitHub] [incubator-mxnet] TaoLv commented on issue #15803: [v1.5.x] Fix _copy_to on MKLDNN backend (#15637)

2019-08-23 Thread GitBox
TaoLv commented on issue #15803: [v1.5.x] Fix _copy_to on MKLDNN backend (#15637) URL: https://github.com/apache/incubator-mxnet/pull/15803#issuecomment-524330313 @juliusshufan Could you please re-trigger the CI? Thanks. This

[GitHub] [incubator-mxnet] NRauschmayr closed issue #15978: mx.nd.tile: too many reduction axes

2019-08-23 Thread GitBox
NRauschmayr closed issue #15978: mx.nd.tile: too many reduction axes URL: https://github.com/apache/incubator-mxnet/issues/15978 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-mxnet] NRauschmayr commented on issue #15978: mx.nd.tile: too many reduction axes

2019-08-23 Thread GitBox
NRauschmayr commented on issue #15978: mx.nd.tile: too many reduction axes URL: https://github.com/apache/incubator-mxnet/issues/15978#issuecomment-524364855 closing the issue because the user already opened one: https://github.com/apache/incubator-mxnet/issues/15958 -

[GitHub] [incubator-mxnet] gyshi opened a new pull request #15987: Numpy add numpy op rot90

2019-08-23 Thread GitBox
gyshi opened a new pull request #15987: Numpy add numpy op rot90 URL: https://github.com/apache/incubator-mxnet/pull/15987 ## Description ## (https://docs.scipy.org/doc/numpy/reference/generated/numpy.rot90.html) ## Checklist ## ### Essentials ### Please feel free to remove i

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #15757: [Discussion] Unified performance tests and dashboard

2019-08-23 Thread GitBox
ChaiBapchya commented on issue #15757: [Discussion] Unified performance tests and dashboard URL: https://github.com/apache/incubator-mxnet/issues/15757#issuecomment-524378166 @wuxun-zhang For p50,90 and 99 numbers, I've this PR https://github.com/apache/incubator-mxnet/pull/15953

[incubator-mxnet] branch v1.5.x updated (6f7fbc8 -> 42746bc)

2019-08-23 Thread apeforest
This is an automated email from the ASF dual-hosted git repository. apeforest pushed a change to branch v1.5.x in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 6f7fbc8 fix license issues (#15806) (#15860) add 42746bc Fix a memory misalignment in topk operato

[GitHub] [incubator-mxnet] samskalicky commented on a change in pull request #15886: Graph Partition API

2019-08-23 Thread GitBox
samskalicky commented on a change in pull request #15886: Graph Partition API URL: https://github.com/apache/incubator-mxnet/pull/15886#discussion_r317222835 ## File path: src/operator/subgraph/subgraph_property.h ## @@ -221,6 +222,12 @@ class SubgraphProperty { return

[GitHub] [incubator-mxnet] apeforest commented on issue #15948: Fix a memory misalignment in topk operator

2019-08-23 Thread GitBox
apeforest commented on issue #15948: Fix a memory misalignment in topk operator URL: https://github.com/apache/incubator-mxnet/pull/15948#issuecomment-524391345 @TaoLv I have cherry-picked it to v1.5.x (commit hash: 42746bc73e8bcb75bfcadd1398e6f71bc170fa10) Somehow the v1.5.x branch

[GitHub] [incubator-mxnet] apeforest closed pull request #14251: [WIP] Fix unary operator ceil/floor/trunc when data type is integer

2019-08-23 Thread GitBox
apeforest closed pull request #14251: [WIP] Fix unary operator ceil/floor/trunc when data type is integer URL: https://github.com/apache/incubator-mxnet/pull/14251 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #15531: [MXNET-978] Higher Order Gradient Support `arctan`, `arctanh`, `radians`.

2019-08-23 Thread GitBox
larroy commented on a change in pull request #15531: [MXNET-978] Higher Order Gradient Support `arctan`, `arctanh`, `radians`. URL: https://github.com/apache/incubator-mxnet/pull/15531#discussion_r317224729 ## File path: src/operator/tensor/util/node_op_util.h ## @@ -0,0 +

[GitHub] [incubator-mxnet] larroy edited a comment on issue #15808: Add option to choose between OMP implementations

2019-08-23 Thread GitBox
larroy edited a comment on issue #15808: Add option to choose between OMP implementations URL: https://github.com/apache/incubator-mxnet/pull/15808#issuecomment-524392968 Can you guys review again? I made the suggested changes. --

[GitHub] [incubator-mxnet] larroy commented on issue #15808: Add option to choose between OMP implementations

2019-08-23 Thread GitBox
larroy commented on issue #15808: Add option to choose between OMP implementations URL: https://github.com/apache/incubator-mxnet/pull/15808#issuecomment-524392968 Can you guys review again? This is an automated message from

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15808: Add option to choose between OMP implementations

2019-08-23 Thread GitBox
apeforest commented on a change in pull request #15808: Add option to choose between OMP implementations URL: https://github.com/apache/incubator-mxnet/pull/15808#discussion_r317228643 ## File path: CMakeLists.txt ## @@ -431,10 +433,14 @@ endif() # ---[ OpenMP if(USE_O

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15808: Add option to choose between OMP implementations

2019-08-23 Thread GitBox
apeforest commented on a change in pull request #15808: Add option to choose between OMP implementations URL: https://github.com/apache/incubator-mxnet/pull/15808#discussion_r317228948 ## File path: CMakeLists.txt ## @@ -16,15 +16,15 @@ if(EXISTS ${CMAKE_CURRENT_SOURCE_DI

[GitHub] [incubator-mxnet] apeforest commented on issue #15808: Add option to choose between OMP implementations

2019-08-23 Thread GitBox
apeforest commented on issue #15808: Add option to choose between OMP implementations URL: https://github.com/apache/incubator-mxnet/pull/15808#issuecomment-524397493 @larroy I thought the suggested change is to use multi-selection for USE_OPENMP not adding additional variable: It w

[GitHub] [incubator-mxnet] apeforest removed a comment on issue #15808: Add option to choose between OMP implementations

2019-08-23 Thread GitBox
apeforest removed a comment on issue #15808: Add option to choose between OMP implementations URL: https://github.com/apache/incubator-mxnet/pull/15808#issuecomment-524397493 @larroy I thought the suggested change is to use multi-selection for USE_OPENMP not adding additional variable:

[GitHub] [incubator-mxnet] mseth10 commented on issue #15886: Graph Partition API

2019-08-23 Thread GitBox
mseth10 commented on issue #15886: Graph Partition API URL: https://github.com/apache/incubator-mxnet/pull/15886#issuecomment-524397993 @ZhennanQin ping for review This is an automated message from the Apache Git Service. To r

[GitHub] [incubator-mxnet] anirudhacharya commented on issue #15560: Add fp16 support for topk

2019-08-23 Thread GitBox
anirudhacharya commented on issue #15560: Add fp16 support for topk URL: https://github.com/apache/incubator-mxnet/pull/15560#issuecomment-524399420 > Yes, I do have a solution, but it is not quite completed yet. > > 1. I added the `np.float16` testing to `test_ndarray.py:test_order`

[GitHub] [incubator-mxnet] mseth10 commented on a change in pull request #15886: Graph Partition API

2019-08-23 Thread GitBox
mseth10 commented on a change in pull request #15886: Graph Partition API URL: https://github.com/apache/incubator-mxnet/pull/15886#discussion_r317238730 ## File path: src/operator/subgraph/subgraph_property.h ## @@ -221,6 +222,12 @@ class SubgraphProperty { return nul

[GitHub] [incubator-mxnet] reminisce commented on a change in pull request #15984: [DO NOT REVIEW] [DO NOT MERGE] General reduce compute for tvm ops and TVM version of sum

2019-08-23 Thread GitBox
reminisce commented on a change in pull request #15984: [DO NOT REVIEW] [DO NOT MERGE] General reduce compute for tvm ops and TVM version of sum URL: https://github.com/apache/incubator-mxnet/pull/15984#discussion_r317239519 ## File path: src/operator/contrib/tvmop/reduce.cc ##

[GitHub] [incubator-mxnet] reminisce commented on a change in pull request #15984: [DO NOT REVIEW] [DO NOT MERGE] General reduce compute for tvm ops and TVM version of sum

2019-08-23 Thread GitBox
reminisce commented on a change in pull request #15984: [DO NOT REVIEW] [DO NOT MERGE] General reduce compute for tvm ops and TVM version of sum URL: https://github.com/apache/incubator-mxnet/pull/15984#discussion_r317236658 ## File path: contrib/tvmop/basic/reduce.py ## @

[GitHub] [incubator-mxnet] reminisce commented on a change in pull request #15984: [DO NOT REVIEW] [DO NOT MERGE] General reduce compute for tvm ops and TVM version of sum

2019-08-23 Thread GitBox
reminisce commented on a change in pull request #15984: [DO NOT REVIEW] [DO NOT MERGE] General reduce compute for tvm ops and TVM version of sum URL: https://github.com/apache/incubator-mxnet/pull/15984#discussion_r317236946 ## File path: contrib/tvmop/basic/reduce.py ## @

[GitHub] [incubator-mxnet] reminisce commented on a change in pull request #15984: [DO NOT REVIEW] [DO NOT MERGE] General reduce compute for tvm ops and TVM version of sum

2019-08-23 Thread GitBox
reminisce commented on a change in pull request #15984: [DO NOT REVIEW] [DO NOT MERGE] General reduce compute for tvm ops and TVM version of sum URL: https://github.com/apache/incubator-mxnet/pull/15984#discussion_r317236329 ## File path: contrib/tvmop/basic/reduce.py ## @

[GitHub] [incubator-mxnet] mseth10 commented on a change in pull request #15886: Graph Partition API

2019-08-23 Thread GitBox
mseth10 commented on a change in pull request #15886: Graph Partition API URL: https://github.com/apache/incubator-mxnet/pull/15886#discussion_r317239988 ## File path: tests/python/unittest/test_subgraph_op.py ## @@ -146,11 +146,135 @@ def get_executor(sym, subgraph_backend

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #15808: Add option to choose between OMP implementations

2019-08-23 Thread GitBox
larroy commented on a change in pull request #15808: Add option to choose between OMP implementations URL: https://github.com/apache/incubator-mxnet/pull/15808#discussion_r317241672 ## File path: CMakeLists.txt ## @@ -431,10 +433,14 @@ endif() # ---[ OpenMP if(USE_OPEN

[GitHub] [incubator-mxnet] szha merged pull request #15985: [Python] Remove requests_failed_to_import handling

2019-08-23 Thread GitBox
szha merged pull request #15985: [Python] Remove requests_failed_to_import handling URL: https://github.com/apache/incubator-mxnet/pull/15985 This is an automated message from the Apache Git Service. To respond to the messag

[incubator-mxnet] branch master updated (d8c2d85 -> 717d883)

2019-08-23 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 d8c2d85 update support MKLDNN BN conditions (#15870) add 717d883 Remove requests_failed_to_import h

[GitHub] [incubator-mxnet] reminisce commented on a change in pull request #15984: [DO NOT REVIEW] [DO NOT MERGE] General reduce compute for tvm ops and TVM version of sum

2019-08-23 Thread GitBox
reminisce commented on a change in pull request #15984: [DO NOT REVIEW] [DO NOT MERGE] General reduce compute for tvm ops and TVM version of sum URL: https://github.com/apache/incubator-mxnet/pull/15984#discussion_r317239519 ## File path: src/operator/contrib/tvmop/reduce.cc ##

[GitHub] [incubator-mxnet] eric-haibin-lin commented on issue #14836: Refactor AGInfo and Imperative

2019-08-23 Thread GitBox
eric-haibin-lin commented on issue #14836: Refactor AGInfo and Imperative URL: https://github.com/apache/incubator-mxnet/pull/14836#issuecomment-524419172 LGTM This is an automated message from the Apache Git Service. To respo

[GitHub] [incubator-mxnet] sxjscience commented on a change in pull request #15560: Add fp16 support for topk

2019-08-23 Thread GitBox
sxjscience commented on a change in pull request #15560: Add fp16 support for topk URL: https://github.com/apache/incubator-mxnet/pull/15560#discussion_r317255526 ## File path: src/operator/tensor/sort_op-inl.cuh ## @@ -227,18 +251,18 @@ SortByKeyImpl(mshadow::Tensor keys,

[GitHub] [incubator-mxnet] sxjscience commented on a change in pull request #15560: Add fp16 support for topk

2019-08-23 Thread GitBox
sxjscience commented on a change in pull request #15560: Add fp16 support for topk URL: https://github.com/apache/incubator-mxnet/pull/15560#discussion_r317256691 ## File path: tests/python/unittest/test_operator.py ## @@ -4372,6 +4347,45 @@ def get_large_matrix():

[GitHub] [incubator-mxnet] drivanov commented on a change in pull request #15560: Add fp16 support for topk

2019-08-23 Thread GitBox
drivanov commented on a change in pull request #15560: Add fp16 support for topk URL: https://github.com/apache/incubator-mxnet/pull/15560#discussion_r317257895 ## File path: src/operator/tensor/sort_op-inl.cuh ## @@ -227,18 +251,18 @@ SortByKeyImpl(mshadow::Tensor keys,

[GitHub] [incubator-mxnet] sxjscience commented on a change in pull request #15560: Add fp16 support for topk

2019-08-23 Thread GitBox
sxjscience commented on a change in pull request #15560: Add fp16 support for topk URL: https://github.com/apache/incubator-mxnet/pull/15560#discussion_r317258904 ## File path: src/operator/tensor/sort_op-inl.cuh ## @@ -227,18 +251,18 @@ SortByKeyImpl(mshadow::Tensor keys,

[GitHub] [incubator-mxnet] drivanov commented on a change in pull request #15560: Add fp16 support for topk

2019-08-23 Thread GitBox
drivanov commented on a change in pull request #15560: Add fp16 support for topk URL: https://github.com/apache/incubator-mxnet/pull/15560#discussion_r317257895 ## File path: src/operator/tensor/sort_op-inl.cuh ## @@ -227,18 +251,18 @@ SortByKeyImpl(mshadow::Tensor keys,

[GitHub] [incubator-mxnet] reminisce merged pull request #15831: Numpy-compatible element-wise unary operators upstream

2019-08-23 Thread GitBox
reminisce merged pull request #15831: Numpy-compatible element-wise unary operators upstream URL: https://github.com/apache/incubator-mxnet/pull/15831 This is an automated message from the Apache Git Service. To respond to t

[incubator-mxnet] branch master updated (717d883 -> 8d0b895)

2019-08-23 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 717d883 Remove requests_failed_to_import handling add 8d0b895 np elemwise unary ops upstream (#158

[GitHub] [incubator-mxnet] anirudhacharya commented on a change in pull request #15560: Add fp16 support for topk

2019-08-23 Thread GitBox
anirudhacharya commented on a change in pull request #15560: Add fp16 support for topk URL: https://github.com/apache/incubator-mxnet/pull/15560#discussion_r317262979 ## File path: tests/python/unittest/test_operator.py ## @@ -4372,6 +4347,45 @@ def get_large_matrix():

[GitHub] [incubator-mxnet] anirudhacharya commented on a change in pull request #15560: Add fp16 support for topk

2019-08-23 Thread GitBox
anirudhacharya commented on a change in pull request #15560: Add fp16 support for topk URL: https://github.com/apache/incubator-mxnet/pull/15560#discussion_r317262979 ## File path: tests/python/unittest/test_operator.py ## @@ -4372,6 +4347,45 @@ def get_large_matrix():

[GitHub] [incubator-mxnet] roywei merged pull request #15964: remove Julia cat image for license issue

2019-08-23 Thread GitBox
roywei merged pull request #15964: remove Julia cat image for license issue URL: https://github.com/apache/incubator-mxnet/pull/15964 This is an automated message from the Apache Git Service. To respond to the message, please

[incubator-mxnet] branch master updated (8d0b895 -> 1eb1925)

2019-08-23 Thread roywei
This is an automated email from the ASF dual-hosted git repository. roywei pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 8d0b895 np elemwise unary ops upstream (#15831) add 1eb1925 remove Julia cat image for license issue

[GitHub] [incubator-mxnet] samskalicky commented on issue #15969: [WIP] Partitioning Gluon HybridBlocks

2019-08-23 Thread GitBox
samskalicky commented on issue #15969: [WIP] Partitioning Gluon HybridBlocks URL: https://github.com/apache/incubator-mxnet/pull/15969#issuecomment-524430581 Waiting on #15886 to be merged to re-use optimize_for API call on symbol

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15855: [Numpy] cumprod

2019-08-23 Thread GitBox
haojin2 commented on a change in pull request #15855: [Numpy] cumprod URL: https://github.com/apache/incubator-mxnet/pull/15855#discussion_r317273874 ## File path: contrib/tvmop/core/fromnumeric.py ## @@ -0,0 +1,203 @@ +# Licensed to the Apache Software Foundation (ASF) und

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15902: Numpy add numpy op roll

2019-08-23 Thread GitBox
haojin2 commented on a change in pull request #15902: Numpy add numpy op roll URL: https://github.com/apache/incubator-mxnet/pull/15902#discussion_r317276993 ## File path: src/operator/numpy/np_roll_op.cc ## @@ -0,0 +1,103 @@ +/* + * Licensed to the Apache Software Foundati

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15902: Numpy add numpy op roll

2019-08-23 Thread GitBox
haojin2 commented on a change in pull request #15902: Numpy add numpy op roll URL: https://github.com/apache/incubator-mxnet/pull/15902#discussion_r317276993 ## File path: src/operator/numpy/np_roll_op.cc ## @@ -0,0 +1,103 @@ +/* + * Licensed to the Apache Software Foundati

[GitHub] [incubator-mxnet] sxjscience merged pull request #15560: Add fp16 support for topk

2019-08-23 Thread GitBox
sxjscience merged pull request #15560: Add fp16 support for topk URL: https://github.com/apache/incubator-mxnet/pull/15560 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-mxnet] sxjscience closed issue #14125: FP16 support for topK

2019-08-23 Thread GitBox
sxjscience closed issue #14125: FP16 support for topK URL: https://github.com/apache/incubator-mxnet/issues/14125 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15902: Numpy add numpy op roll

2019-08-23 Thread GitBox
haojin2 commented on a change in pull request #15902: Numpy add numpy op roll URL: https://github.com/apache/incubator-mxnet/pull/15902#discussion_r317277304 ## File path: src/operator/numpy/np_roll_op.cc ## @@ -0,0 +1,103 @@ +/* Review comment: This op seems to belon

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

2019-08-23 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu 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 e3b3f11 Bump the publish

[incubator-mxnet] branch master updated (1eb1925 -> cba7c4e)

2019-08-23 Thread sxjscience
This is an automated email from the ASF dual-hosted git repository. sxjscience pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 1eb1925 remove Julia cat image for license issue (#15964) add cba7c4e Add fp16 support for topk (

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #15967: Tracking Large Tensor vector test coverage

2019-08-23 Thread GitBox
ChaiBapchya commented on issue #15967: Tracking Large Tensor vector test coverage URL: https://github.com/apache/incubator-mxnet/issues/15967#issuecomment-524446296 From my PR #15941 SequenceLast ``` == ERROR:

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #14174: CI fails for unix cpu Static build CPU 14.04

2019-08-23 Thread GitBox
ChaiBapchya commented on issue #14174: CI fails for unix cpu Static build CPU 14.04 URL: https://github.com/apache/incubator-mxnet/issues/14174#issuecomment-524446743 Again #15941 http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Funix-cpu/detail/PR-15

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15811: [MXNET-891] Support tuple of scales in upsample operator

2019-08-23 Thread GitBox
apeforest commented on a change in pull request #15811: [MXNET-891] Support tuple of scales in upsample operator URL: https://github.com/apache/incubator-mxnet/pull/15811#discussion_r317290135 ## File path: src/operator/nn/upsampling-inl.h ## @@ -48,16 +48,18 @@ enum UpSam

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15811: [MXNET-891] Support tuple of scales in upsample operator

2019-08-23 Thread GitBox
apeforest commented on a change in pull request #15811: [MXNET-891] Support tuple of scales in upsample operator URL: https://github.com/apache/incubator-mxnet/pull/15811#discussion_r317295370 ## File path: src/operator/nn/upsampling-inl.h ## @@ -84,6 +86,23 @@ struct UpSa

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15811: [MXNET-891] Support tuple of scales in upsample operator

2019-08-23 Thread GitBox
apeforest commented on a change in pull request #15811: [MXNET-891] Support tuple of scales in upsample operator URL: https://github.com/apache/incubator-mxnet/pull/15811#discussion_r317295616 ## File path: tests/python/unittest/test_operator.py ## @@ -1599,14 +1607,21 @@

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #14836: Refactor AGInfo and Imperative

2019-08-23 Thread GitBox
apeforest commented on a change in pull request #14836: Refactor AGInfo and Imperative URL: https://github.com/apache/incubator-mxnet/pull/14836#discussion_r317296584 ## File path: src/imperative/imperative.cc ## @@ -124,29 +124,34 @@ void Imperative::MarkVariables( c

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #14836: Refactor AGInfo and Imperative

2019-08-23 Thread GitBox
apeforest commented on a change in pull request #14836: Refactor AGInfo and Imperative URL: https://github.com/apache/incubator-mxnet/pull/14836#discussion_r317297099 ## File path: src/imperative/imperative.cc ## @@ -316,181 +312,223 @@ std::vector Imperative::Backward(

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #14836: Refactor AGInfo and Imperative

2019-08-23 Thread GitBox
apeforest commented on a change in pull request #14836: Refactor AGInfo and Imperative URL: https://github.com/apache/incubator-mxnet/pull/14836#discussion_r317297602 ## File path: src/imperative/imperative.cc ## @@ -316,181 +312,223 @@ std::vector Imperative::Backward(

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #14836: Refactor AGInfo and Imperative

2019-08-23 Thread GitBox
apeforest commented on a change in pull request #14836: Refactor AGInfo and Imperative URL: https://github.com/apache/incubator-mxnet/pull/14836#discussion_r317297894 ## File path: src/imperative/imperative.cc ## @@ -316,181 +308,220 @@ std::vector Imperative::Backward(

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops

2019-08-23 Thread GitBox
access2rohit commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops URL: https://github.com/apache/incubator-mxnet/pull/15943#discussion_r317299711 ## File path: tests/nightly/test_large_vector.py ## @@ -33,6 +35,29

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops

2019-08-23 Thread GitBox
access2rohit commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops URL: https://github.com/apache/incubator-mxnet/pull/15943#discussion_r317299759 ## File path: tests/nightly/test_large_vector.py ## @@ -33,6 +35,29

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

2019-08-23 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu 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 686a850 Bump the publish

[GitHub] [incubator-mxnet] benhe2011 commented on a change in pull request #15811: [MXNET-891] Support tuple of scales in upsample operator

2019-08-23 Thread GitBox
benhe2011 commented on a change in pull request #15811: [MXNET-891] Support tuple of scales in upsample operator URL: https://github.com/apache/incubator-mxnet/pull/15811#discussion_r317305265 ## File path: src/operator/nn/upsampling-inl.h ## @@ -84,6 +86,23 @@ struct UpSa

[GitHub] [incubator-mxnet] benhe2011 commented on a change in pull request #15811: [MXNET-891] Support tuple of scales in upsample operator

2019-08-23 Thread GitBox
benhe2011 commented on a change in pull request #15811: [MXNET-891] Support tuple of scales in upsample operator URL: https://github.com/apache/incubator-mxnet/pull/15811#discussion_r317305547 ## File path: src/operator/nn/upsampling-inl.h ## @@ -48,16 +48,18 @@ enum UpSam

[GitHub] [incubator-mxnet] ChaiBapchya edited a comment on issue #15967: Tracking Large Tensor vector test coverage

2019-08-23 Thread GitBox
ChaiBapchya edited a comment on issue #15967: Tracking Large Tensor vector test coverage URL: https://github.com/apache/incubator-mxnet/issues/15967#issuecomment-524446296 From my PR #15941 SequenceLast ``` ==

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops

2019-08-23 Thread GitBox
apeforest commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops URL: https://github.com/apache/incubator-mxnet/pull/15943#discussion_r317309189 ## File path: python/mxnet/test_utils.py ## @@ -262,6 +262,19 @@ def a

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops

2019-08-23 Thread GitBox
apeforest commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops URL: https://github.com/apache/incubator-mxnet/pull/15943#discussion_r317309985 ## File path: tests/nightly/test_large_array.py ## @@ -19,7 +19,7 @@

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops

2019-08-23 Thread GitBox
apeforest commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops URL: https://github.com/apache/incubator-mxnet/pull/15943#discussion_r317310057 ## File path: tests/nightly/test_large_vector.py ## @@ -18,13 +18,15 @

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops

2019-08-23 Thread GitBox
apeforest commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops URL: https://github.com/apache/incubator-mxnet/pull/15943#discussion_r317310175 ## File path: tests/nightly/test_large_vector.py ## @@ -33,6 +35,273 @

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops

2019-08-23 Thread GitBox
apeforest commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops URL: https://github.com/apache/incubator-mxnet/pull/15943#discussion_r317310275 ## File path: tests/nightly/test_large_vector.py ## @@ -18,13 +18,15 @

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops

2019-08-23 Thread GitBox
apeforest commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops URL: https://github.com/apache/incubator-mxnet/pull/15943#discussion_r317310400 ## File path: tests/nightly/test_large_vector.py ## @@ -33,6 +35,273 @

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops

2019-08-23 Thread GitBox
apeforest commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops URL: https://github.com/apache/incubator-mxnet/pull/15943#discussion_r317310379 ## File path: tests/nightly/test_large_vector.py ## @@ -33,6 +35,273 @

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops

2019-08-23 Thread GitBox
apeforest commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops URL: https://github.com/apache/incubator-mxnet/pull/15943#discussion_r317310366 ## File path: tests/nightly/test_large_vector.py ## @@ -33,6 +35,273 @

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops

2019-08-23 Thread GitBox
apeforest commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops URL: https://github.com/apache/incubator-mxnet/pull/15943#discussion_r317310429 ## File path: tests/nightly/test_large_vector.py ## @@ -33,6 +35,273 @

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15953: Add Median, p50, p99 to python profiler

2019-08-23 Thread GitBox
apeforest commented on a change in pull request #15953: Add Median,p50,p99 to python profiler URL: https://github.com/apache/incubator-mxnet/pull/15953#discussion_r317311170 ## File path: benchmark/opperf/utils/common_utils.py ## @@ -108,12 +112,19 @@ def _prepare_op_bench

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops

2019-08-23 Thread GitBox
access2rohit commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops URL: https://github.com/apache/incubator-mxnet/pull/15943#discussion_r317311976 ## File path: python/mxnet/test_utils.py ## @@ -262,6 +262,19 @@ de

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops

2019-08-23 Thread GitBox
access2rohit commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops URL: https://github.com/apache/incubator-mxnet/pull/15943#discussion_r317312251 ## File path: tests/nightly/test_large_array.py ## @@ -19,7 +19,7 @

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops

2019-08-23 Thread GitBox
apeforest commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops URL: https://github.com/apache/incubator-mxnet/pull/15943#discussion_r317309189 ## File path: python/mxnet/test_utils.py ## @@ -262,6 +262,19 @@ def a

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops

2019-08-23 Thread GitBox
access2rohit commented on a change in pull request #15943: Added tests to verify Large Vector Support for initial set of ops URL: https://github.com/apache/incubator-mxnet/pull/15943#discussion_r317311976 ## File path: python/mxnet/test_utils.py ## @@ -262,6 +262,19 @@ de

[GitHub] [incubator-mxnet] IvyBazan commented on issue #15914: Correct ONNX documentation

2019-08-23 Thread GitBox
IvyBazan commented on issue #15914: Correct ONNX documentation URL: https://github.com/apache/incubator-mxnet/pull/15914#issuecomment-524479094 LGTM, thanks for the fix! You may want to re-trigger CI This is an automated m

[GitHub] [incubator-mxnet] samskalicky commented on a change in pull request #15811: [MXNET-891] Support tuple of scales in upsample operator

2019-08-23 Thread GitBox
samskalicky commented on a change in pull request #15811: [MXNET-891] Support tuple of scales in upsample operator URL: https://github.com/apache/incubator-mxnet/pull/15811#discussion_r317319361 ## File path: src/operator/nn/upsampling-inl.h ## @@ -48,16 +48,18 @@ enum UpS

[GitHub] [incubator-mxnet] samskalicky commented on a change in pull request #15811: [MXNET-891] Support tuple of scales in upsample operator

2019-08-23 Thread GitBox
samskalicky commented on a change in pull request #15811: [MXNET-891] Support tuple of scales in upsample operator URL: https://github.com/apache/incubator-mxnet/pull/15811#discussion_r317319555 ## File path: src/operator/nn/upsampling-inl.h ## @@ -84,6 +86,21 @@ struct Up

[GitHub] [incubator-mxnet] samskalicky commented on a change in pull request #15811: [MXNET-891] Support tuple of scales in upsample operator

2019-08-23 Thread GitBox
samskalicky commented on a change in pull request #15811: [MXNET-891] Support tuple of scales in upsample operator URL: https://github.com/apache/incubator-mxnet/pull/15811#discussion_r317319555 ## File path: src/operator/nn/upsampling-inl.h ## @@ -84,6 +86,21 @@ struct Up

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #15953: Add Median, p50, p99 to python profiler

2019-08-23 Thread GitBox
ChaiBapchya commented on a change in pull request #15953: Add Median,p50,p99 to python profiler URL: https://github.com/apache/incubator-mxnet/pull/15953#discussion_r317319651 ## File path: benchmark/opperf/utils/common_utils.py ## @@ -108,12 +112,19 @@ def _prepare_op_ben

[GitHub] [incubator-mxnet] samskalicky commented on issue #15811: [MXNET-891] Support tuple of scales in upsample operator

2019-08-23 Thread GitBox
samskalicky commented on issue #15811: [MXNET-891] Support tuple of scales in upsample operator URL: https://github.com/apache/incubator-mxnet/pull/15811#issuecomment-524481100 Is there a test somewhere that you test upsample using a scalar instead of a tuple?

[GitHub] [incubator-mxnet] rondogency commented on a change in pull request #15886: Graph Partition API

2019-08-23 Thread GitBox
rondogency commented on a change in pull request #15886: Graph Partition API URL: https://github.com/apache/incubator-mxnet/pull/15886#discussion_r317321471 ## File path: tests/python/unittest/test_subgraph_op.py ## @@ -146,11 +146,137 @@ def get_executor(sym, subgraph_back

[GitHub] [incubator-mxnet] IvyBazan commented on issue #15869: [DOC] Fix doc for nn.Embedding, nn.Dense and nd.Embedding

2019-08-23 Thread GitBox
IvyBazan commented on issue #15869: [DOC] Fix doc for nn.Embedding, nn.Dense and nd.Embedding URL: https://github.com/apache/incubator-mxnet/pull/15869#issuecomment-524482563 You may want to trigger CI to run again This is an

  1   2   >