[GitHub] panovr opened a new issue #9527: Python Segmentation fault with commit afd82410165576dec7c673cd0438a1a9d39a9af6

2018-01-22 Thread GitBox
panovr opened a new issue #9527: Python Segmentation fault with commit afd82410165576dec7c673cd0438a1a9d39a9af6 URL: https://github.com/apache/incubator-mxnet/issues/9527 Note: Providing complete information in the most concise form is the best way to get help. This issue template serves

[GitHub] kmul00 commented on issue #9525: How to print loaded models ?

2018-01-22 Thread GitBox
kmul00 commented on issue #9525: How to print loaded models ? URL: https://github.com/apache/incubator-mxnet/issues/9525#issuecomment-359701199 Yeah, that solved the problem. Thank you so much @reminisce ! This is an

[GitHub] kmul00 closed issue #9525: How to print loaded models ?

2018-01-22 Thread GitBox
kmul00 closed issue #9525: How to print loaded models ? URL: https://github.com/apache/incubator-mxnet/issues/9525 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

[GitHub] szha closed pull request #9524: example/image-classification work with python3

2018-01-22 Thread GitBox
szha closed pull request #9524: example/image-classification work with python3 URL: https://github.com/apache/incubator-mxnet/pull/9524 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: example/image-classification work with python3 (#9524)

2018-01-22 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 fbbb4ab

[GitHub] jiang1st commented on issue #6968: Is mxnet symbol support conditional control flow?

2018-01-22 Thread GitBox
jiang1st commented on issue #6968: Is mxnet symbol support conditional control flow? URL: https://github.com/apache/incubator-mxnet/issues/6968#issuecomment-359685366 @Godricly Thanks. But moving my project from symbol to gluon might be a hard work.

[GitHub] parallelgithub opened a new issue #9526: The method reset() of DataIter does not work

2018-01-22 Thread GitBox
parallelgithub opened a new issue #9526: The method reset() of DataIter does not work URL: https://github.com/apache/incubator-mxnet/issues/9526 ## Description I have a question and find a bug about Scala on MXNet: (1) I didn't find out the implementation Scala codes of classes

[GitHub] reminisce commented on issue #9525: How to print loaded models ?

2018-01-22 Thread GitBox
reminisce commented on issue #9525: How to print loaded models ? URL: https://github.com/apache/incubator-mxnet/issues/9525#issuecomment-359694083 `_symbol` is a data member of `Module`. You can use `model._symbol` to get the symbol.

[GitHub] shitou112 commented on issue #9455: An error occurred while calculating the square of ndarray by using gpu context

2018-01-22 Thread GitBox
shitou112 commented on issue #9455: An error occurred while calculating the square of ndarray by using gpu context URL: https://github.com/apache/incubator-mxnet/issues/9455#issuecomment-359693124 Thank you a lot. I upgrade mxnet and the result is right.

[GitHub] kmul00 opened a new issue #9525: How to print loaded models (in mx.module format) ?

2018-01-22 Thread GitBox
kmul00 opened a new issue #9525: How to print loaded models (in mx.module format) ? URL: https://github.com/apache/incubator-mxnet/issues/9525 Hi, This may seem to be a very basic question, and creating an issue for it may not be the right approach to get it solved. If that's the

[GitHub] szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#discussion_r163153728 ## File path: python/mxnet/gluon/data/sampler.py ## @@ -136,3 +136,30 @@ def __len__(self):

[GitHub] szha commented on issue #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
szha commented on issue #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#issuecomment-359688753 @zhreshold thanks. @astonzhang shall we consider a different name for indexer, like the aforementioned "vocabulary"?

[GitHub] jiang1st commented on issue #6968: Is mxnet symbol support conditional control flow?

2018-01-22 Thread GitBox
jiang1st commented on issue #6968: Is mxnet symbol support conditional control flow? URL: https://github.com/apache/incubator-mxnet/issues/6968#issuecomment-359685366 @Godricly Thanks. But moving my project from symbol to gluon might a hard work.

