[GitHub] [incubator-mxnet] apeforest commented on issue #15499: Improve diagnose.py, adding build features info and binary library path.

2019-07-12 Thread GitBox
apeforest commented on issue #15499: Improve diagnose.py, adding build features info and binary library path. URL: https://github.com/apache/incubator-mxnet/pull/15499#issuecomment-511091560 One minor comment: the ✖ etc. symbols were not recognized when I import the txt file into MS

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15499: Improve diagnose.py, adding build features info and binary library path.

2019-07-12 Thread GitBox
apeforest commented on a change in pull request #15499: Improve diagnose.py, adding build features info and binary library path. URL: https://github.com/apache/incubator-mxnet/pull/15499#discussion_r303192850 ## File path: tools/diagnose.py ## @@ -105,13 +110,20 @@ def

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15499: Improve diagnose.py, adding build features info and binary library path.

2019-07-12 Thread GitBox
apeforest commented on a change in pull request #15499: Improve diagnose.py, adding build features info and binary library path. URL: https://github.com/apache/incubator-mxnet/pull/15499#discussion_r303192850 ## File path: tools/diagnose.py ## @@ -105,13 +110,20 @@ def

[GitHub] [incubator-mxnet] intgogo opened a new issue #15529: MXNET_CUDNN_AUTOTUNE_DEFAULT problems

2019-07-12 Thread GitBox
intgogo opened a new issue #15529: MXNET_CUDNN_AUTOTUNE_DEFAULT problems URL: https://github.com/apache/incubator-mxnet/issues/15529 When I set MXNET_CUDNN_AUTOTUNE_DEFAULT =1, it won't stop.WTF~ ``` [12:39:44] /root/lilw/mxnet/src/operator/nn/./cudnn/./cudnn_algoreg-inl.h:97:

[GitHub] [incubator-mxnet] mxnet-label-bot commented on issue #15529: MXNET_CUDNN_AUTOTUNE_DEFAULT problems

