[GitHub] ThomasDelteil commented on issue #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
ThomasDelteil commented on issue #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#issuecomment-383170262 @piiswrong if you want to review This is an automated

[GitHub] ThomasDelteil opened a new pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
ThomasDelteil opened a new pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628 ## Description ## [MXNET-342](https://issues.apache.org/jira/browse/MXNET-342) Fix

[GitHub] piiswrong commented on a change in pull request #10354: Expose the number of GPUs.

2018-04-20 Thread GitBox
piiswrong commented on a change in pull request #10354: Expose the number of GPUs. URL: https://github.com/apache/incubator-mxnet/pull/10354#discussion_r183124984 ## File path: include/mxnet/base.h ## @@ -307,6 +312,21 @@ inline Context Context::GPU(int32_t dev_id) {

[GitHub] piiswrong commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
piiswrong commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#discussion_r183123306 ## File path: python/mxnet/gluon/data/dataloader.py ## @@ -112,6 +113,9 @@ def

[GitHub] ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#discussion_r183123655 ## File path: python/mxnet/gluon/data/dataloader.py ## @@ -112,6 +113,9 @@ def

[GitHub] piiswrong commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
piiswrong commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#discussion_r183123741 ## File path: python/mxnet/gluon/data/dataset.py ## @@ -173,8 +173,15 @@ class

[GitHub] WellyZhang commented on issue #3432: OSError: /usr/lib/libgdal.so.1: undefined symbol: sqlite3_column_table_name

2018-04-20 Thread GitBox
WellyZhang commented on issue #3432: OSError: /usr/lib/libgdal.so.1: undefined symbol: sqlite3_column_table_name URL: https://github.com/apache/incubator-mxnet/issues/3432#issuecomment-383146955 @khurramHazen If you build OpenCV and MXNet using system lib but run with Anaconda IPython,

[GitHub] yajiedesign commented on issue #10626: Could you help build windows pypi package mxnet-cu90 for version 1.0.0 and version 1.1.0?

2018-04-20 Thread GitBox
yajiedesign commented on issue #10626: Could you help build windows pypi package mxnet-cu90 for version 1.0.0 and version 1.1.0? URL: https://github.com/apache/incubator-mxnet/issues/10626#issuecomment-383164382 ok.i know

[GitHub] piiswrong commented on a change in pull request #10354: Expose the number of GPUs.

2018-04-20 Thread GitBox
piiswrong commented on a change in pull request #10354: Expose the number of GPUs. URL: https://github.com/apache/incubator-mxnet/pull/10354#discussion_r183125415 ## File path: include/mxnet/base.h ## @@ -307,6 +312,21 @@ inline Context Context::GPU(int32_t dev_id) {

[GitHub] ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#discussion_r183124801 ## File path: python/mxnet/gluon/data/dataset.py ## @@ -173,8 +173,15 @@ class

[GitHub] ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#discussion_r183124801 ## File path: python/mxnet/gluon/data/dataset.py ## @@ -173,8 +173,15 @@ class

[GitHub] ArmageddonKnight commented on issue #10356: MXNet from Source is slower than Pip Package

2018-04-20 Thread GitBox
ArmageddonKnight commented on issue #10356: MXNet from Source is slower than Pip Package URL: https://github.com/apache/incubator-mxnet/issues/10356#issuecomment-383195630 @rahul003 Thanks a lot for your help and @spidyDev sorry for forgetting to close this issue.

[GitHub] szha commented on issue #10626: Could you help build windows pypi package mxnet-cu90 for version 1.0.0 and version 1.1.0?

2018-04-20 Thread GitBox
szha commented on issue #10626: Could you help build windows pypi package mxnet-cu90 for version 1.0.0 and version 1.1.0? URL: https://github.com/apache/incubator-mxnet/issues/10626#issuecomment-383161120 @yajiedesign This

[GitHub] ArmageddonKnight closed issue #10356: MXNet from Source is slower than Pip Package

2018-04-20 Thread GitBox
ArmageddonKnight closed issue #10356: MXNet from Source is slower than Pip Package URL: https://github.com/apache/incubator-mxnet/issues/10356 This is an automated message from the Apache Git Service. To respond to the

[GitHub] ThomasDelteil commented on issue #9974: DataLoader with workers not compatible with ImageRecordDataset

2018-04-20 Thread GitBox
ThomasDelteil commented on issue #9974: DataLoader with workers not compatible with ImageRecordDataset URL: https://github.com/apache/incubator-mxnet/issues/9974#issuecomment-383170083 @jwfromm opened a PR with the fix https://github.com/apache/incubator-mxnet/pull/10628

[GitHub] ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#discussion_r183124801 ## File path: python/mxnet/gluon/data/dataset.py ## @@ -173,8 +173,15 @@ class

[GitHub] yajiedesign opened a new pull request #10629: [MXNET-343]fix Mkldnn with msvc

2018-04-20 Thread GitBox
yajiedesign opened a new pull request #10629: [MXNET-343]fix Mkldnn with msvc URL: https://github.com/apache/incubator-mxnet/pull/10629 ## Description ## fix Mkldnn with msvc ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your PR. -

[GitHub] piiswrong commented on a change in pull request #10623: [MXNET-339] Add warning on input data for RNN for layout NTC

2018-04-20 Thread GitBox
piiswrong commented on a change in pull request #10623: [MXNET-339] Add warning on input data for RNN for layout NTC URL: https://github.com/apache/incubator-mxnet/pull/10623#discussion_r183115582 ## File path: python/mxnet/gluon/rnn/rnn_layer.py ## @@ -286,6 +286,9 @@

[GitHub] piiswrong commented on a change in pull request #10621: Updated tutorials page.

2018-04-20 Thread GitBox
piiswrong commented on a change in pull request #10621: Updated tutorials page. URL: https://github.com/apache/incubator-mxnet/pull/10621#discussion_r183117310 ## File path: docs/tutorials/index.md ## @@ -1,269 +1,153 @@ # Tutorials -MXNet has two primary high-level

[GitHub] ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#discussion_r183123655 ## File path: python/mxnet/gluon/data/dataloader.py ## @@ -112,6 +113,9 @@ def

[GitHub] ThomasDelteil commented on a change in pull request #10623: [MXNET-339] Add warning on input data for RNN for layout NTC

2018-04-20 Thread GitBox
ThomasDelteil commented on a change in pull request #10623: [MXNET-339] Add warning on input data for RNN for layout NTC URL: https://github.com/apache/incubator-mxnet/pull/10623#discussion_r183123088 ## File path: python/mxnet/gluon/rnn/rnn_layer.py ## @@ -286,6 +286,9

[GitHub] piiswrong commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build

2018-04-20 Thread GitBox
piiswrong commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build URL: https://github.com/apache/incubator-mxnet/pull/10485#discussion_r183124327 ## File path: docs/build_version_doc/artifacts/0.12.0.index.html ## @@ -0,0 +1,1241 @@ + Review

[GitHub] aaronmarkham commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build

2018-04-20 Thread GitBox
aaronmarkham commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build URL: https://github.com/apache/incubator-mxnet/pull/10485#discussion_r183130265 ## File path: docs/build_version_doc/artifacts/0.12.0.index.html ## @@ -0,0 +1,1241 @@ +

[GitHub] aaronmarkham commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build

2018-04-20 Thread GitBox
aaronmarkham commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build URL: https://github.com/apache/incubator-mxnet/pull/10485#discussion_r183130051 ## File path: docs/build_version_doc/artifacts/0.12.0.index.html ## @@ -0,0 +1,1241 @@ +

[GitHub] thomelane commented on issue #10621: Updated tutorials page.

2018-04-20 Thread GitBox
thomelane commented on issue #10621: Updated tutorials page. URL: https://github.com/apache/incubator-mxnet/pull/10621#issuecomment-383164936 JIRA: https://issues.apache.org/jira/browse/MXNET-340 This is an automated message

[GitHub] ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#discussion_r183124801 ## File path: python/mxnet/gluon/data/dataset.py ## @@ -173,8 +173,15 @@ class

[GitHub] ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#discussion_r183131243 ## File path: python/mxnet/gluon/data/dataloader.py ## @@ -112,6 +113,9 @@ def

[GitHub] piiswrong commented on a change in pull request #10630: make the slice of sequential a sequential