[GitHub] zhreshold commented on issue #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
zhreshold commented on issue #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#issuecomment-359684522 I am more concerned with the name `Indexer` and the related behavior(manually extracting Indexer and synchronize between train/valid

[GitHub] marcoabreu commented on issue #9295: test_operator.test_laop_3 hangs

2018-01-22 Thread GitBox
marcoabreu commented on issue #9295: test_operator.test_laop_3 hangs URL: https://github.com/apache/incubator-mxnet/issues/9295#issuecomment-359684218 Sorry for the late response. I didn't see any hangs since the upgrade, but it would be good if we could figure out whether that specific

[GitHub] yzhliu commented on issue #9389: [SUBMODULE] Upgrade ps-lite to the latest version

2018-01-22 Thread GitBox
yzhliu commented on issue #9389: [SUBMODULE] Upgrade ps-lite to the latest version URL: https://github.com/apache/incubator-mxnet/pull/9389#issuecomment-359682511 I agree with @cjolivier01 that we should make incremental changes, while I prefer to merge it after a stable release.

[GitHub] yzhliu commented on issue #9389: [SUBMODULE] Upgrade ps-lite to the latest version

2018-01-22 Thread GitBox
yzhliu commented on issue #9389: [SUBMODULE] Upgrade ps-lite to the latest version URL: https://github.com/apache/incubator-mxnet/pull/9389#issuecomment-359681629 The changes are good to me as far as I know. But I still suggest **not** to include it as a part of v1.0.1. This PR does

[GitHub] yzhliu commented on issue #9389: [SUBMODULE] Upgrade ps-lite to the latest version

2018-01-22 Thread GitBox
yzhliu commented on issue #9389: [SUBMODULE] Upgrade ps-lite to the latest version URL: https://github.com/apache/incubator-mxnet/pull/9389#issuecomment-359681629 The changes are good to me as far as I know. But I still suggest **not** to include it as a part of v1.0.1. This PR does

[GitHub] cjolivier01 commented on issue #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc.

2018-01-22 Thread GitBox
cjolivier01 commented on issue #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc. URL: https://github.com/apache/incubator-mxnet/pull/8972#issuecomment-359680452 i think that will be good to make another PR. why do you think? where do you think you?d want

[GitHub] higumachan opened a new pull request #9524: example/image-classification work with python3

2018-01-22 Thread GitBox
higumachan opened a new pull request #9524: example/image-classification work with python3 URL: https://github.com/apache/incubator-mxnet/pull/9524 ## Description ## has_key is a method not defined in python3. I modified it in a form that works even with python 2. ## Checklist

[GitHub] pengzhao-intel commented on issue #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc.

2018-01-22 Thread GitBox
pengzhao-intel commented on issue #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc. URL: https://github.com/apache/incubator-mxnet/pull/8972#issuecomment-359678505 @TaoLv it's a very useful tool. @cjolivier01 do you think it's necessary to PR this script

[GitHub] TaoLv commented on issue #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc.

2018-01-22 Thread GitBox
TaoLv commented on issue #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc. URL: https://github.com/apache/incubator-mxnet/pull/8972#issuecomment-359677391 @cjolivier01 I have implemented a simple tool to parse the profile.json which is generated by mxnet.

[GitHub] anirudh2290 commented on issue #8000: Docs changes for launch.py usage

2018-01-22 Thread GitBox
anirudh2290 commented on issue #8000: Docs changes for launch.py usage URL: https://github.com/apache/incubator-mxnet/pull/8000#issuecomment-359675828 @eric-haibin-lin Done! This is an automated message from the Apache Git

[GitHub] pengzhao-intel commented on issue #8302: Refactor operators & MKLDNN

2018-01-22 Thread GitBox
pengzhao-intel commented on issue #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#issuecomment-359674818 @eric-haibin-lin Yes, I think the new MKL-DNN is OK for both inference and training. The full size of ImageNet training is under the

[GitHub] opringle commented on issue #9523: MXNet bucking custom weighted loss error

