[GitHub] qw42 opened a new issue #12239: Scale to many CPU cores

2018-08-17 Thread GitBox
qw42 opened a new issue #12239: Scale to many CPU cores URL: https://github.com/apache/incubator-mxnet/issues/12239 What is the best way to achieve best throughput (only forward runs) on a multi-cpu/many-core server? ## Description I am using CPP package. My problem is

[GitHub] indhub closed issue #8165: missing arguments in SSD example train.py

2018-08-17 Thread GitBox
indhub closed issue #8165: missing arguments in SSD example train.py URL: https://github.com/apache/incubator-mxnet/issues/8165 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] HorsonLiu commented on issue #10948: get stuck with subprocess in multithread

2018-08-17 Thread GitBox
HorsonLiu commented on issue #10948: get stuck with subprocess in multithread URL: https://github.com/apache/incubator-mxnet/issues/10948#issuecomment-414027276 @Roshrini Thank you for your suggestion. I am using python-2.7.5 mxnet-1.3.0 on centos-7. I have tried you code, it results

[GitHub] vandanavk commented on issue #12024: Python, Bug: Speech_recognition crashes in threaded_engine.h:379: Error: compute_ctc_loss, stat = execution failed

2018-08-17 Thread GitBox
vandanavk commented on issue #12024: Python, Bug: Speech_recognition crashes in threaded_engine.h:379: Error: compute_ctc_loss, stat = execution failed URL: https://github.com/apache/incubator-mxnet/issues/12024#issuecomment-414024311 Update: I don't see the issue with mxboard.