2018-04-20 Thread GitBox
piiswrong commented on a change in pull request #10630: make the slice of sequential a sequential URL: https://github.com/apache/incubator-mxnet/pull/10630#discussion_r183164673 ## File path: python/mxnet/gluon/nn/basic_layers.py ## @@ -62,7 +62,15 @@ def __repr__(self):

[GitHub] ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#discussion_r183161704 ## File path: python/mxnet/gluon/data/dataset.py ## @@ -173,8 +173,15 @@ class

[GitHub] ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#discussion_r183161704 ## File path: python/mxnet/gluon/data/dataset.py ## @@ -173,8 +173,15 @@ class

[GitHub] ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#discussion_r183161704 ## File path: python/mxnet/gluon/data/dataset.py ## @@ -173,8 +173,15 @@ class

[GitHub] szha commented on a change in pull request #10630: make the slice of sequential a sequential

2018-04-20 Thread GitBox
szha commented on a change in pull request #10630: make the slice of sequential a sequential URL: https://github.com/apache/incubator-mxnet/pull/10630#discussion_r183175657 ## File path: python/mxnet/gluon/nn/basic_layers.py ## @@ -62,7 +62,15 @@ def __repr__(self):

[GitHub] ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#discussion_r183156381 ## File path: python/mxnet/gluon/data/dataset.py ## @@ -173,8 +173,15 @@ class

[GitHub] ThomasDelteil commented on issue #10280: Flaky test_operator.test_correlation

2018-04-20 Thread GitBox
ThomasDelteil commented on issue #10280: Flaky test_operator.test_correlation URL: https://github.com/apache/incubator-mxnet/issues/10280#issuecomment-383207727 Happened to me too

[GitHub] ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#discussion_r183161704 ## File path: python/mxnet/gluon/data/dataset.py ## @@ -173,8 +173,15 @@ class

[GitHub] ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#discussion_r183161704 ## File path: python/mxnet/gluon/data/dataset.py ## @@ -173,8 +173,15 @@ class

[GitHub] zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-20 Thread GitBox
zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183132338 ## File path: python/mxnet/contrib/quantization.py ## @@ -72,7 +72,7 @@ def

[GitHub] zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-20 Thread GitBox
zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183136960 ## File path: src/operator/quantization/mkldnn/mkldnn_quantized_conv.cc ## @@ -0,0

[GitHub] zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-20 Thread GitBox
zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183166403 ## File path: src/operator/quantization/mkldnn/mkldnn_requantize-inl.h ## @@ -0,0

[GitHub] zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-20 Thread GitBox
zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183166634 ## File path: src/operator/quantization/mkldnn/mkldnn_requantize-inl.h ## @@ -0,0

[GitHub] zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-20 Thread GitBox
zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183138812 ## File path: src/operator/quantization/mkldnn/mkldnn_quantized_pooling.cc ## @@ -0,0

[GitHub] zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-20 Thread GitBox
zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183134939 ## File path: src/operator/quantization/mkldnn/mkldnn_quantize-inl.h ## @@ -0,0 +1,107

[GitHub] zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-20 Thread GitBox
zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183132722 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -211,6 +211,7 @@

[GitHub] zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-20 Thread GitBox
zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183134801 ## File path: src/operator/quantization/mkldnn/mkldnn_quantize-inl.h ## @@ -0,0 +1,107

[GitHub] reminisce commented on a change in pull request #10482: [MXNET-303] Update Jetpack to version 3.2

2018-04-20 Thread GitBox
reminisce commented on a change in pull request #10482: [MXNET-303] Update Jetpack to version 3.2 URL: https://github.com/apache/incubator-mxnet/pull/10482#discussion_r183176339 ## File path: Makefile ## @@ -254,8 +254,10 @@ ifneq ($(ADD_LDFLAGS), NONE) LDFLAGS +=

[GitHub] szha opened a new pull request #10630: make the slice of sequential a sequential

2018-04-20 Thread GitBox
szha opened a new pull request #10630: make the slice of sequential a sequential URL: https://github.com/apache/incubator-mxnet/pull/10630 ## Description ## make the slice of sequential a sequential ## Checklist ## ### Essentials ### Please feel free to remove inapplicable

[GitHub] zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-20 Thread GitBox
zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183168038 ## File path: src/operator/quantization/quantize_graph_pass.cc ## @@ -250,6 +250,165 @@

[GitHub] zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-20 Thread GitBox
zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183167546 ## File path: src/operator/quantization/quantize_graph_pass.cc ## @@ -198,7 +198,7 @@

[GitHub] zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-20 Thread GitBox
zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183139227 ## File path: src/operator/quantization/mkldnn/mkldnn_quantized_pooling.cc ## @@ -0,0

[GitHub] zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-20 Thread GitBox
zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183165895 ## File path: src/operator/quantization/mkldnn/mkldnn_quantized_pooling.cc ## @@ -0,0

[GitHub] szha commented on a change in pull request #10630: make the slice of sequential a sequential

2018-04-20 Thread GitBox
szha commented on a change in pull request #10630: make the slice of sequential a sequential URL: https://github.com/apache/incubator-mxnet/pull/10630#discussion_r183175559 ## File path: python/mxnet/gluon/nn/basic_layers.py ## @@ -62,7 +62,15 @@ def __repr__(self):

[GitHub] KellenSunderland commented on a change in pull request #10482: [MXNET-303] Update Jetpack to version 3.2

2018-04-20 Thread GitBox
KellenSunderland commented on a change in pull request #10482: [MXNET-303] Update Jetpack to version 3.2 URL: https://github.com/apache/incubator-mxnet/pull/10482#discussion_r183178442 ## File path: Makefile ## @@ -254,8 +254,10 @@ ifneq ($(ADD_LDFLAGS), NONE)

[GitHub] ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#discussion_r183156381 ## File path: python/mxnet/gluon/data/dataset.py ## @@ -173,8 +173,15 @@ class

[GitHub] ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#discussion_r183161704 ## File path: python/mxnet/gluon/data/dataset.py ## @@ -173,8 +173,15 @@ class

[GitHub] ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#discussion_r183161704 ## File path: python/mxnet/gluon/data/dataset.py ## @@ -173,8 +173,15 @@ class

[GitHub] piiswrong commented on a change in pull request #10630: make the slice of sequential a sequential

2018-04-20 Thread GitBox
piiswrong commented on a change in pull request #10630: make the slice of sequential a sequential URL: https://github.com/apache/incubator-mxnet/pull/10630#discussion_r183164014 ## File path: python/mxnet/gluon/nn/basic_layers.py ## @@ -62,7 +62,15 @@ def __repr__(self):

[GitHub] piiswrong opened a new pull request #10632: move parameters to backend for hybridblock

2018-04-20 Thread GitBox
piiswrong opened a new pull request #10632: move parameters to backend for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/10632 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove

[GitHub] piiswrong commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
piiswrong commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#discussion_r183152972 ## File path: python/mxnet/gluon/data/dataset.py ## @@ -173,8 +173,15 @@ class

[GitHub] ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader

2018-04-20 Thread GitBox
ThomasDelteil commented on a change in pull request #10628: [MXNET-342] Fix the multi worker Dataloader URL: https://github.com/apache/incubator-mxnet/pull/10628#discussion_r183161704 ## File path: python/mxnet/gluon/data/dataset.py ## @@ -173,8 +173,15 @@ class

[GitHub] piiswrong commented on issue #10627: 8 gpu no speed up, delay caused by main thread

2018-04-20 Thread GitBox
piiswrong commented on issue #10627: 8 gpu no speed up, delay caused by main thread URL: https://github.com/apache/incubator-mxnet/issues/10627#issuecomment-383219398 We are working on optimizing gluon performence. One of the things we are trying is using multiple threads for forward.

[GitHub] zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-20 Thread GitBox
zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183134591 ## File path: src/operator/quantization/mkldnn/mkldnn_dequantize-inl.h ## @@ -0,0

[GitHub] zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-20 Thread GitBox
zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183135955 ## File path: src/operator/quantization/mkldnn/mkldnn_quantized_conv.cc ## @@ -0,0

[GitHub] zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-20 Thread GitBox
zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183165762 ## File path: src/operator/quantization/mkldnn/mkldnn_quantized_pooling.cc ## @@ -0,0

[GitHub] zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-20 Thread GitBox
zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183132584 ## File path: src/c_api/c_api_symbolic.cc ## @@ -595,7 +596,12 @@ int

[GitHub] zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-20 Thread GitBox
zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183135075 ## File path: src/operator/quantization/mkldnn/mkldnn_dequantize-inl.h ## @@ -0,0

[GitHub] zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-20 Thread GitBox
zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183166565 ## File path: src/operator/quantization/mkldnn/mkldnn_requantize-inl.h ## @@ -0,0

[GitHub] zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-20 Thread GitBox
zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183136102 ## File path: src/operator/quantization/mkldnn/mkldnn_quantized_conv.cc ## @@ -0,0

[GitHub] piiswrong commented on issue #10627: 8 gpu no speed up, delay caused by main thread

2018-04-20 Thread GitBox
piiswrong commented on issue #10627: 8 gpu no speed up, delay caused by main thread URL: https://github.com/apache/incubator-mxnet/issues/10627#issuecomment-383221250 btw what gpus are you using? This is an automated

[GitHub] zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-20 Thread GitBox
zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183167005 ## File path: src/operator/quantization/quantize_graph_pass.cc ## @@ -129,7 +129,7 @@

[GitHub] SerailHydra opened a new issue #10631: Better auto-tuning strategy for multi-machine training performance?

2018-04-20 Thread GitBox
SerailHydra opened a new issue #10631: Better auto-tuning strategy for multi-machine training performance? URL: https://github.com/apache/incubator-mxnet/issues/10631 ## Description The performance of multi-machine training is greatly affected by this MXNET_CPU_WORKER_NTHREADS

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

2018-04-20 Thread GitBox
KellenSunderland commented on issue #10553: Build error for OSX with CUDA support URL: https://github.com/apache/incubator-mxnet/issues/10553#issuecomment-383201036 @spidyDev I don't think I have permission to label. Maybe @marcoabreu can help us out.

[GitHub] haojin2 opened a new pull request #10633: [MXNET-346] Hard Sigmoid Operator

2018-04-20 Thread GitBox
haojin2 opened a new pull request #10633: [MXNET-346] Hard Sigmoid Operator URL: https://github.com/apache/incubator-mxnet/pull/10633 ## Description ## Implementation of Hard Sigmoid function ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items

[GitHub] ThomasDelteil commented on issue #9845: Flaky test_operator.test_reduce

2018-04-20 Thread GitBox
ThomasDelteil commented on issue #9845: Flaky test_operator.test_reduce URL: https://github.com/apache/incubator-mxnet/issues/9845#issuecomment-383217825 Happened to me too: http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/PR-10628/4/pipeline/656

[incubator-mxnet] branch master updated: [MXNET-322] Elemwise_mul(row_sparse, dense) = row_sparse on CPU (#10557)

2018-04-20 Thread haibin
This is an automated email from the ASF dual-hosted git repository. haibin 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 6df1979 [MXNET-322]

[GitHub] eric-haibin-lin closed pull request #10557: [MXNET-322] Elemwise_mul(row_sparse, dense) = row_sparse on CPU

2018-04-20 Thread GitBox
eric-haibin-lin closed pull request #10557: [MXNET-322] Elemwise_mul(row_sparse, dense) = row_sparse on CPU URL: https://github.com/apache/incubator-mxnet/pull/10557 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] haojin2 commented on issue #10633: [MXNET-346] Hard Sigmoid Operator

2018-04-20 Thread GitBox
haojin2 commented on issue #10633: [MXNET-346] Hard Sigmoid Operator URL: https://github.com/apache/incubator-mxnet/pull/10633#issuecomment-383254383 @rahul003 @reminisce @anirudh2290 @eric-haibin-lin Could you please give a review for this?

[GitHub] eric-haibin-lin closed pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors

2018-04-20 Thread GitBox
eric-haibin-lin closed pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors URL: https://github.com/apache/incubator-mxnet/pull/10550 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[incubator-mxnet] branch master updated: [MXNET-320] Support elemwise_add/sub between dense and csr tensors (#10550)

2018-04-20 Thread haibin
This is an automated email from the ASF dual-hosted git repository. haibin 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 73d879c [MXNET-320] Support

[GitHub] anirudhacharya commented on a change in pull request #10633: [MXNET-346] Hard Sigmoid Operator

2018-04-20 Thread GitBox
anirudhacharya commented on a change in pull request #10633: [MXNET-346] Hard Sigmoid Operator URL: https://github.com/apache/incubator-mxnet/pull/10633#discussion_r183190237 ## File path: src/operator/tensor/elemwise_unary_op_basic.cc ## @@ -106,6 +106,35 @@ The storage

[GitHub] szha commented on issue #10630: make the slice of sequential a sequential

2018-04-20 Thread GitBox
szha commented on issue #10630: make the slice of sequential a sequential URL: https://github.com/apache/incubator-mxnet/pull/10630#issuecomment-383249954 Found a minor bug in using _brief_print_list. in py3, dict.keys() doesn't return a list.

[incubator-mxnet] branch master updated: Change Model url to S3 (#10617)

2018-04-20 Thread nswamy
This is an automated email from the ASF dual-hosted git repository. nswamy pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new ba40121 Change Model url to S3

[GitHub] nswamy commented on issue #10432: [scala-infer]move test files for scala infer API to s3

2018-04-20 Thread GitBox
nswamy commented on issue #10432: [scala-infer]move test files for scala infer API to s3 URL: https://github.com/apache/incubator-mxnet/issues/10432#issuecomment-383251219 #10617 This is an automated message from the

[GitHub] nswamy closed pull request #10617: [MXNET-337] Change Model url to S3

2018-04-20 Thread GitBox
nswamy closed pull request #10617: [MXNET-337] Change Model url to S3 URL: https://github.com/apache/incubator-mxnet/pull/10617 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] haojin2 commented on a change in pull request #10633: [MXNET-346] Hard Sigmoid Operator

2018-04-20 Thread GitBox
haojin2 commented on a change in pull request #10633: [MXNET-346] Hard Sigmoid Operator URL: https://github.com/apache/incubator-mxnet/pull/10633#discussion_r183191838 ## File path: src/operator/tensor/elemwise_unary_op_basic.cc ## @@ -106,6 +106,35 @@ The storage type of

[GitHub] nswamy closed issue #10432: [scala-infer]move test files for scala infer API to s3

2018-04-20 Thread GitBox
nswamy closed issue #10432: [scala-infer]move test files for scala infer API to s3 URL: https://github.com/apache/incubator-mxnet/issues/10432 This is an automated message from the Apache Git Service. To respond to the

[GitHub] yajiedesign commented on issue #10626: Could you help build windows pypi package mxnet-cu90 for version 1.0.0 and version 1.1.0?

2018-04-20 Thread GitBox
yajiedesign commented on issue #10626: Could you help build windows pypi package mxnet-cu90 for version 1.0.0 and version 1.1.0? URL: https://github.com/apache/incubator-mxnet/issues/10626#issuecomment-383263082 1.1.0 is done,1.0.0 come soon on

[GitHub] qxin commented on issue #9275: "bad" mAP score on COCO

2018-04-20 Thread GitBox
qxin commented on issue #9275: "bad" mAP score on COCO URL: https://github.com/apache/incubator-mxnet/issues/9275#issuecomment-382994431 @sandeep-krishnamurthy Thanks ~ (and sorry, for the inactivity) This is an automated

[GitHub] zheng-da commented on issue #10599: [MKLDNN Bug] MKLDNN eats lots of memory and then crash down.

2018-04-20 Thread GitBox
zheng-da commented on issue #10599: [MKLDNN Bug] MKLDNN eats lots of memory and then crash down. URL: https://github.com/apache/incubator-mxnet/issues/10599#issuecomment-382989367 The memory used by MKLDNN is also allocated by the MXNet memory allocator. MKLDNN NDArrays are reused in the

[GitHub] zheng-da commented on issue #9835: Failed to export gluon model zoo vision model to symbol file

2018-04-20 Thread GitBox
zheng-da commented on issue #9835: Failed to export gluon model zoo vision model to symbol file URL: https://github.com/apache/incubator-mxnet/issues/9835#issuecomment-383001871 i just saw this message. @ashokei is working on this.

[GitHub] lionel92 commented on issue #10599: [MKLDNN Bug] MKLDNN eats lots of memory and then crash down.

2018-04-20 Thread GitBox
lionel92 commented on issue #10599: [MKLDNN Bug] MKLDNN eats lots of memory and then crash down. URL: https://github.com/apache/incubator-mxnet/issues/10599#issuecomment-382997495 @zheng-da @pengzhao-intel I reproduced this case, and per the memory allocation profiling (via the

[GitHub] zheng-da commented on issue #9812: Unrepeatable test_gluon_model_zoo_gpu.py:test_training CI failures seen.

2018-04-20 Thread GitBox
zheng-da commented on issue #9812: Unrepeatable test_gluon_model_zoo_gpu.py:test_training CI failures seen. URL: https://github.com/apache/incubator-mxnet/issues/9812#issuecomment-383001501 This bug has been fixed by https://github.com/apache/incubator-mxnet/pull/9862 I think we can

[GitHub] IrvingShu commented on issue #9920: Asynchronous Issue on CustomOP and mxnet.image.ImageDetIter

2018-04-20 Thread GitBox
IrvingShu commented on issue #9920: Asynchronous Issue on CustomOP and mxnet.image.ImageDetIter URL: https://github.com/apache/incubator-mxnet/issues/9920#issuecomment-382997920 @wkcn thanks for sharing your ways. I added ctx = context.cpu(), however, the new problem happens:

[GitHub] zheng-da commented on issue #7652: Need a clear and thorough tutorial for writing custom operators in c++ with nnvm

2018-04-20 Thread GitBox
zheng-da commented on issue #7652: Need a clear and thorough tutorial for writing custom operators in c++ with nnvm URL: https://github.com/apache/incubator-mxnet/issues/7652#issuecomment-383000802 just saw the comment. I'll update the tutorial

[GitHub] chinakook commented on issue #10599: [MKLDNN Bug] MKLDNN eats lots of memory and then crash down.

2018-04-20 Thread GitBox
chinakook commented on issue #10599: [MKLDNN Bug] MKLDNN eats lots of memory and then crash down. URL: https://github.com/apache/incubator-mxnet/issues/10599#issuecomment-383005826 @lionel92 The model is a widely used Caffe-Resnet101 with some 1x1 convolutions added and with big input

[GitHub] TaoLv commented on issue #10580: inference results unstable in mxnet_mkl-1.2.0b20180416

2018-04-20 Thread GitBox
TaoLv commented on issue #10580: inference results unstable in mxnet_mkl-1.2.0b20180416 URL: https://github.com/apache/incubator-mxnet/issues/10580#issuecomment-383007445 Please ignore that. @zheng-da has submitted the final fix in #10624 . You can try the nightly build after that PR is

[GitHub] FCInter opened a new issue #10625: ImageRecordIter loading small data reports out of memory

2018-04-20 Thread GitBox
FCInter opened a new issue #10625: ImageRecordIter loading small data reports out of memory URL: https://github.com/apache/incubator-mxnet/issues/10625 ## Description (Brief description of the problem in no more than 2 sentences.) mx.io.ImageRecordIter reports CUDA out of memory when

[GitHub] wkcn closed issue #10288: no negative check for reps in mx.nd.tile and mx.sym.tile

2018-04-20 Thread GitBox
wkcn closed issue #10288: no negative check for reps in mx.nd.tile and mx.sym.tile URL: https://github.com/apache/incubator-mxnet/issues/10288 This is an automated message from the Apache Git Service. To respond to the

[GitHub] dwSun commented on issue #10580: inference results unstable in mxnet_mkl-1.2.0b20180416

2018-04-20 Thread GitBox
dwSun commented on issue #10580: inference results unstable in mxnet_mkl-1.2.0b20180416 URL: https://github.com/apache/incubator-mxnet/issues/10580#issuecomment-383003163 @TaoLv can you provide a pre-compiled pip package? Thanks

[GitHub] wkcn commented on issue #9920: Asynchronous Issue on CustomOP and mxnet.image.ImageDetIter

2018-04-20 Thread GitBox
wkcn commented on issue #9920: Asynchronous Issue on CustomOP and mxnet.image.ImageDetIter URL: https://github.com/apache/incubator-mxnet/issues/9920#issuecomment-383041481 @IrvingShu It seems that the data types of source ndarray and target ndarray are different. Could you upload

[GitHub] tdomhan commented on issue #10354: Expose the number of GPUs.

2018-04-20 Thread GitBox
tdomhan commented on issue #10354: Expose the number of GPUs. URL: https://github.com/apache/incubator-mxnet/pull/10354#issuecomment-383013176 that is correct. I'm not sure why it happens or how it can be fixed. With the recent changes that is what will happen now that it will raise an

  1   2   >