2018-01-22 Thread GitBox
opringle commented on issue #9523: MXNet bucking custom weighted loss error URL: https://github.com/apache/incubator-mxnet/issues/9523#issuecomment-359670748 I found the solution to this problem was to flatten the shape of the weights, labels and predictions before applying the custom

[GitHub] kice commented on issue #9438: MXNet crash while getting output of the network

2018-01-22 Thread GitBox
kice commented on issue #9438: MXNet crash while getting output of the network URL: https://github.com/apache/incubator-mxnet/issues/9438#issuecomment-359670723 I did not see any error on Windows when mxnet crashes. But I will reduce the model and try to run it. I will update the issue

[GitHub] szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#discussion_r163121885 ## File path: python/mxnet/gluon/data/sampler.py ## @@ -136,3 +136,30 @@ def __len__(self):

[GitHub] szha commented on issue #8027: Optional reshape of predictions in Perplexity metric

2018-01-22 Thread GitBox
szha commented on issue #8027: Optional reshape of predictions in Perplexity metric URL: https://github.com/apache/incubator-mxnet/pull/8027#issuecomment-359669294 Yes, I was looking into the metrics, and I didn't feel like having a separate perplexity metric was necessary in gluon. The

[GitHub] szha commented on issue #8027: Optional reshape of predictions in Perplexity metric

2018-01-22 Thread GitBox
szha commented on issue #8027: Optional reshape of predictions in Perplexity metric URL: https://github.com/apache/incubator-mxnet/pull/8027#issuecomment-359669294 Yes, I was looking into the metrics, and I didn't feel like having a separate perplexity metric was necessary in gluon. The

[GitHub] opringle opened a new issue #9523: MXNet bucking custom weighted loss error

2018-01-22 Thread GitBox
opringle opened a new issue #9523: MXNet bucking custom weighted loss error URL: https://github.com/apache/incubator-mxnet/issues/9523 ## Description When using a custom loss function which multiplies the loss by weight array, in a bucketing module, the code fails if I include a

[GitHub] reminisce commented on issue #8027: Optional reshape of predictions in Perplexity metric

2018-01-22 Thread GitBox
reminisce commented on issue #8027: Optional reshape of predictions in Perplexity metric URL: https://github.com/apache/incubator-mxnet/pull/8027#issuecomment-359667577 Sorry I didn't notice that I was tagged. @piiswrong Did you mean changing this line into using `pick`? I remember

[GitHub] anirudhacharya commented on issue #5290: libcblas.so.3: cannot open shared object file: No such file or directory

2018-01-22 Thread GitBox
anirudhacharya commented on issue #5290: libcblas.so.3: cannot open shared object file: No such file or directory URL: https://github.com/apache/incubator-mxnet/issues/5290#issuecomment-359659923 The AMI images of AWS does not have these packages. Installing them worked. sudo yum -y

[GitHub] anirudhacharya commented on issue #5290: libcblas.so.3: cannot open shared object file: No such file or directory

2018-01-22 Thread GitBox
anirudhacharya commented on issue #5290: libcblas.so.3: cannot open shared object file: No such file or directory URL: https://github.com/apache/incubator-mxnet/issues/5290#issuecomment-359657250 Also for the following packages, I am not able to find corresponding yum packages -

[GitHub] CodingCat commented on issue #9389: [SUBMODULE] Upgrade ps-lite to the latest version

2018-01-22 Thread GitBox
CodingCat commented on issue #9389: [SUBMODULE] Upgrade ps-lite to the latest version URL: https://github.com/apache/incubator-mxnet/pull/9389#issuecomment-359656699 ping @eric-haibin-lin This is an automated message from

[GitHub] anirudhacharya commented on issue #5290: libcblas.so.3: cannot open shared object file: No such file or directory

2018-01-22 Thread GitBox
anirudhacharya commented on issue #5290: libcblas.so.3: cannot open shared object file: No such file or directory URL: https://github.com/apache/incubator-mxnet/issues/5290#issuecomment-359655980 @szha $ yum provides libgfortran.so.3 Loaded plugins: priorities, update-motd,

[GitHub] szha commented on issue #5290: libcblas.so.3: cannot open shared object file: No such file or directory

2018-01-22 Thread GitBox
szha commented on issue #5290: libcblas.so.3: cannot open shared object file: No such file or directory URL: https://github.com/apache/incubator-mxnet/issues/5290#issuecomment-359653853 `yum` has a subcommand to help you find the file you need, `yum provides libgfortran.so.3`. The output

[GitHub] anirudhacharya commented on issue #5290: libcblas.so.3: cannot open shared object file: No such file or directory

2018-01-22 Thread GitBox
anirudhacharya commented on issue #5290: libcblas.so.3: cannot open shared object file: No such file or directory URL: https://github.com/apache/incubator-mxnet/issues/5290#issuecomment-359653380 @szha yum does not have gfortran just like libatlas-base-dev. $ sudo yum install gfortran

[GitHub] szha commented on issue #5290: libcblas.so.3: cannot open shared object file: No such file or directory

2018-01-22 Thread GitBox
szha commented on issue #5290: libcblas.so.3: cannot open shared object file: No such file or directory URL: https://github.com/apache/incubator-mxnet/issues/5290#issuecomment-359652905 @anirudhacharya for missing `libgfortran.so.3` you need gfortran package instead of atlas.

[GitHub] anirudhacharya commented on issue #5290: libcblas.so.3: cannot open shared object file: No such file or directory

2018-01-22 Thread GitBox
anirudhacharya commented on issue #5290: libcblas.so.3: cannot open shared object file: No such file or directory URL: https://github.com/apache/incubator-mxnet/issues/5290#issuecomment-359650854 Using OS - "Linux ip-172-31-7-81 4.9.76-3.78.amzn1.x86_64" and python 2.7. Tried - >>>

[GitHub] solin319 commented on issue #8107: "add warmup lr_scheduler" create a new pr

2018-01-22 Thread GitBox
solin319 commented on issue #8107: "add warmup lr_scheduler" create a new pr URL: https://github.com/apache/incubator-mxnet/pull/8107#issuecomment-359641487 @zhreshold @piiswrong @szha I combined warmup and multi step scheduler in #8005. Now, need I separate them?

[GitHub] szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#discussion_r163121885 ## File path: python/mxnet/gluon/data/sampler.py ## @@ -136,3 +136,30 @@ def __len__(self):

[GitHub] rahul003 commented on issue #9438: MXNet crash while getting output of the network

2018-01-22 Thread GitBox
rahul003 commented on issue #9438: MXNet crash while getting output of the network URL: https://github.com/apache/incubator-mxnet/issues/9438#issuecomment-359636498 Could you try decreasing the size of your model? Your code goes out of memory for me with a 11439MB GPU. Do you see

[GitHub] piiswrong commented on issue #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
piiswrong commented on issue #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#issuecomment-359636506 Then you would have problem when you want only the top 2000 tokens. Since this is not a very common use case, I think the current

[GitHub] piiswrong commented on a change in pull request #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
piiswrong commented on a change in pull request #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#discussion_r163119435 ## File path: python/mxnet/gluon/data/sampler.py ## @@ -136,3 +136,30 @@ def __len__(self):

[GitHub] szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#discussion_r163114151 ## File path: python/mxnet/gluon/data/sampler.py ## @@ -136,3 +136,30 @@ def __len__(self):

[GitHub] piiswrong commented on a change in pull request #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
piiswrong commented on a change in pull request #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#discussion_r163113390 ## File path: python/mxnet/gluon/data/sampler.py ## @@ -136,3 +136,30 @@ def __len__(self):

[GitHub] piiswrong commented on a change in pull request #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
piiswrong commented on a change in pull request #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#discussion_r163113390 ## File path: python/mxnet/gluon/data/sampler.py ## @@ -136,3 +136,30 @@ def __len__(self):

[GitHub] szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#discussion_r163112768 ## File path: python/mxnet/gluon/data/sampler.py ## @@ -136,3 +136,30 @@ def __len__(self):

[GitHub] szha commented on issue #8360: How to bind different input shape to executor in c++?

2018-01-22 Thread GitBox
szha commented on issue #8360: How to bind different input shape to executor in c++? URL: https://github.com/apache/incubator-mxnet/issues/8360#issuecomment-359624616 @apache/mxnet-committers: This issue has been inactive for the past 90 days. It has no label and needs triage. For

[GitHub] szha commented on issue #8387: test_operator_gpu.test_cuda_rtc failure - Jetson TX2 (GPU/CUDA)

2018-01-22 Thread GitBox
szha commented on issue #8387: test_operator_gpu.test_cuda_rtc failure - Jetson TX2 (GPU/CUDA) URL: https://github.com/apache/incubator-mxnet/issues/8387#issuecomment-359624601 @apache/mxnet-committers: This issue has been inactive for the past 90 days. It has no label and needs triage.

[GitHub] szha commented on issue #6693: [C++] MxPredCreate Error / operands shape mismatch

2018-01-22 Thread GitBox
szha commented on issue #6693: [C++] MxPredCreate Error / operands shape mismatch URL: https://github.com/apache/incubator-mxnet/issues/6693#issuecomment-359624597 @apache/mxnet-committers: This issue has been inactive for the past 90 days. It has no label and needs triage. For

[GitHub] szha commented on issue #8385: Save weights at any time between integral epochs

2018-01-22 Thread GitBox
szha commented on issue #8385: Save weights at any time between integral epochs URL: https://github.com/apache/incubator-mxnet/issues/8385#issuecomment-359624612 @apache/mxnet-committers: This issue has been inactive for the past 90 days. It has no label and needs triage. For

[GitHub] szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#discussion_r163109767 ## File path: python/mxnet/gluon/data/text.py ## @@ -0,0 +1,160 @@ +# Licensed to the Apache Software

[GitHub] szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#discussion_r163108754 ## File path: python/mxnet/gluon/data/text.py ## @@ -0,0 +1,160 @@ +# Licensed to the Apache Software

[GitHub] szha commented on issue #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
szha commented on issue #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#issuecomment-359615947 Indeed, I wasn't considering such case because it isn't good practice to index using anything other than training set. That said, providing

[GitHub] rahul003 commented on issue #9029: Replace slower mshadow op with kernel launch in kvstore_dist_server

2018-01-22 Thread GitBox
rahul003 commented on issue #9029: Replace slower mshadow op with kernel launch in kvstore_dist_server URL: https://github.com/apache/incubator-mxnet/pull/9029#issuecomment-359614271 Changes are complete. Verified on my end and the CI passed as well (I did some cleanup after it passed

[GitHub] safrooze commented on issue #9214: CTC example trains very slowly (~250 samples/sec)

2018-01-22 Thread GitBox
safrooze commented on issue #9214: CTC example trains very slowly (~250 samples/sec) URL: https://github.com/apache/incubator-mxnet/issues/9214#issuecomment-359612429 This example was re-written with multiprocess data generation.

[GitHub] safrooze closed issue #9214: CTC example trains very slowly (~250 samples/sec)

2018-01-22 Thread GitBox
safrooze closed issue #9214: CTC example trains very slowly (~250 samples/sec) URL: https://github.com/apache/incubator-mxnet/issues/9214 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] piiswrong commented on issue #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
piiswrong commented on issue #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#issuecomment-359611670 This Dataset.indexer design doesn't work for the use case where you want to combine (or take intersection of) the vocab of too datasets

[GitHub] szha closed issue #9490: Unable to install mxnet on MacOS .. no version seemingly compatible with Python3

2018-01-22 Thread GitBox
szha closed issue #9490: Unable to install mxnet on MacOS .. no version seemingly compatible with Python3 URL: https://github.com/apache/incubator-mxnet/issues/9490 This is an automated message from the Apache Git Service.

[GitHub] szha commented on issue #9490: Unable to install mxnet on MacOS .. no version seemingly compatible with Python3