2019-07-12 Thread GitBox
mxnet-label-bot commented on issue #15529: MXNET_CUDNN_AUTOTUNE_DEFAULT problems URL: https://github.com/apache/incubator-mxnet/issues/15529#issuecomment-511088398 Hey, this is the MXNet Label Bot. Thank you for submitting the issue! I will try and suggest some labels so that the

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303189996 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,763 @@ +/* + *

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303189612 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,763 @@ +/* + *

[GitHub] [incubator-mxnet] anirudh2290 opened a new pull request #15528: [WIP] Add AMP Conversion support for BucketingModule

2019-07-12 Thread GitBox
anirudh2290 opened a new pull request #15528: [WIP] Add AMP Conversion support for BucketingModule URL: https://github.com/apache/incubator-mxnet/pull/15528 ## Description ## - Add conversion APIs for bucketing module - Add support for saving and loading BucketingModule from a

[GitHub] [incubator-mxnet] anirudh2290 commented on issue #13335: MXPredCreate lost dtype

2019-07-12 Thread GitBox
anirudh2290 commented on issue #13335: MXPredCreate lost dtype URL: https://github.com/apache/incubator-mxnet/issues/13335#issuecomment-511080169 Resolved by #15245 This is an automated message from the Apache Git Service.

[GitHub] [incubator-mxnet] anirudh2290 closed issue #13335: MXPredCreate lost dtype

2019-07-12 Thread GitBox
anirudh2290 closed issue #13335: MXPredCreate lost dtype URL: https://github.com/apache/incubator-mxnet/issues/13335 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[incubator-mxnet] branch master updated (cbb6f7f -> d677d1a)

2019-07-12 Thread anirudh2290
This is an automated email from the ASF dual-hosted git repository. anirudh2290 pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from cbb6f7f Docs: Fix misprints (#15505) add d677d1a FP16 Support for C Predict API (#15245) No

[GitHub] [incubator-mxnet] anirudh2290 merged pull request #15245: FP16 Support for C Predict API

2019-07-12 Thread GitBox
anirudh2290 merged pull request #15245: FP16 Support for C Predict API URL: https://github.com/apache/incubator-mxnet/pull/15245 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-mxnet] anirudh2290 closed issue #14159: [Feature Request] Support fp16 for C Predict API

2019-07-12 Thread GitBox
anirudh2290 closed issue #14159: [Feature Request] Support fp16 for C Predict API URL: https://github.com/apache/incubator-mxnet/issues/14159 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-mxnet] braindotai removed a comment on issue #12185: from_logits definition seems different from what is expected?

2019-07-12 Thread GitBox
braindotai removed a comment on issue #12185: from_logits definition seems different from what is expected? URL: https://github.com/apache/incubator-mxnet/issues/12185#issuecomment-510939653 [Check this

[incubator-mxnet] branch master updated: Docs: Fix misprints (#15505)

2019-07-12 Thread wkcn
This is an automated email from the ASF dual-hosted git repository. wkcn 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 cbb6f7f Docs: Fix misprints (#15505)

[GitHub] [incubator-mxnet] wkcn merged pull request #15505: Docs: Fix misprints

2019-07-12 Thread GitBox
wkcn merged pull request #15505: Docs: Fix misprints URL: https://github.com/apache/incubator-mxnet/pull/15505 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [incubator-mxnet] wkcn commented on issue #14894: Accelerate ROIPooling layer

2019-07-12 Thread GitBox
wkcn commented on issue #14894: Accelerate ROIPooling layer URL: https://github.com/apache/incubator-mxnet/pull/14894#issuecomment-511070139 @larroy I think the main reason is that the indices of the maximum numbers are saved in this PR.

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #15516: Fix memory leak reported by ASAN in NNVM to ONNX conversion

2019-07-12 Thread GitBox
ChaiBapchya commented on issue #15516: Fix memory leak reported by ASAN in NNVM to ONNX conversion URL: https://github.com/apache/incubator-mxnet/pull/15516#issuecomment-511069536 Gotcha! Makes sense! This is an automated

[GitHub] [incubator-mxnet] larroy edited a comment on issue #14894: Accelerate ROIPooling layer

2019-07-12 Thread GitBox
larroy edited a comment on issue #14894: Accelerate ROIPooling layer URL: https://github.com/apache/incubator-mxnet/pull/14894#issuecomment-511066311 This is great, is the performance increase only due to type changes as in the description?

[GitHub] [incubator-mxnet] larroy commented on issue #14894: Accelerate ROIPooling layer

2019-07-12 Thread GitBox
larroy commented on issue #14894: Accelerate ROIPooling layer URL: https://github.com/apache/incubator-mxnet/pull/14894#issuecomment-511066311 This is great, is the performance increase only due to type changes as in the description?

[GitHub] [incubator-mxnet] KellenSunderland opened a new pull request #15527: Small typo fixes in batch_norm-inl.h

2019-07-12 Thread GitBox
KellenSunderland opened a new pull request #15527: Small typo fixes in batch_norm-inl.h URL: https://github.com/apache/incubator-mxnet/pull/15527 ## Description ## Small typo fixes in batch_norm-inl.h This is an automated

[incubator-mxnet] branch KellenSunderland-patch-2 created (now 6c38f3e)

2019-07-12 Thread kellen
This is an automated email from the ASF dual-hosted git repository. kellen pushed a change to branch KellenSunderland-patch-2 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. at 6c38f3e Small typo fixes in batch_norm-inl.h No new revisions were added by this

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
ChaiBapchya commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303175784 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,763 @@ +/* + *

[GitHub] [incubator-mxnet] zhreshold commented on issue #14894: Accelerate ROIPooling layer

2019-07-12 Thread GitBox
zhreshold commented on issue #14894: Accelerate ROIPooling layer URL: https://github.com/apache/incubator-mxnet/pull/14894#issuecomment-511060877 Would be nice to apply to ROIAlign similarly since GluonCV have transitioned to use ROIAlign in recent networks.

[incubator-mxnet] branch master updated (2565fa2 -> 9c5acb4)

2019-07-12 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 2565fa2 fix nightly CI failure (#15452) add 9c5acb4 Accelerate ROIPooling layer (#14894) No new

[GitHub] [incubator-mxnet] sxjscience merged pull request #14894: Accelerate ROIPooling layer

2019-07-12 Thread GitBox
sxjscience merged pull request #14894: Accelerate ROIPooling layer URL: https://github.com/apache/incubator-mxnet/pull/14894 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #15517: [WIP]Tensor Inspector Tutorial

2019-07-12 Thread GitBox
ChaiBapchya commented on a change in pull request #15517: [WIP]Tensor Inspector Tutorial URL: https://github.com/apache/incubator-mxnet/pull/15517#discussion_r303174181 ## File path: docs/faq/tensor_inspector_tutorial.md ## @@ -0,0 +1,164 @@ + + + + + + + + + + + + + + +

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #15517: [WIP]Tensor Inspector Tutorial

2019-07-12 Thread GitBox
ChaiBapchya commented on a change in pull request #15517: [WIP]Tensor Inspector Tutorial URL: https://github.com/apache/incubator-mxnet/pull/15517#discussion_r303174854 ## File path: docs/faq/tensor_inspector_tutorial.md ## @@ -0,0 +1,164 @@ + + + + + + + + + + + + + + +

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #15517: [WIP]Tensor Inspector Tutorial

2019-07-12 Thread GitBox
ChaiBapchya commented on a change in pull request #15517: [WIP]Tensor Inspector Tutorial URL: https://github.com/apache/incubator-mxnet/pull/15517#discussion_r303174924 ## File path: docs/faq/tensor_inspector_tutorial.md ## @@ -0,0 +1,164 @@ + + + + + + + + + + + + + + +

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #15517: [WIP]Tensor Inspector Tutorial

2019-07-12 Thread GitBox
ChaiBapchya commented on a change in pull request #15517: [WIP]Tensor Inspector Tutorial URL: https://github.com/apache/incubator-mxnet/pull/15517#discussion_r303174371 ## File path: docs/faq/tensor_inspector_tutorial.md ## @@ -0,0 +1,164 @@ + + + + + + + + + + + + + + +

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #15517: [WIP]Tensor Inspector Tutorial

2019-07-12 Thread GitBox
ChaiBapchya commented on a change in pull request #15517: [WIP]Tensor Inspector Tutorial URL: https://github.com/apache/incubator-mxnet/pull/15517#discussion_r303173725 ## File path: docs/faq/tensor_inspector_tutorial.md ## @@ -0,0 +1,164 @@ + + + + + + + + + + + + + + +

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #15517: [WIP]Tensor Inspector Tutorial

2019-07-12 Thread GitBox
ChaiBapchya commented on a change in pull request #15517: [WIP]Tensor Inspector Tutorial URL: https://github.com/apache/incubator-mxnet/pull/15517#discussion_r303173914 ## File path: docs/faq/tensor_inspector_tutorial.md ## @@ -0,0 +1,164 @@ + + + + + + + + + + + + + + +

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #15517: [WIP]Tensor Inspector Tutorial

2019-07-12 Thread GitBox
ChaiBapchya commented on a change in pull request #15517: [WIP]Tensor Inspector Tutorial URL: https://github.com/apache/incubator-mxnet/pull/15517#discussion_r303174576 ## File path: docs/faq/tensor_inspector_tutorial.md ## @@ -0,0 +1,164 @@ + + + + + + + + + + + + + + +

[GitHub] [incubator-mxnet] sxjscience commented on issue #14894: Accelerate ROIPooling layer

2019-07-12 Thread GitBox
sxjscience commented on issue #14894: Accelerate ROIPooling layer URL: https://github.com/apache/incubator-mxnet/pull/14894#issuecomment-511059573 I think it should be good. One concern is it may be backward incompatible due to the `atomicAdd`. Although I feel it's reasonable to use the

[GitHub] [incubator-mxnet] sxjscience commented on a change in pull request #14894: Accelerate ROIPooling layer

2019-07-12 Thread GitBox
sxjscience commented on a change in pull request #14894: Accelerate ROIPooling layer URL: https://github.com/apache/incubator-mxnet/pull/14894#discussion_r303173596 ## File path: src/operator/roi_pooling.cc ## @@ -137,91 +140,18 @@ template inline void

[GitHub] [incubator-mxnet] sxjscience commented on a change in pull request #14894: Accelerate ROIPooling layer

2019-07-12 Thread GitBox
sxjscience commented on a change in pull request #14894: Accelerate ROIPooling layer URL: https://github.com/apache/incubator-mxnet/pull/14894#discussion_r303173424 ## File path: src/operator/roi_pooling.cc ## @@ -137,91 +140,18 @@ template inline void

[GitHub] [incubator-mxnet] sxjscience commented on a change in pull request #14894: Accelerate ROIPooling layer

2019-07-12 Thread GitBox
sxjscience commented on a change in pull request #14894: Accelerate ROIPooling layer URL: https://github.com/apache/incubator-mxnet/pull/14894#discussion_r303173424 ## File path: src/operator/roi_pooling.cc ## @@ -137,91 +140,18 @@ template inline void

[GitHub] [incubator-mxnet] anirudh2290 opened a new pull request #15526: Fix AMP Tutorial warnings and failures

2019-07-12 Thread GitBox
anirudh2290 opened a new pull request #15526: Fix AMP Tutorial warnings and failures URL: https://github.com/apache/incubator-mxnet/pull/15526 ## Description ## - Ignored warnings - removed usage of __file__. ## Checklist ## ### Essentials ### Please feel free to remove

[GitHub] [incubator-mxnet] aaronmarkham commented on issue #15524: refine Nano setup directions

2019-07-12 Thread GitBox
aaronmarkham commented on issue #15524: refine Nano setup directions URL: https://github.com/apache/incubator-mxnet/pull/15524#issuecomment-511056119 Nudging CI again... This is an automated message from the Apache Git

[GitHub] [incubator-mxnet] aaronmarkham commented on issue #15523: add julia env settings

2019-07-12 Thread GitBox
aaronmarkham commented on issue #15523: add julia env settings URL: https://github.com/apache/incubator-mxnet/pull/15523#issuecomment-511055829 Nudging CI since so many tests failed for no apparent reason. This is an

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #15522: [Opperf] Add optimizer update operator benchmarks to opperf

2019-07-12 Thread GitBox
ChaiBapchya commented on issue #15522: [Opperf] Add optimizer update operator benchmarks to opperf URL: https://github.com/apache/incubator-mxnet/pull/15522#issuecomment-511055731 @mxnet-label-bot add [Benchmark, Operator]

[GitHub] [incubator-mxnet] larroy edited a comment on issue #15516: Fix memory leak reported by ASAN in NNVM to ONNX conversion

2019-07-12 Thread GitBox
larroy edited a comment on issue #15516: Fix memory leak reported by ASAN in NNVM to ONNX conversion URL: https://github.com/apache/incubator-mxnet/pull/15516#issuecomment-511052670 Not really, in those cases you want a semantic name, even if you are wrapping your Cat in a pointer you

[GitHub] [incubator-mxnet] larroy commented on issue #15516: Fix memory leak reported by ASAN in NNVM to ONNX conversion

2019-07-12 Thread GitBox
larroy commented on issue #15516: Fix memory leak reported by ASAN in NNVM to ONNX conversion URL: https://github.com/apache/incubator-mxnet/pull/15516#issuecomment-511052670 Not really, in those cases you want a semantic name, even if you are wrapping your Cat in a pointer you want to

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303166837 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303166837 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303166316 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] KellenSunderland commented on a change in pull request #15399: Add unit tests for TensorRT integration and fix some bugs

2019-07-12 Thread GitBox
KellenSunderland commented on a change in pull request #15399: Add unit tests for TensorRT integration and fix some bugs URL: https://github.com/apache/incubator-mxnet/pull/15399#discussion_r302338157 ## File path: src/operator/subgraph/tensorrt/tensorrt-inl.h ## @@

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303153431 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303150705 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303139896 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303139943 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[incubator-mxnet] branch master updated (9ff6c46 -> 2565fa2)

2019-07-12 Thread anirudh2290
This is an automated email from the ASF dual-hosted git repository. anirudh2290 pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 9ff6c46 Add -R option to ci/build.py to avoid rebuilding containers (#15426) add 2565fa2 fix

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303133599 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303132894 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303129096 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303130822 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303130854 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303130688 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303130767 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303130652 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303130733 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303129096 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303129096 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] Zha0q1 commented on issue #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on issue #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#issuecomment-511013592 > Nice work, just a few comments, looks good. Thank you for the detailed advices!

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303118403 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303113524 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303113874 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303118876 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303118036 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303113816 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303115908 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303119170 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303117010 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303119928 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
larroy commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303119597 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] larroy commented on issue #15465: [RFC] Integrate TVM into Apache MXNet

2019-07-12 Thread GitBox
larroy commented on issue #15465: [RFC] Integrate TVM into Apache MXNet URL: https://github.com/apache/incubator-mxnet/issues/15465#issuecomment-510999368 Could we add more information on how will it impact the build systems and dependencies? While integrating TVM is exciting, I think is

[GitHub] [incubator-mxnet] marcoabreu commented on issue #15525: Nightly tests / integration tests in PRs are not tested before merging

2019-07-12 Thread GitBox
marcoabreu commented on issue #15525: Nightly tests / integration tests in PRs are not tested before merging URL: https://github.com/apache/incubator-mxnet/issues/15525#issuecomment-510999095 I'm certainly in favour of a context sensitive bot that gives certain comments if certain types

[GitHub] [incubator-mxnet] larroy commented on issue #15270: Fix warnings in CLang.

2019-07-12 Thread GitBox
larroy commented on issue #15270: Fix warnings in CLang. URL: https://github.com/apache/incubator-mxnet/pull/15270#issuecomment-510997782 Done, could we get those small PRs that fix warnings merged? This is an automated

[GitHub] [incubator-mxnet] larroy edited a comment on issue #15426: Add -R option to ci/build.py to avoid rebuilding containers

2019-07-12 Thread GitBox
larroy edited a comment on issue #15426: Add -R option to ci/build.py to avoid rebuilding containers URL: https://github.com/apache/incubator-mxnet/pull/15426#issuecomment-510997277 Danke für die merge. This is an automated

[GitHub] [incubator-mxnet] larroy commented on issue #15426: Add -R option to ci/build.py to avoid rebuilding containers

2019-07-12 Thread GitBox
larroy commented on issue #15426: Add -R option to ci/build.py to avoid rebuilding containers URL: https://github.com/apache/incubator-mxnet/pull/15426#issuecomment-510997277 Danke fuer die merge. This is an automated

[GitHub] [incubator-mxnet] anirudh2290 commented on a change in pull request #15245: FP16 Support for C Predict API

2019-07-12 Thread GitBox
anirudh2290 commented on a change in pull request #15245: FP16 Support for C Predict API URL: https://github.com/apache/incubator-mxnet/pull/15245#discussion_r303110259 ## File path: amalgamation/python/mxnet_predict.py ## @@ -160,10 +249,18 @@ def forward(self,

[GitHub] [incubator-mxnet] roywei edited a comment on issue #15525: Nightly tests / integration tests in PRs are not tested before merging

2019-07-12 Thread GitBox
roywei edited a comment on issue #15525: Nightly tests / integration tests in PRs are not tested before merging URL: https://github.com/apache/incubator-mxnet/issues/15525#issuecomment-510993342 - Nightly tests are supposed to fail and we can revert whichever PR caused it, other wise

[GitHub] [incubator-mxnet] roywei edited a comment on issue #15525: Nightly tests / integration tests in PRs are not tested before merging

2019-07-12 Thread GitBox
roywei edited a comment on issue #15525: Nightly tests / integration tests in PRs are not tested before merging URL: https://github.com/apache/incubator-mxnet/issues/15525#issuecomment-510993342 - Nightly tests are supposed to fail and we can revert whichever PR caused it, other wise

[GitHub] [incubator-mxnet] roywei edited a comment on issue #15525: Nightly tests / integration tests in PRs are not tested before merging

2019-07-12 Thread GitBox
roywei edited a comment on issue #15525: Nightly tests / integration tests in PRs are not tested before merging URL: https://github.com/apache/incubator-mxnet/issues/15525#issuecomment-510993342 - Nightly tests are supposed to fail and we can revert whichever PR caused it, other wise

[GitHub] [incubator-mxnet] roywei edited a comment on issue #15525: Nightly tests / integration tests in PRs are not tested before merging

2019-07-12 Thread GitBox
roywei edited a comment on issue #15525: Nightly tests / integration tests in PRs are not tested before merging URL: https://github.com/apache/incubator-mxnet/issues/15525#issuecomment-510993342 - Nightly tests are supposed to fail and we can revert whichever PR caused it, other wise

[GitHub] [incubator-mxnet] roywei edited a comment on issue #15525: Nightly tests / integration tests in PRs are not tested before merging

2019-07-12 Thread GitBox
roywei edited a comment on issue #15525: Nightly tests / integration tests in PRs are not tested before merging URL: https://github.com/apache/incubator-mxnet/issues/15525#issuecomment-510993342 - Nightly tests are supposed to fail and we can revert whichever PR caused it, other wise

[GitHub] [incubator-mxnet] roywei edited a comment on issue #15525: Nightly tests / integration tests in PRs are not tested before merging

2019-07-12 Thread GitBox
roywei edited a comment on issue #15525: Nightly tests / integration tests in PRs are not tested before merging URL: https://github.com/apache/incubator-mxnet/issues/15525#issuecomment-510993342 - Nightly tests are supposed to fail and we can revert whichever PR caused it, other wise

[GitHub] [incubator-mxnet] roywei commented on issue #15525: Nightly tests / integration tests in PRs are not tested before merging

2019-07-12 Thread GitBox
roywei commented on issue #15525: Nightly tests / integration tests in PRs are not tested before merging URL: https://github.com/apache/incubator-mxnet/issues/15525#issuecomment-510993342 - Nightly tests are supposed to fail and we can revert whichever PR caused it, other wise it's too

[GitHub] [incubator-mxnet] marcoabreu commented on issue #15525: Nightly tests / integration tests in PRs are not tested before merging

2019-07-12 Thread GitBox
marcoabreu commented on issue #15525: Nightly tests / integration tests in PRs are not tested before merging URL: https://github.com/apache/incubator-mxnet/issues/15525#issuecomment-510985789 I'd love to have context sensitive validations. Unfortunately, this feature doesn't exist out of

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-12 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r303081013 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,733 @@ +/* + *

[GitHub] [incubator-mxnet] sandeep-krishnamurthy opened a new issue #15525: Nightly tests / integration tests in PRs are not tested before merging

2019-07-12 Thread GitBox
sandeep-krishnamurthy opened a new issue #15525: Nightly tests / integration tests in PRs are not tested before merging URL: https://github.com/apache/incubator-mxnet/issues/15525 We do not test nightly/integration tests added in a PR at PR CI stage. This leads to a PR with failing

[GitHub] [incubator-mxnet] mxnet-label-bot commented on issue #15525: Nightly tests / integration tests in PRs are not tested before merging

2019-07-12 Thread GitBox
mxnet-label-bot commented on issue #15525: Nightly tests / integration tests in PRs are not tested before merging URL: https://github.com/apache/incubator-mxnet/issues/15525#issuecomment-510983068 Hey, this is the MXNet Label Bot. Thank you for submitting the issue! I will try and

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15349: Numpy Tensordot Operator

2019-07-12 Thread GitBox
haojin2 commented on a change in pull request #15349: Numpy Tensordot Operator URL: https://github.com/apache/incubator-mxnet/pull/15349#discussion_r303094273 ## File path: src/operator/numpy/np_tensordot_op.cc ## @@ -0,0 +1,222 @@ +/* + * Licensed to the Apache Software

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15349: Numpy Tensordot Operator

2019-07-12 Thread GitBox
haojin2 commented on a change in pull request #15349: Numpy Tensordot Operator URL: https://github.com/apache/incubator-mxnet/pull/15349#discussion_r303094358 ## File path: tests/python/unittest/test_numpy_op.py ## @@ -26,6 +26,151 @@ from mxnet.test_utils import

[GitHub] [incubator-mxnet] aaronmarkham opened a new pull request #15524: refine Nano setup directions

2019-07-12 Thread GitBox
aaronmarkham opened a new pull request #15524: refine Nano setup directions URL: https://github.com/apache/incubator-mxnet/pull/15524 ## Description ## After another run-through on a fresh Nano, plus trying out these new pip wheels, we found some refinements that should make the setup

[incubator-mxnet] branch master updated (6a564be -> 9ff6c46)

2019-07-12 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 6a564be cuda/cuDNN lib version checking. Force cuDNN v7 usage. (#15449) add 9ff6c46 Add -R

[GitHub] [incubator-mxnet] marcoabreu merged pull request #15426: Add -R option to ci/build.py to avoid rebuilding containers

2019-07-12 Thread GitBox
marcoabreu merged pull request #15426: Add -R option to ci/build.py to avoid rebuilding containers URL: https://github.com/apache/incubator-mxnet/pull/15426 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-mxnet] marcoabreu commented on issue #15426: Add -R option to ci/build.py to avoid rebuilding containers

2019-07-12 Thread GitBox
marcoabreu commented on issue #15426: Add -R option to ci/build.py to avoid rebuilding containers URL: https://github.com/apache/incubator-mxnet/pull/15426#issuecomment-510979685 Ye it's probably user id - for whatever reason docker cares about that...

[GitHub] [incubator-mxnet] aaronmarkham opened a new pull request #15523: add julia env settings

2019-07-12 Thread GitBox
aaronmarkham opened a new pull request #15523: add julia env settings URL: https://github.com/apache/incubator-mxnet/pull/15523 ## Description ## I missed this env var requirement in the Julia docs PR, #15454. ## Comment We should probably add some CI test for the website

[GitHub] [incubator-mxnet] ChaiBapchya opened a new pull request #15522: Add optimizer update operator benchmarks to opperf

2019-07-12 Thread GitBox
ChaiBapchya opened a new pull request #15522: Add optimizer update operator benchmarks to opperf URL: https://github.com/apache/incubator-mxnet/pull/15522 ## Description ## Added neural network optimizer (Adam, sgd, etc) update operators to the existing set of opperf benchmarks

  1   2   >