[incubator-mxnet] branch master updated: Add new test case for topk to cover large size of matrix as input (#12217)

2018-08-17 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng 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 ca92e3d Add new test case for topk

[GitHub] szha closed pull request #12217: Add new test case for topk to cover large size of matrix as input

2018-08-17 Thread GitBox
szha closed pull request #12217: Add new test case for topk to cover large size of matrix as input URL: https://github.com/apache/incubator-mxnet/pull/12217 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] pengzhao-intel removed a comment on issue #12217: Add new test case for topk to cover large size of matrix as input

2018-08-17 Thread GitBox
pengzhao-intel removed a comment on issue #12217: Add new test case for topk to cover large size of matrix as input URL: https://github.com/apache/incubator-mxnet/pull/12217#issuecomment-414022474 @ciyongch retrigger the CI

[GitHub] pengzhao-intel commented on issue #12217: Add new test case for topk to cover large size of matrix as input

2018-08-17 Thread GitBox
pengzhao-intel commented on issue #12217: Add new test case for topk to cover large size of matrix as input URL: https://github.com/apache/incubator-mxnet/pull/12217#issuecomment-414022474 @ciyongch retrigger the CI This is

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

2018-08-17 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng 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 5786087 Bump the publish

[GitHub] cclauss opened a new issue #8270: 5 undefined names in Python code

2018-08-17 Thread GitBox
cclauss opened a new issue #8270: 5 undefined names in Python code URL: https://github.com/apache/incubator-mxnet/issues/8270 Undefined names can raise NameError at runtime. flake8 testing of https://github.com/apache/incubator-mxnet on Python 2.7.14 $ __flake8 . --count

[GitHub] nswamy closed pull request #12225: [MXNET-703] Remove race condition in onnx build

2018-08-17 Thread GitBox
nswamy closed pull request #12225: [MXNET-703] Remove race condition in onnx build URL: https://github.com/apache/incubator-mxnet/pull/12225 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

[GitHub] sandeep-krishnamurthy closed issue #11711: test_operator.test_bilinear_sampler has fixed seed that can mask flakiness

2018-08-17 Thread GitBox
sandeep-krishnamurthy closed issue #11711: test_operator.test_bilinear_sampler has fixed seed that can mask flakiness URL: https://github.com/apache/incubator-mxnet/issues/11711 This is an automated message from the Apache

[incubator-mxnet] branch master updated: re-enable randomized test_bilinear_sampler (#12227)

2018-08-17 Thread skm
This is an automated email from the ASF dual-hosted git repository. skm 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 89fbb29 re-enable randomized

[GitHub] sandeep-krishnamurthy closed pull request #12227: Re-enable randomized test_bilinear_sampler

2018-08-17 Thread GitBox
sandeep-krishnamurthy closed pull request #12227: Re-enable randomized test_bilinear_sampler URL: https://github.com/apache/incubator-mxnet/pull/12227 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] sandeep-krishnamurthy closed issue #11084: Undefined Behavior of mx.sym.where with shape-mismatched cond

2018-08-17 Thread GitBox
sandeep-krishnamurthy closed issue #11084: Undefined Behavior of mx.sym.where with shape-mismatched cond URL: https://github.com/apache/incubator-mxnet/issues/11084 This is an automated message from the Apache Git Service.

[GitHub] sandeep-krishnamurthy commented on issue #11084: Undefined Behavior of mx.sym.where with shape-mismatched cond

2018-08-17 Thread GitBox
sandeep-krishnamurthy commented on issue #11084: Undefined Behavior of mx.sym.where with shape-mismatched cond URL: https://github.com/apache/incubator-mxnet/issues/11084#issuecomment-414017924 Resolving as the fix is merged.

[GitHub] sandeep-krishnamurthy closed pull request #12174: [MXNET-806] Report error when shape mismatch in "where" operator

2018-08-17 Thread GitBox
sandeep-krishnamurthy closed pull request #12174: [MXNET-806] Report error when shape mismatch in "where" operator URL: https://github.com/apache/incubator-mxnet/pull/12174 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[incubator-mxnet] branch master updated: [MXNET-806] Report error when shape mismatch in "where" operator (#12174)

2018-08-17 Thread skm
This is an automated email from the ASF dual-hosted git repository. skm 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 54af0cc [MXNET-806] Report error when

[GitHub] sandeep-krishnamurthy closed issue #11883: ImageIter last batch / batch padding behavior

2018-08-17 Thread GitBox
sandeep-krishnamurthy closed issue #11883: ImageIter last batch / batch padding behavior URL: https://github.com/apache/incubator-mxnet/issues/11883 This is an automated message from the Apache Git Service. To respond to

[GitHub] sandeep-krishnamurthy commented on issue #11883: ImageIter last batch / batch padding behavior

2018-08-17 Thread GitBox
sandeep-krishnamurthy commented on issue #11883: ImageIter last batch / batch padding behavior URL: https://github.com/apache/incubator-mxnet/issues/11883#issuecomment-414017503 This is now supported. Resolving. This is an

[incubator-mxnet] branch master updated: [MXNET-737]Add last batch handle for imageiter (#12131)

2018-08-17 Thread skm
This is an automated email from the ASF dual-hosted git repository. skm 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 afb77f8 [MXNET-737]Add last batch handle

[GitHub] sandeep-krishnamurthy closed pull request #12131: [MXNET-737]Add last batch handle for imageiter

2018-08-17 Thread GitBox
sandeep-krishnamurthy closed pull request #12131: [MXNET-737]Add last batch handle for imageiter URL: https://github.com/apache/incubator-mxnet/pull/12131 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] haojin2 commented on a change in pull request #12230: Update Operator Implementation Tutorial

2018-08-17 Thread GitBox
haojin2 commented on a change in pull request #12230: Update Operator Implementation Tutorial URL: https://github.com/apache/incubator-mxnet/pull/12230#discussion_r211056017 ## File path: docs/faq/add_op_in_backend.md ## @@ -543,53 +545,81 @@ If you use python, when you

[GitHub] anirudhacharya commented on a change in pull request #12230: Update Operator Implementation Tutorial

2018-08-17 Thread GitBox
anirudhacharya commented on a change in pull request #12230: Update Operator Implementation Tutorial URL: https://github.com/apache/incubator-mxnet/pull/12230#discussion_r211055958 ## File path: docs/faq/add_op_in_backend.md ## @@ -543,53 +545,81 @@ If you use python,

[GitHub] haojin2 commented on a change in pull request #12230: Update Operator Implementation Tutorial

2018-08-17 Thread GitBox
haojin2 commented on a change in pull request #12230: Update Operator Implementation Tutorial URL: https://github.com/apache/incubator-mxnet/pull/12230#discussion_r211055943 ## File path: docs/faq/add_op_in_backend.md ## @@ -543,53 +545,81 @@ If you use python, when you

[GitHub] haojin2 commented on a change in pull request #12230: Update Operator Implementation Tutorial

2018-08-17 Thread GitBox
haojin2 commented on a change in pull request #12230: Update Operator Implementation Tutorial URL: https://github.com/apache/incubator-mxnet/pull/12230#discussion_r211055611 ## File path: docs/faq/add_op_in_backend.md ## @@ -543,53 +545,81 @@ If you use python, when you

[GitHub] anirudhacharya commented on a change in pull request #12230: Update Operator Implementation Tutorial

2018-08-17 Thread GitBox
anirudhacharya commented on a change in pull request #12230: Update Operator Implementation Tutorial URL: https://github.com/apache/incubator-mxnet/pull/12230#discussion_r211055442 ## File path: docs/faq/add_op_in_backend.md ## @@ -543,53 +545,81 @@ If you use python,

[GitHub] anirudhacharya commented on a change in pull request #12230: Update Operator Implementation Tutorial

2018-08-17 Thread GitBox
anirudhacharya commented on a change in pull request #12230: Update Operator Implementation Tutorial URL: https://github.com/apache/incubator-mxnet/pull/12230#discussion_r211054346 ## File path: docs/faq/add_op_in_backend.md ## @@ -543,53 +545,81 @@ If you use python,

[GitHub] anirudhacharya commented on a change in pull request #12230: Update Operator Implementation Tutorial

2018-08-17 Thread GitBox
anirudhacharya commented on a change in pull request #12230: Update Operator Implementation Tutorial URL: https://github.com/apache/incubator-mxnet/pull/12230#discussion_r211054681 ## File path: docs/faq/add_op_in_backend.md ## @@ -543,53 +545,81 @@ If you use python,

[GitHub] anirudhacharya commented on a change in pull request #12230: Update Operator Implementation Tutorial

2018-08-17 Thread GitBox
anirudhacharya commented on a change in pull request #12230: Update Operator Implementation Tutorial URL: https://github.com/apache/incubator-mxnet/pull/12230#discussion_r211054749 ## File path: docs/faq/add_op_in_backend.md ## @@ -543,53 +545,81 @@ If you use python,

[GitHub] anirudhacharya commented on a change in pull request #12230: Update Operator Implementation Tutorial

2018-08-17 Thread GitBox
anirudhacharya commented on a change in pull request #12230: Update Operator Implementation Tutorial URL: https://github.com/apache/incubator-mxnet/pull/12230#discussion_r211054493 ## File path: docs/faq/add_op_in_backend.md ## @@ -543,53 +545,81 @@ If you use python,

[GitHub] haojin2 commented on a change in pull request #12230: Update Operator Implementation Tutorial

2018-08-17 Thread GitBox
haojin2 commented on a change in pull request #12230: Update Operator Implementation Tutorial URL: https://github.com/apache/incubator-mxnet/pull/12230#discussion_r211054713 ## File path: docs/faq/add_op_in_backend.md ## @@ -543,53 +545,81 @@ If you use python, when you

[GitHub] anirudhacharya commented on a change in pull request #12230: Update Operator Implementation Tutorial

2018-08-17 Thread GitBox
anirudhacharya commented on a change in pull request #12230: Update Operator Implementation Tutorial URL: https://github.com/apache/incubator-mxnet/pull/12230#discussion_r211054208 ## File path: docs/faq/add_op_in_backend.md ## @@ -543,53 +545,81 @@ If you use python,

[GitHub] anirudhacharya commented on a change in pull request #12230: Update Operator Implementation Tutorial

2018-08-17 Thread GitBox
anirudhacharya commented on a change in pull request #12230: Update Operator Implementation Tutorial URL: https://github.com/apache/incubator-mxnet/pull/12230#discussion_r211054181 ## File path: docs/faq/add_op_in_backend.md ## @@ -543,53 +545,81 @@ If you use python,

[GitHub] cclauss closed pull request #12237: [MXNET-696] Delete comment line that contains a Unicode character

2018-08-17 Thread GitBox
cclauss closed pull request #12237: [MXNET-696] Delete comment line that contains a Unicode character URL: https://github.com/apache/incubator-mxnet/pull/12237 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] marcoabreu commented on issue #12237: [MXNET-696] Delete comment line that contains a Unicode character

2018-08-17 Thread GitBox
marcoabreu commented on issue #12237: [MXNET-696] Delete comment line that contains a Unicode character URL: https://github.com/apache/incubator-mxnet/pull/12237#issuecomment-414012008 Thank you and sorry for the revert. Please feel free to re-add the functionality you added. We just

[incubator-mxnet] branch master updated: Revert "[MXNET-696] Define cmp() in Python 3 for line 222 (#12191)" (#12231)

2018-08-17 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu 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 24e42a0 Revert "[MXNET-696]

[GitHub] marcoabreu closed pull request #12231: Revert "[MXNET-696] Define cmp() in Python 3 for line 222 (#12191)"

2018-08-17 Thread GitBox
marcoabreu closed pull request #12231: Revert "[MXNET-696] Define cmp() in Python 3 for line 222 (#12191)" URL: https://github.com/apache/incubator-mxnet/pull/12231 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 a change in pull request #12230: Update Operator Implementation Tutorial

2018-08-17 Thread GitBox
haojin2 commented on a change in pull request #12230: Update Operator Implementation Tutorial URL: https://github.com/apache/incubator-mxnet/pull/12230#discussion_r211052702 ## File path: docs/faq/add_op_in_backend.md ## @@ -543,53 +545,81 @@ If you use python, when you

[GitHub] haojin2 commented on a change in pull request #12230: Update Operator Implementation Tutorial

2018-08-17 Thread GitBox
haojin2 commented on a change in pull request #12230: Update Operator Implementation Tutorial URL: https://github.com/apache/incubator-mxnet/pull/12230#discussion_r211052636 ## File path: docs/faq/add_op_in_backend.md ## @@ -543,53 +545,81 @@ If you use python, when you

[GitHub] haojin2 commented on a change in pull request #12230: Update Operator Implementation Tutorial

2018-08-17 Thread GitBox
haojin2 commented on a change in pull request #12230: Update Operator Implementation Tutorial URL: https://github.com/apache/incubator-mxnet/pull/12230#discussion_r211052310 ## File path: docs/faq/add_op_in_backend.md ## @@ -543,53 +545,81 @@ If you use python, when you

[GitHub] larroy opened a new pull request #12238: [WIP][Don't merge][Don't review] use jom

2018-08-17 Thread GitBox
larroy opened a new pull request #12238: [WIP][Don't merge][Don't review] use jom URL: https://github.com/apache/incubator-mxnet/pull/12238 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove

[GitHub] aaronmarkham opened a new pull request #12236: Remove duplicate scala package runs during the website builds

2018-08-17 Thread GitBox
aaronmarkham opened a new pull request #12236: Remove duplicate scala package runs during the website builds URL: https://github.com/apache/incubator-mxnet/pull/12236 ## Description ## The pipeline is currently running `make scalapkg` twice when both Scala and Clojure API docs are

[GitHub] cclauss opened a new pull request #12237: [MXNET-696] Delete comment line that contains a Unicode character

2018-08-17 Thread GitBox
cclauss opened a new pull request #12237: [MXNET-696] Delete comment line that contains a Unicode character URL: https://github.com/apache/incubator-mxnet/pull/12237 Python 2 has low tolerance for Unicode characters in source files without proper encoding specified. This PR removes the

[GitHub] haojin2 commented on a change in pull request #12230: Update Operator Implementation Tutorial

2018-08-17 Thread GitBox
haojin2 commented on a change in pull request #12230: Update Operator Implementation Tutorial URL: https://github.com/apache/incubator-mxnet/pull/12230#discussion_r211051654 ## File path: docs/faq/add_op_in_backend.md ## @@ -543,53 +545,81 @@ If you use python, when you

[GitHub] haojin2 commented on a change in pull request #12230: Update Operator Implementation Tutorial

2018-08-17 Thread GitBox
haojin2 commented on a change in pull request #12230: Update Operator Implementation Tutorial URL: https://github.com/apache/incubator-mxnet/pull/12230#discussion_r211051490 ## File path: docs/faq/add_op_in_backend.md ## @@ -543,53 +545,81 @@ If you use python, when you

[GitHub] haojin2 commented on a change in pull request #12230: Update Operator Implementation Tutorial

2018-08-17 Thread GitBox
haojin2 commented on a change in pull request #12230: Update Operator Implementation Tutorial URL: https://github.com/apache/incubator-mxnet/pull/12230#discussion_r211051000 ## File path: docs/faq/add_op_in_backend.md ## @@ -543,53 +545,81 @@ If you use python, when you

[GitHub] haojin2 commented on issue #12227: Re-enable randomized test_bilinear_sampler

2018-08-17 Thread GitBox
haojin2 commented on issue #12227: Re-enable randomized test_bilinear_sampler URL: https://github.com/apache/incubator-mxnet/pull/12227#issuecomment-414008433 @anirudh2290 @nswamy @sandeep-krishnamurthy Should be good to merge.

[GitHub] marcoabreu commented on issue #12200: [MXNET-696] 'make pylint' should run a current version of PyLint

2018-08-17 Thread GitBox
marcoabreu commented on issue #12200: [MXNET-696] 'make pylint' should run a current version of PyLint URL: https://github.com/apache/incubator-mxnet/pull/12200#issuecomment-414007922 @cclaus please fix merge conflicts This

[GitHub] marcoabreu commented on issue #12200: [MXNET-696] 'make pylint' should run a current version of PyLint

2018-08-17 Thread GitBox
marcoabreu commented on issue #12200: [MXNET-696] 'make pylint' should run a current version of PyLint URL: https://github.com/apache/incubator-mxnet/pull/12200#issuecomment-414007815 Kellen is on it. Sorry for the inconveniences.

[GitHub] marcoabreu commented on issue #12231: Revert "[MXNET-696] Define cmp() in Python 3 for line 222 (#12191)"

2018-08-17 Thread GitBox
marcoabreu commented on issue #12231: Revert "[MXNET-696] Define cmp() in Python 3 for line 222 (#12191)" URL: https://github.com/apache/incubator-mxnet/pull/12231#issuecomment-414007516

[GitHub] marcoabreu commented on issue #12235: WIP: Disable TRT due to flaky builds

2018-08-17 Thread GitBox
marcoabreu commented on issue #12235: WIP: Disable TRT due to flaky builds URL: https://github.com/apache/incubator-mxnet/pull/12235#issuecomment-414007268 Please wait with merging until we give green light. This is an

[GitHub] andrewfayres commented on issue #11719: test_operator.test_laop_2 has fixed seed that can mask flakiness

2018-08-17 Thread GitBox
andrewfayres commented on issue #11719: test_operator.test_laop_2 has fixed seed that can mask flakiness URL: https://github.com/apache/incubator-mxnet/issues/11719#issuecomment-414003799 Issue has been fixed and can be closed

[GitHub] KellenSunderland opened a new pull request #12235: Disable TRT due to flaky builds

2018-08-17 Thread GitBox
KellenSunderland opened a new pull request #12235: Disable TRT due to flaky builds URL: https://github.com/apache/incubator-mxnet/pull/12235 ## Description ## This PR disables TRT. It can be used to remove flakiness in if https://github.com/apache/incubator-mxnet/pull/12225 does not

[GitHub] ankkhedia commented on issue #5052: MXNET R, GPU speed-up much less for regression example than classification

2018-08-17 Thread GitBox
ankkhedia commented on issue #5052: MXNET R, GPU speed-up much less for regression example than classification URL: https://github.com/apache/incubator-mxnet/issues/5052#issuecomment-414001418 @matt32106 @khalida With the latest MXNetR version (Windows GPU - cuda 80), I got the

[GitHub] marcoabreu commented on issue #12225: [MXNET-703] Remove race condition in onnx build

2018-08-17 Thread GitBox
marcoabreu commented on issue #12225: [MXNET-703] Remove race condition in onnx build URL: https://github.com/apache/incubator-mxnet/pull/12225#issuecomment-414001105 For the record: Tracked at https://github.com/onnx/onnx/issues/933

[GitHub] KellenSunderland opened a new issue #12234: TensorRT builds flaky

2018-08-17 Thread GitBox
KellenSunderland opened a new issue #12234: TensorRT builds flaky URL: https://github.com/apache/incubator-mxnet/issues/12234 TensorRT builds are currently flaky due to https://github.com/onnx/onnx/pull/934/files Example failure here:

[GitHub] cclauss commented on issue #12231: Revert "[MXNET-696] Define cmp() in Python 3 for line 222 (#12191)"

2018-08-17 Thread GitBox
cclauss commented on issue #12231: Revert "[MXNET-696] Define cmp() in Python 3 for line 222 (#12191)" URL: https://github.com/apache/incubator-mxnet/pull/12231#issuecomment-413999694 What problem did #12191 cause? This is

[GitHub] rahul003 commented on a change in pull request #11234: [MXNET-535] Fix bugs in LR Schedulers and add warmup

2018-08-17 Thread GitBox
rahul003 commented on a change in pull request #11234: [MXNET-535] Fix bugs in LR Schedulers and add warmup URL: https://github.com/apache/incubator-mxnet/pull/11234#discussion_r211042848 ## File path: python/mxnet/lr_scheduler.py ## @@ -29,8 +30,31 @@ class

[GitHub] rahul003 commented on a change in pull request #11234: [MXNET-535] Fix bugs in LR Schedulers and add warmup

2018-08-17 Thread GitBox
rahul003 commented on a change in pull request #11234: [MXNET-535] Fix bugs in LR Schedulers and add warmup URL: https://github.com/apache/incubator-mxnet/pull/11234#discussion_r211042880 ## File path: python/mxnet/lr_scheduler.py ## @@ -138,33 +171,73 @@ def

[GitHub] rahul003 edited a comment on issue #11234: [MXNET-535] Fix bugs in LR Schedulers and add warmup

2018-08-17 Thread GitBox
rahul003 edited a comment on issue #11234: [MXNET-535] Fix bugs in LR Schedulers and add warmup URL: https://github.com/apache/incubator-mxnet/pull/11234#issuecomment-413998969 Hopefully this will give committers confidence to merge ## Interfaces ### CosineScheduler ``` def

[GitHub] rahul003 commented on issue #11234: [MXNET-535] Fix bugs in LR Schedulers and add warmup

2018-08-17 Thread GitBox
rahul003 commented on issue #11234: [MXNET-535] Fix bugs in LR Schedulers and add warmup URL: https://github.com/apache/incubator-mxnet/pull/11234#issuecomment-413998969 Hopefully this will give committers confidence to merge ## Interfaces ### CosineScheduler ``` def

[GitHub] cclauss closed issue #8270: 5 undefined names in Python code

2018-08-17 Thread GitBox
cclauss closed issue #8270: 5 undefined names in Python code URL: https://github.com/apache/incubator-mxnet/issues/8270 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] larroy commented on issue #11875: fix flaky test: test_broadcast_binary_op

2018-08-17 Thread GitBox
larroy commented on issue #11875: fix flaky test: test_broadcast_binary_op URL: https://github.com/apache/incubator-mxnet/pull/11875#issuecomment-413998325 (lgtm) This is an automated message from the Apache Git Service. To

[GitHub] larroy commented on issue #11875: fix flaky test: test_broadcast_binary_op

2018-08-17 Thread GitBox
larroy commented on issue #11875: fix flaky test: test_broadcast_binary_op URL: https://github.com/apache/incubator-mxnet/pull/11875#issuecomment-413997372 Got it,  can you remove the extra cast? or does it have a purpose / side effect?

[GitHub] KellenSunderland opened a new pull request #12233: WIP

2018-08-17 Thread GitBox
KellenSunderland opened a new pull request #12233: WIP URL: https://github.com/apache/incubator-mxnet/pull/12233 WIP This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] azai91 commented on issue #11875: fix flaky test: test_broadcast_binary_op

2018-08-17 Thread GitBox
azai91 commented on issue #11875: fix flaky test: test_broadcast_binary_op URL: https://github.com/apache/incubator-mxnet/pull/11875#issuecomment-413997251 you added the fix for test_binary_op, test_broadcast_binary_op has the same issue.

[GitHub] anirudh2290 closed issue #10026: MXNET_MKLDNN_DEBUG=1 produces errors

2018-08-17 Thread GitBox
anirudh2290 closed issue #10026: MXNET_MKLDNN_DEBUG=1 produces errors URL: https://github.com/apache/incubator-mxnet/issues/10026 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] larroy commented on a change in pull request #11875: fix flaky test: test_broadcast_binary_op

2018-08-17 Thread GitBox
larroy commented on a change in pull request #11875: fix flaky test: test_broadcast_binary_op URL: https://github.com/apache/incubator-mxnet/pull/11875#discussion_r211040540 ## File path: tests/python/unittest/test_operator.py ## @@ -1911,10 +1911,18 @@ def test_bdiv(a,

[GitHub] larroy commented on a change in pull request #11875: fix flaky test: test_broadcast_binary_op

2018-08-17 Thread GitBox
larroy commented on a change in pull request #11875: fix flaky test: test_broadcast_binary_op URL: https://github.com/apache/incubator-mxnet/pull/11875#discussion_r211040540 ## File path: tests/python/unittest/test_operator.py ## @@ -1911,10 +1911,18 @@ def test_bdiv(a,

[GitHub] lanking520 opened a new pull request #12232: [MXNET-729][WIP] Scala Examples memory leak fix

2018-08-17 Thread GitBox
lanking520 opened a new pull request #12232: [MXNET-729][WIP] Scala Examples memory leak fix URL: https://github.com/apache/incubator-mxnet/pull/12232 ## Description ## Currently the Scala integration test running are strong influenced by CUDA memory not enough. In order to address

[GitHub] azai91 commented on issue #11875: fix flaky test: test_broadcast_binary_op

2018-08-17 Thread GitBox
azai91 commented on issue #11875: fix flaky test: test_broadcast_binary_op URL: https://github.com/apache/incubator-mxnet/pull/11875#issuecomment-413997043 test_bmod is still commented as flaky. This is an automated message

[GitHub] larroy commented on a change in pull request #11875: fix flaky test: test_broadcast_binary_op

2018-08-17 Thread GitBox
larroy commented on a change in pull request #11875: fix flaky test: test_broadcast_binary_op URL: https://github.com/apache/incubator-mxnet/pull/11875#discussion_r211040540 ## File path: tests/python/unittest/test_operator.py ## @@ -1911,10 +1911,18 @@ def test_bdiv(a,

[GitHub] azai91 commented on issue #10026: MXNET_MKLDNN_DEBUG=1 produces errors

2018-08-17 Thread GitBox
azai91 commented on issue #10026: MXNET_MKLDNN_DEBUG=1 produces errors URL: https://github.com/apache/incubator-mxnet/issues/10026#issuecomment-413996391 above PR addresses issue. @marcoabreu can you close? This is an

[GitHub] larroy commented on issue #11875: fix flaky test: test_broadcast_binary_op

2018-08-17 Thread GitBox
larroy commented on issue #11875: fix flaky test: test_broadcast_binary_op URL: https://github.com/apache/incubator-mxnet/pull/11875#issuecomment-413996057 Was this broken again? I had fixed it. Sorry I have deja vu now This

[GitHub] nswamy closed issue #7525: Different Results Generated from 0.10.1 MXNET R Package and 0.9.4 MXNET R Package

2018-08-17 Thread GitBox
nswamy closed issue #7525: Different Results Generated from 0.10.1 MXNET R Package and 0.9.4 MXNET R Package URL: https://github.com/apache/incubator-mxnet/issues/7525 This is an automated message from the Apache Git

[GitHub] nswamy closed issue #7262: use R3.3.0 and im2rec.py to run cnn and always failed

2018-08-17 Thread GitBox
nswamy closed issue #7262: use R3.3.0 and im2rec.py to run cnn and always failed URL: https://github.com/apache/incubator-mxnet/issues/7262 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] rahul003 commented on issue #11842: Strange results from fp16 benchmarks on Alexnet on CIFAR10

2018-08-17 Thread GitBox
rahul003 commented on issue #11842: Strange results from fp16 benchmarks on Alexnet on CIFAR10 URL: https://github.com/apache/incubator-mxnet/issues/11842#issuecomment-413995610 @mklissa Did you notice a speedup with a single GPU as well? Maybe it has improved now. With multi GPU I

[GitHub] KellenSunderland commented on issue #12225: [MXNET-703] Remove race condition in onnx build

2018-08-17 Thread GitBox
KellenSunderland commented on issue #12225: [MXNET-703] Remove race condition in onnx build URL: https://github.com/apache/incubator-mxnet/pull/12225#issuecomment-413993849 There's a build target race condition in the onnx cmakelist file. Basically target 1 relies on target 2 but they

[GitHub] marcoabreu commented on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222"

2018-08-17 Thread GitBox
marcoabreu commented on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222" URL: https://github.com/apache/incubator-mxnet/pull/12229#issuecomment-413993245 Thank you! This is an automated message from

[GitHub] marcoabreu closed pull request #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222"

2018-08-17 Thread GitBox
marcoabreu closed pull request #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222" URL: https://github.com/apache/incubator-mxnet/pull/12229 This is an automated message from the Apache Git Service. To

[GitHub] vandanavk commented on issue #12231: Revert "[MXNET-696] Define cmp() in Python 3 for line 222 (#12191)"

2018-08-17 Thread GitBox
vandanavk commented on issue #12231: Revert "[MXNET-696] Define cmp() in Python 3 for line 222 (#12191)" URL: https://github.com/apache/incubator-mxnet/pull/12231#issuecomment-413992868 @marcoabreu @haojin2 @piyushghai @cclauss

[GitHub] piyushghai commented on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222"

2018-08-17 Thread GitBox
piyushghai commented on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222" URL: https://github.com/apache/incubator-mxnet/pull/12229#issuecomment-413992790 Check PR #12231 . That includes just one commit required. @marcoabreu Feel free to close this one.

[GitHub] vandanavk opened a new pull request #12231: Revert "[MXNET-696] Define cmp() in Python 3 for line 222 (#12191)"

2018-08-17 Thread GitBox
vandanavk opened a new pull request #12231: Revert "[MXNET-696] Define cmp() in Python 3 for line 222 (#12191)" URL: https://github.com/apache/incubator-mxnet/pull/12231 This reverts commit 54ed3e5d95b997bf1957b4e5cd7f117894ed4a19. ## Description ## Raising PR instead of

[GitHub] marcoabreu commented on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222"

2018-08-17 Thread GitBox
marcoabreu commented on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222" URL: https://github.com/apache/incubator-mxnet/pull/12229#issuecomment-413991385 If you click on the commits in this PR, you will see that they are all empty except the last one. Please feel

[GitHub] piyushghai commented on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222"

2018-08-17 Thread GitBox
piyushghai commented on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222" URL: https://github.com/apache/incubator-mxnet/pull/12229#issuecomment-413991314 The file diff looks good as it was the only change in that PR. Not sure, why the other commits are showing

[GitHub] haojin2 commented on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222"

2018-08-17 Thread GitBox
haojin2 commented on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222" URL: https://github.com/apache/incubator-mxnet/pull/12229#issuecomment-413990925 @marcoabreu Could you do a double-check please? Cause one of the commits on the list shows that it's reverting an

[GitHub] thomelane commented on issue #11006: bug about sum in loss

2018-08-17 Thread GitBox
thomelane commented on issue #11006: bug about sum in loss URL: https://github.com/apache/incubator-mxnet/issues/11006#issuecomment-413990840 @sandeep-krishnamurthy looks like we can close this issue due to some other bugs elsewhere in script, as found by @Ishitori. `MakeLoss` issues look

[GitHub] marcoabreu edited a comment on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222"

2018-08-17 Thread GitBox
marcoabreu edited a comment on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222" URL: https://github.com/apache/incubator-mxnet/pull/12229#issuecomment-413988592 I just pressed the revert button, I don't know what GitHub is doing there ._. Please have a look

[GitHub] anirudhacharya opened a new pull request #12230: Update Operator Implementation Tutorial

2018-08-17 Thread GitBox
anirudhacharya opened a new pull request #12230: Update Operator Implementation Tutorial URL: https://github.com/apache/incubator-mxnet/pull/12230 ## Description ## Update the tutorial for implementing operators in the backend. ### Changes ### - [ ] Adds more details for the

[GitHub] thomelane commented on issue #11079: Is it possible to apply intersection/union on mx.nd.array

2018-08-17 Thread GitBox
thomelane commented on issue #11079: Is it possible to apply intersection/union on mx.nd.array URL: https://github.com/apache/incubator-mxnet/issues/11079#issuecomment-413989225 @reminisce @indhub can we remove the `Question` tag please, thanks!

[GitHub] marcoabreu commented on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222"

2018-08-17 Thread GitBox
marcoabreu commented on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222" URL: https://github.com/apache/incubator-mxnet/pull/12229#issuecomment-413988900 I'm only on my phone right now, so I can't make a revert with the command line. In the end, the commits will be

[GitHub] thomelane commented on issue #11144: Save/Load Model?

2018-08-17 Thread GitBox
thomelane commented on issue #11144: Save/Load Model? URL: https://github.com/apache/incubator-mxnet/issues/11144#issuecomment-413988836 @szha @piiswrong any comments to the above? otherwise can we close this issue due to not hearing back from author, thanks! @anchen1011 can be

[GitHub] marcoabreu commented on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222"

2018-08-17 Thread GitBox
marcoabreu commented on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222" URL: https://github.com/apache/incubator-mxnet/pull/12229#issuecomment-413988592 I just pressed the revert but, I don't know what GitHub is doing there ._. Have a look at the diff, it

[GitHub] marcoabreu edited a comment on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222"

2018-08-17 Thread GitBox
marcoabreu edited a comment on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222" URL: https://github.com/apache/incubator-mxnet/pull/12229#issuecomment-413988592 I just pressed the revert button, I don't know what GitHub is doing there ._. Have a look at the

[GitHub] haojin2 edited a comment on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222"

2018-08-17 Thread GitBox
haojin2 edited a comment on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222" URL: https://github.com/apache/incubator-mxnet/pull/12229#issuecomment-413988255 Just wonder why are you also reverting a bunch of other un-related changes? (I'm taking a look at the

[GitHub] apeforest commented on issue #12174: [MXNET-806] Report error when shape mismatch in "where" operator

2018-08-17 Thread GitBox
apeforest commented on issue #12174: [MXNET-806] Report error when shape mismatch in "where" operator URL: https://github.com/apache/incubator-mxnet/pull/12174#issuecomment-413988270 @anirudh2290 Can we merge this PR if no other reviewers comments?

[GitHub] haojin2 commented on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222"

2018-08-17 Thread GitBox
haojin2 commented on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222" URL: https://github.com/apache/incubator-mxnet/pull/12229#issuecomment-413988255 Just wonder why are you also reverting a bunch of other un-related changes?

[GitHub] marcoabreu commented on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222"

2018-08-17 Thread GitBox
marcoabreu commented on issue #12229: Revert "[MXNET-696] Define cmp() in Python 3 for line 222" URL: https://github.com/apache/incubator-mxnet/pull/12229#issuecomment-413988140 @piyushghai @cclaus fyi This is an automated

[GitHub] stu1130 commented on a change in pull request #12131: [MXNET-737][WIP] Add last batch handle for imageiter

2018-08-17 Thread GitBox
stu1130 commented on a change in pull request #12131: [MXNET-737][WIP] Add last batch handle for imageiter URL: https://github.com/apache/incubator-mxnet/pull/12131#discussion_r211032169 ## File path: python/mxnet/image/image.py ## @@ -1149,22 +1155,49 @@ def

  1   2   3   >