2018-01-22 Thread GitBox
szha commented on issue #9490: Unable to install mxnet on MacOS .. no version seemingly compatible with Python3 URL: https://github.com/apache/incubator-mxnet/issues/9490#issuecomment-359611499 I can close this. @pabramowitsch let me know if you have more questions and I can re-open this

[GitHub] piiswrong commented on a change in pull request #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
piiswrong commented on a change in pull request #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#discussion_r163103170 ## File path: python/mxnet/gluon/data/text.py ## @@ -0,0 +1,160 @@ +# Licensed to the Apache

[GitHub] szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#discussion_r163103066 ## File path: python/mxnet/gluon/data/sampler.py ## @@ -136,3 +136,30 @@ def __len__(self):

[GitHub] szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#discussion_r163102705 ## File path: python/mxnet/gluon/data/text.py ## @@ -0,0 +1,160 @@ +# Licensed to the Apache Software

[GitHub] szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#discussion_r163102439 ## File path: python/mxnet/gluon/data/text.py ## @@ -0,0 +1,160 @@ +# Licensed to the Apache Software

[GitHub] szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
szha commented on a change in pull request #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#discussion_r163102130 ## File path: python/mxnet/gluon/data/text.py ## @@ -0,0 +1,160 @@ +# Licensed to the Apache Software

[GitHub] piiswrong commented on a change in pull request #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
piiswrong commented on a change in pull request #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#discussion_r163100367 ## File path: python/mxnet/gluon/data/sampler.py ## @@ -136,3 +136,30 @@ def __len__(self):

[GitHub] piiswrong commented on a change in pull request #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
piiswrong commented on a change in pull request #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#discussion_r163099680 ## File path: python/mxnet/gluon/data/text.py ## @@ -0,0 +1,160 @@ +# Licensed to the Apache

[GitHub] piiswrong commented on a change in pull request #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
piiswrong commented on a change in pull request #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#discussion_r163099562 ## File path: python/mxnet/gluon/data/text.py ## @@ -0,0 +1,160 @@ +# Licensed to the Apache

[GitHub] piiswrong commented on a change in pull request #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
piiswrong commented on a change in pull request #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#discussion_r163099217 ## File path: python/mxnet/gluon/data/text.py ## @@ -0,0 +1,160 @@ +# Licensed to the Apache

[GitHub] sxjscience closed issue #9506: Using the same seed will generate different results on different devices

2018-01-22 Thread GitBox
sxjscience closed issue #9506: Using the same seed will generate different results on different devices URL: https://github.com/apache/incubator-mxnet/issues/9506 This is an automated message from the Apache Git Service. To

[incubator-mxnet] branch v1.0.0 updated (9ef1969 -> 790ed30)

