[GitHub] reminisce closed pull request #11937: Fix quantized graphpass bug

reminisce closed pull request #11937: Fix quantized graphpass bug URL: https://github.com/apache/incubator-mxnet/pull/11937 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

[incubator-mxnet] branch master updated: Fix quantized graphpass bug (#11937)

This is an automated email from the ASF dual-hosted git repository. reminisce 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 d076d10 Fix quantized graphpass

[GitHub] wkcn commented on a change in pull request #12047: [MXNET-779]Add DLPack Transformation API

wkcn commented on a change in pull request #12047: [MXNET-779]Add DLPack Transformation API URL: https://github.com/apache/incubator-mxnet/pull/12047#discussion_r209442740 ## File path: python/mxnet/_ctypes/ndarray.py ## @@ -31,21 +31,24 @@ class NDArrayBase(object):

[GitHub] wkcn commented on a change in pull request #12047: [MXNET-779]Add DLPack Transformation API

wkcn commented on a change in pull request #12047: [MXNET-779]Add DLPack Transformation API URL: https://github.com/apache/incubator-mxnet/pull/12047#discussion_r209442740 ## File path: python/mxnet/_ctypes/ndarray.py ## @@ -31,21 +31,24 @@ class NDArrayBase(object):

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

vandanavk commented on issue #12137: [MXNET-696] Fix undefined name errors URL: https://github.com/apache/incubator-mxnet/pull/12137#issuecomment-412317196 @piiswrong @burness @ZihengJiang @iflament @antinucleon This is an

[GitHub] vandanavk opened a new pull request #12137: Fix undefined name errors

vandanavk opened a new pull request #12137: Fix undefined name errors URL: https://github.com/apache/incubator-mxnet/pull/12137 ## Description ## Pylint throws an error for variables that are used but not defined previously. Since these variables are found in paths that are not

[GitHub] vandanavk commented on issue #12099: Fix precision issue of test case test_rnnrelu_bidirectional

vandanavk commented on issue #12099: Fix precision issue of test case test_rnnrelu_bidirectional URL: https://github.com/apache/incubator-mxnet/pull/12099#issuecomment-412311479 @eric-haibin-lin Please change the tag to pr-awaiting-merge

[GitHub] marcoabreu commented on issue #11493: Fix MXPredReshape in the c_predict_api

marcoabreu commented on issue #11493: Fix MXPredReshape in the c_predict_api URL: https://github.com/apache/incubator-mxnet/pull/11493#issuecomment-412311230 No worries, thanks for your contribution and addressing the comments. Seems like you hit a flaky test. Please just make an empty

[GitHub] adaaaaaa commented on issue #12132: [Feature Request] Implementation of L-BFGS optimizer

adaa commented on issue #12132: [Feature Request] Implementation of L-BFGS optimizer URL: https://github.com/apache/incubator-mxnet/issues/12132#issuecomment-412310204 follow with interest... This is an automated

[GitHub] xinyu-intel commented on issue #11937: Fix quantized graphpass bug

xinyu-intel commented on issue #11937: Fix quantized graphpass bug URL: https://github.com/apache/incubator-mxnet/pull/11937#issuecomment-412310145 @reminisce yes, both gpu and mkldnn will come across below error when quantizing residual unit like resnet-50-v1: ``` RuntimeError:

[GitHub] zheng-da commented on a change in pull request #12106: [MXNET-795] Fix a bug that CutSubgraph works only when each subgraph has its distinct name

zheng-da commented on a change in pull request #12106: [MXNET-795] Fix a bug that CutSubgraph works only when each subgraph has its distinct name URL: https://github.com/apache/incubator-mxnet/pull/12106#discussion_r209438332 ## File path:

[GitHub] junrushao1994 commented on a change in pull request #12106: [MXNET-795] Fix a bug that CutSubgraph works only when each subgraph has its distinct name

junrushao1994 commented on a change in pull request #12106: [MXNET-795] Fix a bug that CutSubgraph works only when each subgraph has its distinct name URL: https://github.com/apache/incubator-mxnet/pull/12106#discussion_r209438186 ## File path:

[GitHub] hqucms commented on issue #11493: Fix MXPredReshape in the c_predict_api

hqucms commented on issue #11493: Fix MXPredReshape in the c_predict_api URL: https://github.com/apache/incubator-mxnet/pull/11493#issuecomment-412303107 @marcoabreu @anirudh2290 Sorry for the delay in response. I change to `rtol=1e-5, atol=1e-6` which are the ones used in

[GitHub] zheng-da commented on a change in pull request #12106: [MXNET-795] Fix a bug that CutSubgraph works only when each subgraph has its distinct name

zheng-da commented on a change in pull request #12106: [MXNET-795] Fix a bug that CutSubgraph works only when each subgraph has its distinct name URL: https://github.com/apache/incubator-mxnet/pull/12106#discussion_r209437878 ## File path:

[GitHub] reminisce commented on issue #11937: Fix quantized graphpass bug

reminisce commented on issue #11937: Fix quantized graphpass bug URL: https://github.com/apache/incubator-mxnet/pull/11937#issuecomment-412300028 @xinyu-intel Thanks for the fix. Is the unit test the one that fails without this PR?

[GitHub] nswamy closed pull request #12129: update dmlc-core for security reason

nswamy closed pull request #12129: update dmlc-core for security reason URL: https://github.com/apache/incubator-mxnet/pull/12129 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

[incubator-mxnet] branch master updated: update dmlc-core (#12129)

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 096aeb6 update dmlc-core (#12129)

[incubator-mxnet] branch master updated: zipfian random sampler without replacement (#12113)

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 6f7254c zipfian random sampler

[GitHub] szha closed pull request #12113: zipfian random sampler without replacement

szha closed pull request #12113: zipfian random sampler without replacement URL: https://github.com/apache/incubator-mxnet/pull/12113 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] hhyasdf commented on issue #7924: get stuck training on multiple machines error

hhyasdf commented on issue #7924: get stuck training on multiple machines error URL: https://github.com/apache/incubator-mxnet/issues/7924#issuecomment-412286141 It maybe is because of that there is a extra firewall in CentOS except iptables, and you should have got it closed.

[GitHub] tqchen commented on a change in pull request #12047: [MXNET-779]Add DLPack Transformation API

tqchen commented on a change in pull request #12047: [MXNET-779]Add DLPack Transformation API URL: https://github.com/apache/incubator-mxnet/pull/12047#discussion_r209430981 ## File path: python/mxnet/_ctypes/ndarray.py ## @@ -31,21 +31,24 @@ class NDArrayBase(object):

[GitHub] tqchen commented on a change in pull request #12047: [MXNET-779]Add DLPack Transformation API

tqchen commented on a change in pull request #12047: [MXNET-779]Add DLPack Transformation API URL: https://github.com/apache/incubator-mxnet/pull/12047#discussion_r209430981 ## File path: python/mxnet/_ctypes/ndarray.py ## @@ -31,21 +31,24 @@ class NDArrayBase(object):

[GitHub] larroy commented on issue #12118: fix potential floating number overflow, enable float16

larroy commented on issue #12118: fix potential floating number overflow, enable float16 URL: https://github.com/apache/incubator-mxnet/pull/12118#issuecomment-412278531 ssize_t then? It’s not portable otherwise This is an

[GitHub] marcoabreu closed pull request #12125: CI scripts refinements. Separate Py2 and Py3 installs cripts. Fix perms.

marcoabreu closed pull request #12125: CI scripts refinements. Separate Py2 and Py3 installs cripts. Fix perms. URL: https://github.com/apache/incubator-mxnet/pull/12125 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: CI scripts refinements. Separate Py2 and Py3 installs cripts. Fix perms. (#12125)

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 1b60478 CI scripts refinements.

[GitHub] kppl commented on issue #1161: ImpportError: No module named skimage when running Neural-style example

kppl commented on issue #1161: ImpportError: No module named skimage when running Neural-style example URL: https://github.com/apache/incubator-mxnet/issues/1161#issuecomment-412275820 `from skimage import transform ` This

[GitHub] pengzhao-intel commented on issue #12134: why MKL and MKL-DNN can't be used simultaneously in ChooseBlas.cmake

pengzhao-intel commented on issue #12134: why MKL and MKL-DNN can't be used simultaneously in ChooseBlas.cmake URL: https://github.com/apache/incubator-mxnet/issues/12134#issuecomment-412275435 yes, it's a little trick and this is a piece of legacy code which we want to refine for a

[GitHub] YouRancestor opened a new pull request #12136: accept GPU data as input

YouRancestor opened a new pull request #12136: accept GPU data as input URL: https://github.com/apache/incubator-mxnet/pull/12136 ## Description ## Add a C Predict API interface MXPredSetInputGPU to accept input data on GPU.

[incubator-mxnet] branch master updated: fix cython nnvm include path (#12133)

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 9fd45b8 fix cython nnvm include

[GitHub] marcoabreu closed issue #12123: cython nnvm include path error

marcoabreu closed issue #12123: cython nnvm include path error URL: https://github.com/apache/incubator-mxnet/issues/12123 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] marcoabreu closed pull request #12133: fix cython nnvm include path

marcoabreu closed pull request #12133: fix cython nnvm include path URL: https://github.com/apache/incubator-mxnet/pull/12133 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] marcoabreu commented on issue #12134: why MKL and MKL-DNN can't be used simultaneously in ChooseBlas.cmake

marcoabreu commented on issue #12134: why MKL and MKL-DNN can't be used simultaneously in ChooseBlas.cmake URL: https://github.com/apache/incubator-mxnet/issues/12134#issuecomment-412273016 @lebeg This is an automated

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

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 98d8109 Bump the publish

[GitHub] squidszyd closed issue #12135: Writing own C++ Data Iter

squidszyd closed issue #12135: Writing own C++ Data Iter URL: https://github.com/apache/incubator-mxnet/issues/12135 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] squidszyd opened a new issue #12135: Writing own C++ Data Iter

squidszyd opened a new issue #12135: Writing own C++ Data Iter URL: https://github.com/apache/incubator-mxnet/issues/12135 https://github.com/apache/incubator-mxnet/blob/c44f16b0909d94c9beaf9c5fc0773855bbc91807/src/io/inst_vector.h#L147 I'm currently implementing a data iterator for

[GitHub] luchangli03 commented on issue #12134: why MKL and MKL-DNN can't be used simultaneously in ChooseBlas.cmake

luchangli03 commented on issue #12134: why MKL and MKL-DNN can't be used simultaneously in ChooseBlas.cmake URL: https://github.com/apache/incubator-mxnet/issues/12134#issuecomment-412268644 in the 3rdparty/mshadow/cmake/mshadow.cmake the codes are also if(MKL_FOUND)

[GitHub] luchangli03 opened a new issue #12134: why MKL and MKL-DNN can't be used simultaneously in ChooseBlas.cmake

luchangli03 opened a new issue #12134: why MKL and MKL-DNN can't be used simultaneously in ChooseBlas.cmake URL: https://github.com/apache/incubator-mxnet/issues/12134 if(MKL_FOUND) if(USE_MKLDNN) set(BLAS "open") else() set(BLAS "MKL") endif()

[GitHub] perdasilva commented on issue #12090: [MXNET-791][WIP] Pick with negative indices

perdasilva commented on issue #12090: [MXNET-791][WIP] Pick with negative indices URL: https://github.com/apache/incubator-mxnet/pull/12090#issuecomment-412266275 I believe that's all done now. Thanks for everything ^^ This

[GitHub] lupesko commented on a change in pull request #12102: site-wide social include

lupesko commented on a change in pull request #12102: site-wide social include URL: https://github.com/apache/incubator-mxnet/pull/12102#discussion_r209420814 ## File path: docs/community/contribute.md ## @@ -62,13 +62,37 @@ To join the MXNet slack channel send request to

[GitHub] petronny edited a comment on issue #12123: cython nnvm include path error

petronny edited a comment on issue #12123: cython nnvm include path error URL: https://github.com/apache/incubator-mxnet/issues/12123#issuecomment-412259848 Sorry, I'm afraid not because I have a pool network connection to apache JIRA... But I have created a PR on github, will it work?

[GitHub] petronny opened a new pull request #12133: fix cython nnvm include path

petronny opened a new pull request #12133: fix cython nnvm include path URL: https://github.com/apache/incubator-mxnet/pull/12133 ## Description ## fix cython nnvm include path ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your PR.

[GitHub] petronny commented on issue #12123: cython nnvm include path error

petronny commented on issue #12123: cython nnvm include path error URL: https://github.com/apache/incubator-mxnet/issues/12123#issuecomment-412259848 Sorry, I'm afraid not because I have a pool network connection to apache JIRA...

[GitHub] kaleidoscopical commented on issue #12001: SyncBatchNorm problems

kaleidoscopical commented on issue #12001: SyncBatchNorm problems URL: https://github.com/apache/incubator-mxnet/issues/12001#issuecomment-412258741 @safrooze I have tried both of them. While the `nd` version seems to perform similarly as the original `BatchNorm`, the `sym` version fails

[GitHub] wkcn commented on a change in pull request #12047: [MXNET-779]Add DLPack Transformation API

wkcn commented on a change in pull request #12047: [MXNET-779]Add DLPack Transformation API URL: https://github.com/apache/incubator-mxnet/pull/12047#discussion_r209418577 ## File path: src/c_api/c_api.cc ## @@ -500,6 +500,33 @@ int MXNDArrayGetData(NDArrayHandle handle,

[GitHub] wkcn commented on a change in pull request #12047: [MXNET-779]Add DLPack Transformation API

wkcn commented on a change in pull request #12047: [MXNET-779]Add DLPack Transformation API URL: https://github.com/apache/incubator-mxnet/pull/12047#discussion_r209418577 ## File path: src/c_api/c_api.cc ## @@ -500,6 +500,33 @@ int MXNDArrayGetData(NDArrayHandle handle,

[GitHub] apeforest commented on issue #11773: Update PyPI version number

apeforest commented on issue #11773: Update PyPI version number URL: https://github.com/apache/incubator-mxnet/pull/11773#issuecomment-412255848 @sandeep-krishnamurthy re-build. Please approve. Thanks! This is an automated

[GitHub] wkcn commented on a change in pull request #12047: [MXNET-779]Add DLPack Transformation API

wkcn commented on a change in pull request #12047: [MXNET-779]Add DLPack Transformation API URL: https://github.com/apache/incubator-mxnet/pull/12047#discussion_r209418365 ## File path: src/c_api/c_api.cc ## @@ -494,6 +494,57 @@ int MXNDArrayGetData(NDArrayHandle handle,

[GitHub] wkcn commented on a change in pull request #12047: [MXNET-779]Add DLPack Transformation API

wkcn commented on a change in pull request #12047: [MXNET-779]Add DLPack Transformation API URL: https://github.com/apache/incubator-mxnet/pull/12047#discussion_r209418176 ## File path: python/mxnet/ndarray/ndarray.py ## @@ -3851,3 +3898,117 @@ def histogram(a, bins=10,

[GitHub] wkcn commented on a change in pull request #12047: [MXNET-779]Add DLPack Transformation API

wkcn commented on a change in pull request #12047: [MXNET-779]Add DLPack Transformation API URL: https://github.com/apache/incubator-mxnet/pull/12047#discussion_r209416817 ## File path: include/mxnet/c_api.h ## @@ -737,6 +741,57 @@ MXNET_DLL int

[GitHub] jasonyu1996 opened a new issue #12132: [Feature Request] Implementation of L-BFGS optimizer

jasonyu1996 opened a new issue #12132: [Feature Request] Implementation of L-BFGS optimizer URL: https://github.com/apache/incubator-mxnet/issues/12132 Hi! Would the developers consider adding an implementation of limited memory BFGS to the library? PyTorch offers one here: