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

2018-08-15 Thread GitBox
apeforest commented on a change in pull request #12174: [MXNET-806] Report error when shape mismatch in "where" operator URL: https://github.com/apache/incubator-mxnet/pull/12174#discussion_r210339535 ## File path: tests/python/unittest/test_operator.py ## @@ -4442,6

[GitHub] cclauss commented on issue #8270: 10 Undefined names in Python code

2018-08-15 Thread GitBox
cclauss commented on issue #8270: 10 Undefined names in Python code URL: https://github.com/apache/incubator-mxnet/issues/8270#issuecomment-413264002 See PEP8 for reasons why star imports are for the birds... On the `python/mxnet/ndarray/__init__.py` issue, you could try adding a

[GitHub] sad- commented on issue #11858: Update contribute.md (Fix links to subscribe for users and contributors)

2018-08-15 Thread GitBox
sad- commented on issue #11858: Update contribute.md (Fix links to subscribe for users and contributors) URL: https://github.com/apache/incubator-mxnet/pull/11858#issuecomment-413261893 https://github.com/apache/incubator-mxnet/pull/12183

[GitHub] sad- opened a new pull request #12183: Patch: Update contribute.md (Fix links to subscribe for users and contributors)

2018-08-15 Thread GitBox
sad- opened a new pull request #12183: Patch: Update contribute.md (Fix links to subscribe for users and contributors) URL: https://github.com/apache/incubator-mxnet/pull/12183 ## Description ## This change is to update the contribute.md page to use the right references/links for the

[GitHub] sad- closed pull request #11858: Update contribute.md (Fix links to subscribe for users and contributors)

2018-08-15 Thread GitBox
sad- closed pull request #11858: Update contribute.md (Fix links to subscribe for users and contributors) URL: https://github.com/apache/incubator-mxnet/pull/11858 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] cclauss removed a comment on issue #8270: 10 Undefined names in Python code

2018-08-15 Thread GitBox
cclauss removed a comment on issue #8270: 10 Undefined names in Python code URL: https://github.com/apache/incubator-mxnet/issues/8270#issuecomment-413260434 __ndarray__ --> __mxnet.ndarray__ This is an automated message

[GitHub] cclauss edited a comment on issue #8270: 10 Undefined names in Python code

2018-08-15 Thread GitBox
cclauss edited a comment on issue #8270: 10 Undefined names in Python code URL: https://github.com/apache/incubator-mxnet/issues/8270#issuecomment-413260434 __ndarray__ --> __mxnet.ndarray__ This is an automated message from

[GitHub] cclauss commented on issue #8270: 10 Undefined names in Python code

2018-08-15 Thread GitBox
cclauss commented on issue #8270: 10 Undefined names in Python code URL: https://github.com/apache/incubator-mxnet/issues/8270#issuecomment-413260434 __mxnet__ --> __mxnet.ndarray__ This is an automated message from the

[GitHub] ankkhedia commented on issue #12178: fix flasky unittest for deformable psroi pooling

2018-08-15 Thread GitBox
ankkhedia commented on issue #12178: fix flasky unittest for deformable psroi pooling URL: https://github.com/apache/incubator-mxnet/pull/12178#issuecomment-413260221 The changes looks good to me and have verified it locally for 2500 runs with random seeds.

[incubator-mxnet] branch master updated: [MXNET-744] Docs build tools update (#11990)

2018-08-15 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 c220974 [MXNET-744] Docs build tools

[GitHub] nswamy closed pull request #11990: [MXNET-744] Docs build tools update

2018-08-15 Thread GitBox
nswamy closed pull request #11990: [MXNET-744] Docs build tools update URL: https://github.com/apache/incubator-mxnet/pull/11990 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

[GitHub] cclauss commented on issue #8270: 10 Undefined names in Python code

2018-08-15 Thread GitBox
cclauss commented on issue #8270: 10 Undefined names in Python code URL: https://github.com/apache/incubator-mxnet/issues/8270#issuecomment-413258613 * reload(): https://docs.python.org/3/whatsnew/3.0.html#builtins * Was moved to

[incubator-mxnet] branch master updated (8e60586 -> da2ce8f)

2018-08-15 Thread nswamy
This is an automated email from the ASF dual-hosted git repository. nswamy pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 8e60586 add examples for slicing option (#11918) add da2ce8f Module predict API can accept NDArray

[GitHub] nswamy closed pull request #12166: Module predict API can accept NDArray as input

2018-08-15 Thread GitBox
nswamy closed pull request #12166: Module predict API can accept NDArray as input URL: https://github.com/apache/incubator-mxnet/pull/12166 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

[GitHub] piyushghai commented on a change in pull request #12181: Fixed backwards compatibility checker

2018-08-15 Thread GitBox
piyushghai commented on a change in pull request #12181: Fixed backwards compatibility checker URL: https://github.com/apache/incubator-mxnet/pull/12181#discussion_r210331179 ## File path: ci/docker/runtime_functions.sh ## @@ -925,13 +925,23 @@ nightly_test_javascript() {

[GitHub] sandeep-krishnamurthy commented on issue #11867: we need to update the doc of scatter_nd and gather_nd

2018-08-15 Thread GitBox
sandeep-krishnamurthy commented on issue #11867: we need to update the doc of scatter_nd and gather_nd URL: https://github.com/apache/incubator-mxnet/issues/11867#issuecomment-413257383 Reopening as we have few more pending updates - @haojin2

[GitHub] zheng-da opened a new issue #11867: we need to update the doc of scatter_nd and gather_nd

2018-08-15 Thread GitBox
zheng-da opened a new issue #11867: we need to update the doc of scatter_nd and gather_nd URL: https://github.com/apache/incubator-mxnet/issues/11867 [scatter_nd](http://mxnet.incubator.apache.org/api/python/symbol/symbol.html#mxnet.symbol.scatter_nd) and

[GitHub] sandeep-krishnamurthy closed issue #11736: test_optimizer.test_nadam has fixed seed that can mask flakiness

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

[GitHub] sandeep-krishnamurthy closed issue #11867: we need to update the doc of scatter_nd and gather_nd

2018-08-15 Thread GitBox
sandeep-krishnamurthy closed issue #11867: we need to update the doc of scatter_nd and gather_nd URL: https://github.com/apache/incubator-mxnet/issues/11867 This is an automated message from the Apache Git Service. To

[incubator-mxnet] branch master updated: add examples for slicing option (#11918)

2018-08-15 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 8e60586 add examples for slicing option

[GitHub] sandeep-krishnamurthy closed pull request #11918: Improve gather_nd and scatter_nd doc

2018-08-15 Thread GitBox
sandeep-krishnamurthy closed pull request #11918: Improve gather_nd and scatter_nd doc URL: https://github.com/apache/incubator-mxnet/pull/11918 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 commented on issue #12007: Add tutorial Gotchas using NumPy

2018-08-15 Thread GitBox
sandeep-krishnamurthy commented on issue #12007: Add tutorial Gotchas using NumPy URL: https://github.com/apache/incubator-mxnet/pull/12007#issuecomment-413254616 @aaronmarkham - ping This is an automated message from the

[incubator-mxnet] branch master updated: Partially enable flaky test for norm operator (#12027)

2018-08-15 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 391377a Partially enable flaky test for

[GitHub] sandeep-krishnamurthy closed pull request #12027: [MXNET-768] Partially enable flaky test for norm operator

2018-08-15 Thread GitBox
sandeep-krishnamurthy closed pull request #12027: [MXNET-768] Partially enable flaky test for norm operator URL: https://github.com/apache/incubator-mxnet/pull/12027 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] vandanavk commented on issue #8270: 10 Undefined names in Python code

2018-08-15 Thread GitBox
vandanavk commented on issue #8270: 10 Undefined names in Python code URL: https://github.com/apache/incubator-mxnet/issues/8270#issuecomment-413254384 Thanks @KellenSunderland @cclauss I don't see the errors ``` ./tests/python/unittest/test_engine_import.py:33:13: F821

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

2018-08-15 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_r210325874 ## File path: python/mxnet/image/image.py ## @@ -1149,22 +1158,44 @@ def

[GitHub] vdantu commented on issue #12159: C++ api executor->Forward(false) is much slower than MXPredForward?

2018-08-15 Thread GitBox
vdantu commented on issue #12159: C++ api executor->Forward(false) is much slower than MXPredForward? URL: https://github.com/apache/incubator-mxnet/issues/12159#issuecomment-413252620 @xiaojingxie : This is a good question. Could you also post this on discuss.mxnet.io ? The question

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

2018-08-15 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_r210325874 ## File path: python/mxnet/image/image.py ## @@ -1149,22 +1158,44 @@ def

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

2018-08-15 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_r210325874 ## File path: python/mxnet/image/image.py ## @@ -1149,22 +1158,44 @@ def

[incubator-mxnet] branch master updated: documentation changes. added full reference (#12153)

2018-08-15 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 57313e3 documentation changes. added

[GitHub] sandeep-krishnamurthy closed pull request #12153: documentation changes. added full reference

2018-08-15 Thread GitBox
sandeep-krishnamurthy closed pull request #12153: documentation changes. added full reference URL: https://github.com/apache/incubator-mxnet/pull/12153 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 pull request #12164: Removed fixed seed for test_nadam.

2018-08-15 Thread GitBox
sandeep-krishnamurthy closed pull request #12164: Removed fixed seed for test_nadam. URL: https://github.com/apache/incubator-mxnet/pull/12164 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

[incubator-mxnet] branch master updated: Removed fixed seed and increased learning rate and tolerance for test_nadam (#12164)

2018-08-15 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 5b9251b Removed fixed seed and increased

[GitHub] vdantu commented on issue #12179: Spelling mistake in "mxnet/symbol/image.py" in which "gen_image" is written to be "gen_iamge"

2018-08-15 Thread GitBox
vdantu commented on issue #12179: Spelling mistake in "mxnet/symbol/image.py" in which "gen_image" is written to be "gen_iamge" URL: https://github.com/apache/incubator-mxnet/issues/12179#issuecomment-413251902 @PlumedSerpent Thanks for pointing it out.. @yajiedesign :

[GitHub] vandanavk opened a new pull request #12182: Remove Epoch training metric log

2018-08-15 Thread GitBox
vandanavk opened a new pull request #12182: Remove Epoch training metric log URL: https://github.com/apache/incubator-mxnet/pull/12182 ## Description ## The log `self.logger.info('Epoch[%d] Train-%s=%f', epoch, name, val)` gives the user the impression that the metric printed is for the

[GitHub] sandeep-krishnamurthy commented on a change in pull request #12167: Add test to check that binded is not set when exception thrown

2018-08-15 Thread GitBox
sandeep-krishnamurthy commented on a change in pull request #12167: Add test to check that binded is not set when exception thrown URL: https://github.com/apache/incubator-mxnet/pull/12167#discussion_r210322276 ## File path: tests/python/unittest/test_module.py ## @@

[GitHub] sandeep-krishnamurthy commented on a change in pull request #12167: Add test to check that binded is not set when exception thrown

2018-08-15 Thread GitBox
sandeep-krishnamurthy commented on a change in pull request #12167: Add test to check that binded is not set when exception thrown URL: https://github.com/apache/incubator-mxnet/pull/12167#discussion_r210322310 ## File path: tests/python/unittest/test_module.py ## @@

[GitHub] lebeg commented on issue #11916: [MXNET-744] Sphinx error reduction

2018-08-15 Thread GitBox
lebeg commented on issue #11916: [MXNET-744] Sphinx error reduction URL: https://github.com/apache/incubator-mxnet/pull/11916#issuecomment-413247607 Looks good This is an automated message from the Apache Git Service. To

[GitHub] chsin commented on issue #12176: MXPredReshape bug: need to reshape softmax_label

2018-08-15 Thread GitBox
chsin commented on issue #12176: MXPredReshape bug: need to reshape softmax_label URL: https://github.com/apache/incubator-mxnet/pull/12176#issuecomment-413247433 @chinakook, To predict using the C API, you use `MXPredCreate` to load the pretrained symbols and weights and this works fine

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

2018-08-15 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_r210321441 ## File path: python/mxnet/image/image.py ## @@ -1207,8 +1237,28 @@ def next(self):

[GitHub] lebeg commented on issue #11990: [MXNET-744] Docs build tools update

2018-08-15 Thread GitBox
lebeg commented on issue #11990: [MXNET-744] Docs build tools update URL: https://github.com/apache/incubator-mxnet/pull/11990#issuecomment-413246375 LGTM This is an automated message from the Apache Git Service. To respond

[GitHub] pengzhao-intel commented on issue #12058: MKLDNN can be turned off with env var

2018-08-15 Thread GitBox
pengzhao-intel commented on issue #12058: MKLDNN can be turned off with env var URL: https://github.com/apache/incubator-mxnet/pull/12058#issuecomment-413236915 Looks fine now even the fallback log is appeared many times :) The good thing is the user would not miss this important info.

[GitHub] lebeg commented on a change in pull request #12181: Fixed backwards compatibility checker

2018-08-15 Thread GitBox
lebeg commented on a change in pull request #12181: Fixed backwards compatibility checker URL: https://github.com/apache/incubator-mxnet/pull/12181#discussion_r210300434 ## File path: ci/docker/runtime_functions.sh ## @@ -925,13 +925,23 @@ nightly_test_javascript() {

[GitHub] larroy commented on a change in pull request #12181: Fixed backwards compatibility checker

2018-08-15 Thread GitBox
larroy commented on a change in pull request #12181: Fixed backwards compatibility checker URL: https://github.com/apache/incubator-mxnet/pull/12181#discussion_r210293910 ## File path: ci/docker/runtime_functions.sh ## @@ -925,13 +925,23 @@ nightly_test_javascript() {

[GitHub] Oh233 commented on a change in pull request #12178: fix flasky unittest for deformable psroi pooling

2018-08-15 Thread GitBox
Oh233 commented on a change in pull request #12178: fix flasky unittest for deformable psroi pooling URL: https://github.com/apache/incubator-mxnet/pull/12178#discussion_r210292861 ## File path: tests/python/unittest/test_operator.py ## @@ -5010,15 +5067,11 @@ def

[GitHub] lebeg opened a new pull request #12181: Fixed backwards compatibility checker

2018-08-15 Thread GitBox
lebeg opened a new pull request #12181: Fixed backwards compatibility checker URL: https://github.com/apache/incubator-mxnet/pull/12181 ## Description ## The corresponding python, python2, pip, pip2 were messed up. ## Checklist ## ### Essentials ### - [x] Changes are

[GitHub] marcoabreu commented on issue #12180: Make the Ubuntu scripts executable after download.

2018-08-15 Thread GitBox
marcoabreu commented on issue #12180: Make the Ubuntu scripts executable after download. URL: https://github.com/apache/incubator-mxnet/pull/12180#issuecomment-413210331 Feel free to just change the chmod of the files directly

[GitHub] wcollins-ebsco closed issue #8191: RNN BucketSentenceIter - Must load all data into memory?

2018-08-15 Thread GitBox
wcollins-ebsco closed issue #8191: RNN BucketSentenceIter - Must load all data into memory? URL: https://github.com/apache/incubator-mxnet/issues/8191 This is an automated message from the Apache Git Service. To respond to

[GitHub] wcollins-ebsco commented on issue #8191: RNN BucketSentenceIter - Must load all data into memory?

2018-08-15 Thread GitBox
wcollins-ebsco commented on issue #8191: RNN BucketSentenceIter - Must load all data into memory? URL: https://github.com/apache/incubator-mxnet/issues/8191#issuecomment-413201572 Thank you for your follow up. I switched to using scikit-learn and tensorflow due to this and many other

[GitHub] lebeg commented on issue #12160: Remove conflicting llvm OpenMP from cmake builds

2018-08-15 Thread GitBox
lebeg commented on issue #12160: Remove conflicting llvm OpenMP from cmake builds URL: https://github.com/apache/incubator-mxnet/pull/12160#issuecomment-413201017 @szha yes, but for AppleClang we are failing on this

[GitHub] anirudh2290 commented on a change in pull request #12174: [MXNET-806] Report error when shape mismatch in "where" operator

2018-08-15 Thread GitBox
anirudh2290 commented on a change in pull request #12174: [MXNET-806] Report error when shape mismatch in "where" operator URL: https://github.com/apache/incubator-mxnet/pull/12174#discussion_r210270908 ## File path: tests/python/unittest/test_operator.py ## @@ -4442,6

[GitHub] larroy commented on a change in pull request #12161: [WIP] A solution to prevent zombie containers locally and in CI

2018-08-15 Thread GitBox
larroy commented on a change in pull request #12161: [WIP] A solution to prevent zombie containers locally and in CI URL: https://github.com/apache/incubator-mxnet/pull/12161#discussion_r210256630 ## File path: ci/build.py ## @@ -180,63 +245,133 @@ def

[GitHub] lebeg commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake

2018-08-15 Thread GitBox
lebeg commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake URL: https://github.com/apache/incubator-mxnet/pull/11148#discussion_r210255985 ## File path: src/operator/rnn_impl.h ## @@ -994,7 +998,6 @@ void

[GitHub] marcoabreu edited a comment on issue #12161: [WIP] A solution to prevent zombie containers locally and in CI

2018-08-15 Thread GitBox
marcoabreu edited a comment on issue #12161: [WIP] A solution to prevent zombie containers locally and in CI URL: https://github.com/apache/incubator-mxnet/pull/12161#issuecomment-413163908 See Antons comment about your change deleting the entire shared CCache, for example. I'm closing

[GitHub] marcoabreu edited a comment on issue #12161: [WIP] A solution to prevent zombie containers locally and in CI

2018-08-15 Thread GitBox
marcoabreu edited a comment on issue #12161: [WIP] A solution to prevent zombie containers locally and in CI URL: https://github.com/apache/incubator-mxnet/pull/12161#issuecomment-413163908 See Antons comment about your change deleting the entire shared CCache, for example. I'm closing

[GitHub] marcoabreu edited a comment on issue #12161: [WIP] A solution to prevent zombie containers locally and in CI

2018-08-15 Thread GitBox
marcoabreu edited a comment on issue #12161: [WIP] A solution to prevent zombie containers locally and in CI URL: https://github.com/apache/incubator-mxnet/pull/12161#issuecomment-413163908 See Antons comment about your change deleting the entire shared CCache, for example. I'm closing

[GitHub] marcoabreu commented on issue #12161: [WIP] A solution to prevent zombie containers locally and in CI

2018-08-15 Thread GitBox
marcoabreu commented on issue #12161: [WIP] A solution to prevent zombie containers locally and in CI URL: https://github.com/apache/incubator-mxnet/pull/12161#issuecomment-413163908 See Antons comment about your change deleting the entire shared CCache, for example. I'm closing now since

[GitHub] marcoabreu closed pull request #12161: [WIP] A solution to prevent zombie containers locally and in CI

2018-08-15 Thread GitBox
marcoabreu closed pull request #12161: [WIP] A solution to prevent zombie containers locally and in CI URL: https://github.com/apache/incubator-mxnet/pull/12161 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] lebeg commented on a change in pull request #12161: [WIP] A solution to prevent zombie containers locally and in CI

2018-08-15 Thread GitBox
lebeg commented on a change in pull request #12161: [WIP] A solution to prevent zombie containers locally and in CI URL: https://github.com/apache/incubator-mxnet/pull/12161#discussion_r210226299 ## File path: ci/dockerfiles/runtime_functions.sh ## @@ -305,6 +305,14 @@

[GitHub] lebeg commented on a change in pull request #12161: [WIP] A solution to prevent zombie containers locally and in CI

2018-08-15 Thread GitBox
lebeg commented on a change in pull request #12161: [WIP] A solution to prevent zombie containers locally and in CI URL: https://github.com/apache/incubator-mxnet/pull/12161#discussion_r210226128 ## File path: ci/build.py ## @@ -37,11 +41,54 @@ import platform from copy

[GitHub] lebeg commented on issue #12102: site-wide social include

2018-08-15 Thread GitBox
lebeg commented on issue #12102: site-wide social include URL: https://github.com/apache/incubator-mxnet/pull/12102#issuecomment-413151195 I tested on my Mac / Chrome / 34-inch (3440 x 1440) and the social icons don't look good. ![screen shot 2018-08-15 at 11 46

[GitHub] larroy commented on a change in pull request #12161: [WIP] A solution to prevent zombie containers locally and in CI

2018-08-15 Thread GitBox
larroy commented on a change in pull request #12161: [WIP] A solution to prevent zombie containers locally and in CI URL: https://github.com/apache/incubator-mxnet/pull/12161#discussion_r210222868 ## File path: ci/build.py ## @@ -180,63 +245,133 @@ def

[GitHub] larroy commented on a change in pull request #12161: [WIP] A solution to prevent zombie containers locally and in CI

2018-08-15 Thread GitBox
larroy commented on a change in pull request #12161: [WIP] A solution to prevent zombie containers locally and in CI URL: https://github.com/apache/incubator-mxnet/pull/12161#discussion_r210222518 ## File path: ci/build.py ## @@ -23,7 +23,11 @@ """ __author__ = 'Marco

[GitHub] larroy commented on a change in pull request #12161: [WIP] A solution to prevent zombie containers locally and in CI

2018-08-15 Thread GitBox
larroy commented on a change in pull request #12161: [WIP] A solution to prevent zombie containers locally and in CI URL: https://github.com/apache/incubator-mxnet/pull/12161#discussion_r210222498 ## File path: ci/build.py ## @@ -37,11 +41,54 @@ import platform from

[GitHub] larroy commented on issue #12161: [WIP] A solution to prevent zombie containers locally and in CI

2018-08-15 Thread GitBox
larroy commented on issue #12161: [WIP] A solution to prevent zombie containers locally and in CI URL: https://github.com/apache/incubator-mxnet/pull/12161#issuecomment-413150063 @lebeg addressed your comments. This is an

[GitHub] Biserkov opened a new pull request #12180: Make the Ubuntu scripts executable after download.

2018-08-15 Thread GitBox
Biserkov opened a new pull request #12180: Make the Ubuntu scripts executable after download. URL: https://github.com/apache/incubator-mxnet/pull/12180 Just a documentation fix. This is an automated message from the Apache

[GitHub] marcoabreu commented on issue #11858: Update contribute.md (Fix links to subscribe for users and contributors)

2018-08-15 Thread GitBox
marcoabreu commented on issue #11858: Update contribute.md (Fix links to subscribe for users and contributors) URL: https://github.com/apache/incubator-mxnet/pull/11858#issuecomment-413136673 Exactly This is an automated

[incubator-mxnet] branch master updated: Remove fixed seed for test_huber tests (#12169)

2018-08-15 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 b675d69 Remove fixed seed for

[GitHub] marcoabreu closed pull request #12169: Remove fixed seed for test_huber_loss test

2018-08-15 Thread GitBox
marcoabreu closed pull request #12169: Remove fixed seed for test_huber_loss test URL: https://github.com/apache/incubator-mxnet/pull/12169 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

[GitHub] marcoabreu commented on a change in pull request #12178: fix flasky unittest for deformable psroi pooling

2018-08-15 Thread GitBox
marcoabreu commented on a change in pull request #12178: fix flasky unittest for deformable psroi pooling URL: https://github.com/apache/incubator-mxnet/pull/12178#discussion_r210208035 ## File path: tests/python/unittest/test_operator.py ## @@ -5010,15 +5067,11 @@ def

[GitHub] zhreshold closed pull request #12177: Add worker_fn argument to multiworker function

2018-08-15 Thread GitBox
zhreshold closed pull request #12177: Add worker_fn argument to multiworker function URL: https://github.com/apache/incubator-mxnet/pull/12177 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

[incubator-mxnet] branch master updated: Add worker_fn argument to multiworker function (#12177)

2018-08-15 Thread zhreshold
This is an automated email from the ASF dual-hosted git repository. zhreshold 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 cd6b1cb Add worker_fn argument to

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

2018-08-15 Thread GitBox
zhreshold 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_r210183119 ## File path: python/mxnet/image/image.py ## @@ -1179,17 +1210,16 @@ def

[GitHub] szha commented on issue #12175: Update codeowner

2018-08-15 Thread GitBox
szha commented on issue #12175: Update codeowner URL: https://github.com/apache/incubator-mxnet/pull/12175#issuecomment-413109191 code owner feature only works for people with write access. This is an automated message from

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

2018-08-15 Thread GitBox
zhreshold 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_r210182985 ## File path: python/mxnet/image/image.py ## @@ -1207,8 +1237,28 @@ def next(self):

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

2018-08-15 Thread GitBox
zhreshold 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_r210182923 ## File path: python/mxnet/image/image.py ## @@ -1179,17 +1210,16 @@ def

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

2018-08-15 Thread GitBox
zhreshold 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_r210182860 ## File path: python/mxnet/image/image.py ## @@ -1149,22 +1158,44 @@ def

[GitHub] PlumedSerpent opened a new issue #12179: Spelling mistake in "mxnet/symbol/image.py" in which "gen_image" is written to be "gen_iamge"

2018-08-15 Thread GitBox
PlumedSerpent opened a new issue #12179: Spelling mistake in "mxnet/symbol/image.py" in which "gen_image" is written to be "gen_iamge" URL: https://github.com/apache/incubator-mxnet/issues/12179 Note: Providing complete information in the most concise form is the best way to get help.

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

2018-08-15 Thread GitBox
sandeep-krishnamurthy 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_r210180105 ## File path: python/mxnet/image/image.py ## @@ -1149,22 +1158,44 @@

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

2018-08-15 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_r210180174 ## File path: python/mxnet/image/image.py ## @@ -1149,22 +1158,44 @@ def

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

2018-08-15 Thread GitBox
zhreshold 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_r210179579 ## File path: python/mxnet/image/image.py ## @@ -1149,22 +1158,44 @@ def

[GitHub] Faldict edited a comment on issue #12142: Failed to import MXNet built with TensorRT

2018-08-15 Thread GitBox
Faldict edited a comment on issue #12142: Failed to import MXNet built with TensorRT URL: https://github.com/apache/incubator-mxnet/issues/12142#issuecomment-413095946 @KellenSunderland I uninstalled protobuf 3.5.1 and rebuild the whole toolchain. At present, MXNet could be imported

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

2018-08-15 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_r210178547 ## File path: python/mxnet/image/image.py ## @@ -1149,22 +1158,44 @@ def

[GitHub] sandeep-krishnamurthy commented on issue #12137: [MXNET-696] Fix undefined name errors

2018-08-15 Thread GitBox
sandeep-krishnamurthy commented on issue #12137: [MXNET-696] Fix undefined name errors URL: https://github.com/apache/incubator-mxnet/pull/12137#issuecomment-413102084 @anirudhacharya - If your comments are addressed, this is good to go?

<    1   2   3