2018-01-22 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie pushed a change to branch v1.0.0 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 9ef1969 fix path separator (#9352) add 790ed30 Change the initialization for default constructor to

[GitHub] piiswrong closed pull request #9433: Backport gcc4 compiler fix to v1.0.0

2018-01-22 Thread GitBox
piiswrong closed pull request #9433: Backport gcc4 compiler fix to v1.0.0 URL: https://github.com/apache/incubator-mxnet/pull/9433 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] piiswrong commented on issue #9519: Reorganize the tutorials index page.

2018-01-22 Thread GitBox
piiswrong commented on issue #9519: Reorganize the tutorials index page. URL: https://github.com/apache/incubator-mxnet/pull/9519#issuecomment-359593471 I disagree. I think these changes are making things worse. For example, with old mxnet and module, you don't need mxnet.autograd.

[GitHub] ykim362 commented on issue #8532: mxnet-mkl (v0.12.0) crash when using (conda-installed) numpy with MKL

2018-01-22 Thread GitBox
ykim362 commented on issue #8532: mxnet-mkl (v0.12.0) crash when using (conda-installed) numpy with MKL URL: https://github.com/apache/incubator-mxnet/issues/8532#issuecomment-359591015 @fhieber Could you provide the conda version you used or some more detailed environment when you got

[GitHub] madjam commented on issue #9519: Reorganize the tutorials index page.

2018-01-22 Thread GitBox
madjam commented on issue #9519: Reorganize the tutorials index page. URL: https://github.com/apache/incubator-mxnet/pull/9519#issuecomment-359589536 This change is a improvement over what we have today. I believe this change walks the user through a series of concepts that are in roughly

[GitHub] mbaijal commented on issue #9500: [ReviewRequired] Adding License files for MXNet Dockerfiles.

2018-01-22 Thread GitBox
mbaijal commented on issue #9500: [ReviewRequired] Adding License files for MXNet Dockerfiles. URL: https://github.com/apache/incubator-mxnet/pull/9500#issuecomment-359589232 @eric-haibin-lin Thanks for pointing this out. I do need to remove the header from the file you mentioned.

[GitHub] zhreshold commented on a change in pull request #9514: Language Modeling Datasets and Sampler

2018-01-22 Thread GitBox
zhreshold commented on a change in pull request #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#discussion_r163080913 ## File path: python/mxnet/gluon/data/sampler.py ## @@ -136,3 +136,30 @@ def __len__(self):

[GitHub] kevinthesun commented on issue #8254: Fix softmax_cross_entropy

2018-01-22 Thread GitBox
kevinthesun commented on issue #8254: Fix softmax_cross_entropy URL: https://github.com/apache/incubator-mxnet/pull/8254#issuecomment-359581732 Already fixed. This is an automated message from the Apache Git Service. To

[GitHub] indhub commented on issue #9519: Reorganize the tutorials index page.

2018-01-22 Thread GitBox
indhub commented on issue #9519: Reorganize the tutorials index page. URL: https://github.com/apache/incubator-mxnet/pull/9519#issuecomment-359579635 @piiswrong While I agree there is a lot of work that needs to be done for the tutorials, we need to take incremental steps towards

[GitHub] zhreshold commented on a change in pull request #9457: Move get_images to test_utils.py, so it can be used from other tests. Fix documentation with recent im2rec changes.

2018-01-22 Thread GitBox
zhreshold commented on a change in pull request #9457: Move get_images to test_utils.py, so it can be used from other tests. Fix documentation with recent im2rec changes. URL: https://github.com/apache/incubator-mxnet/pull/9457#discussion_r163079037 ## File path:

[incubator-mxnet] branch master updated: Subscribe to changes (#9522)

2018-01-22 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 afd8241 Subscribe to changes (#9522)

[GitHub] eric-haibin-lin closed pull request #9522: Subscribe to changes

2018-01-22 Thread GitBox
eric-haibin-lin closed pull request #9522: Subscribe to changes URL: https://github.com/apache/incubator-mxnet/pull/9522 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] pracheer commented on a change in pull request #9460: Data-iterator tutorial made python3 compatible.

2018-01-22 Thread GitBox
pracheer commented on a change in pull request #9460: Data-iterator tutorial made python3 compatible. URL: https://github.com/apache/incubator-mxnet/pull/9460#discussion_r163076845 ## File path: docs/tutorials/basic/data.md ## @@ -122,17 +126,17 @@ class

[GitHub] larroy closed pull request #9035: [WIP] Build improvements, for users and for Jenkins

2018-01-22 Thread GitBox
larroy closed pull request #9035: [WIP] Build improvements, for users and for Jenkins URL: https://github.com/apache/incubator-mxnet/pull/9035 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] larroy commented on issue #8578: Add development instructions and update installation instructions

2018-01-22 Thread GitBox
larroy commented on issue #8578: Add development instructions and update installation instructions URL: https://github.com/apache/incubator-mxnet/pull/8578#issuecomment-359569929 @eric-haibin-lin I will have a look in the next couple of days, and either update or close. Thanks.

[GitHub] mbaijal commented on issue #9484: [Review Required] Fixing Licenses: Cleaning up the Top Level LICENSE file

2018-01-22 Thread GitBox
mbaijal commented on issue #9484: [Review Required] Fixing Licenses: Cleaning up the Top Level LICENSE file URL: https://github.com/apache/incubator-mxnet/pull/9484#issuecomment-359126417 @hen Can you please review this change.

[GitHub] mbaijal commented on issue #9500: [ReviewRequired] Adding License files for MXNet Dockerfiles.

2018-01-22 Thread GitBox
mbaijal commented on issue #9500: [ReviewRequired] Adding License files for MXNet Dockerfiles. URL: https://github.com/apache/incubator-mxnet/pull/9500#issuecomment-359568397 @eric-haibin-lin Can you please quickly review (it only adds readme files) and merge this PR

[GitHub] mbaijal commented on issue #9500: [ReviewRequired] Adding License files for MXNet Dockerfiles.

2018-01-22 Thread GitBox
mbaijal commented on issue #9500: [ReviewRequired] Adding License files for MXNet Dockerfiles. URL: https://github.com/apache/incubator-mxnet/pull/9500#issuecomment-359568397 @eric-haibin-lin Can you please merge this PR

[GitHub] szha closed pull request #9505: [WIP][Review Required] License Fixes based on Apache RAT Feedback Part 4

2018-01-22 Thread GitBox
szha closed pull request #9505: [WIP][Review Required] License Fixes based on Apache RAT Feedback Part 4 URL: https://github.com/apache/incubator-mxnet/pull/9505 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[incubator-mxnet] branch master updated: Adding License to MANIFEST.in files (#9505)

2018-01-22 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 bfe2097 Adding License to

[GitHub] indhub closed pull request #9504: [Review Required] License Fixes based on Apache RAT Feedback Part 3

2018-01-22 Thread GitBox
indhub closed pull request #9504: [Review Required] License Fixes based on Apache RAT Feedback Part 3 URL: https://github.com/apache/incubator-mxnet/pull/9504 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

[incubator-mxnet] branch master updated: Some more license adds (#9504)

2018-01-22 Thread indhub
This is an automated email from the ASF dual-hosted git repository. indhub 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 cb52708 Some more license adds

[GitHub] piiswrong closed pull request #9420: add use_global_stats in nn.BatchNorm

2018-01-22 Thread GitBox
piiswrong closed pull request #9420: add use_global_stats in nn.BatchNorm URL: https://github.com/apache/incubator-mxnet/pull/9420 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: add use_global_stats in nn.BatchNorm

2018-01-22 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie 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 dae6cda add use_global_stats in

[GitHub] pracheer commented on issue #9505: [WIP][Review Required] License Fixes based on Apache RAT Feedback Part 4

2018-01-22 Thread GitBox
pracheer commented on issue #9505: [WIP][Review Required] License Fixes based on Apache RAT Feedback Part 4 URL: https://github.com/apache/incubator-mxnet/pull/9505#issuecomment-359563144 The change looks good to me. Thanks @mbaijal !

[GitHub] piiswrong commented on issue #9519: Reorganize the tutorials index page.

2018-01-22 Thread GitBox
piiswrong commented on issue #9519: Reorganize the tutorials index page. URL: https://github.com/apache/incubator-mxnet/pull/9519#issuecomment-359559215 These changes where merged before I had a chance to look at them. Now that I see it, I think the original organization was better.

[GitHub] piiswrong commented on issue #9519: Reorganize the tutorials index page.

2018-01-22 Thread GitBox
piiswrong commented on issue #9519: Reorganize the tutorials index page. URL: https://github.com/apache/incubator-mxnet/pull/9519#issuecomment-359560385 If you actually read the tutorials carefully, you'll notice that although the gluon tutorials and old mxnet tutorials overlap on topic,

[incubator-mxnet-site] branch asf-site updated: fixed getting_started install.html to point to the latest version of install by redirection (#50)

2018-01-22 Thread nswamy
This is an automated email from the ASF dual-hosted git repository. nswamy 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 24026b9 fixed

[GitHub] nswamy closed pull request #50: fixed getting_started install.html to point to the latest version of ?

2018-01-22 Thread GitBox
nswamy closed pull request #50: fixed getting_started install.html to point to the latest version of ? URL: https://github.com/apache/incubator-mxnet-site/pull/50 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

  1   2   >