[GitHub] [incubator-mxnet] hzfan commented on a change in pull request #18197: [Numpy] Add qr backward part 2 for wide matrices with m < n

2020-05-15 Thread GitBox
hzfan commented on a change in pull request #18197: URL: https://github.com/apache/incubator-mxnet/pull/18197#discussion_r425852127 ## File path: src/operator/numpy/linalg/np_qr-inl.h ## @@ -542,36 +591,119 @@ void QrBackwardImpl(const TBlob& grad_a, const

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

2020-05-15 Thread aaronmarkham
This is an automated email from the ASF dual-hosted git repository. aaronmarkham 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 4327a7f Bump the

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18197: [Numpy] Add qr backward part 2 for wide matrices with m < n

2020-05-15 Thread GitBox
mxnet-bot commented on pull request #18197: URL: https://github.com/apache/incubator-mxnet/pull/18197#issuecomment-629227942 Jenkins CI successfully triggered : [centos-cpu, unix-cpu] This is an automated message from the

[GitHub] [incubator-mxnet] D-Roberts commented on pull request #18197: [Numpy] Add qr backward part 2 for wide matrices with m < n

2020-05-15 Thread GitBox
D-Roberts commented on pull request #18197: URL: https://github.com/apache/incubator-mxnet/pull/18197#issuecomment-629227877 @mxnet-bot run ci [centos-cpu, unix-cpu] This is an automated message from the Apache Git Service.

[GitHub] [incubator-mxnet] wkcn commented on a change in pull request #18325: Change SGD with momentum to include momentum correction by default

2020-05-15 Thread GitBox
wkcn commented on a change in pull request #18325: URL: https://github.com/apache/incubator-mxnet/pull/18325#discussion_r425780192 ## File path: python/mxnet/optimizer/lars.py ## @@ -252,6 +253,13 @@ def fused_step(self, indices, weights, grads, states): wd =

[GitHub] [incubator-mxnet] hzfan commented on a change in pull request #17749: Fix races in block scope

2020-05-15 Thread GitBox
hzfan commented on a change in pull request #17749: URL: https://github.com/apache/incubator-mxnet/pull/17749#discussion_r425799124 ## File path: python/mxnet/gluon/block.py ## @@ -91,22 +92,22 @@ def create(prefix, params, hint): def __enter__(self): if

[incubator-mxnet] branch master updated (fec534a -> 3e676fc)

2020-05-15 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from fec534a Fix missing MKLDNN headers (#18310) add 3e676fc Fix memory leaks in Gluon (#18328) No new

[GitHub] [incubator-mxnet] leezu commented on pull request #18328: Fix memory leaks in Gluon

2020-05-15 Thread GitBox
leezu commented on pull request #18328: URL: https://github.com/apache/incubator-mxnet/pull/18328#issuecomment-629372543 @ciyongch should we backport this to 1.7? This is an automated message from the Apache Git Service. To

[GitHub] [incubator-mxnet] ChaiBapchya edited a comment on issue #17980: When compiled with MKL, fully_connected calls DNNL while dot and batch_dot call MKL

2020-05-15 Thread GitBox
ChaiBapchya edited a comment on issue #17980: URL: https://github.com/apache/incubator-mxnet/issues/17980#issuecomment-629385677 > Tested with MXNet [cfb474b](https://github.com/apache/incubator-mxnet/commit/cfb474ba743d5ea85161bf19875488f4cb409d3c). Compiled with mostly-default cmake

[GitHub] [incubator-mxnet] JonTanS closed issue #16822: Trouble building Mxnet 1.6.x

2020-05-15 Thread GitBox
JonTanS closed issue #16822: URL: https://github.com/apache/incubator-mxnet/issues/16822 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [incubator-mxnet] John1231983 opened a new issue #18333: How to implement cross entropy for binary segmentation using symbol only?

2020-05-15 Thread GitBox
John1231983 opened a new issue #18333: URL: https://github.com/apache/incubator-mxnet/issues/18333 Given a softmax classified feature ff (`Bx2xHxW`). And a target label size of `Bx1xHxW` . I want to implement cross entropy loss using symbol only. This is my implementation ``` #

[GitHub] [incubator-mxnet] szha closed issue #18334: test_numpy_ndarray.py::test_np_ndarray_boolean_indexing

2020-05-15 Thread GitBox
szha closed issue #18334: URL: https://github.com/apache/incubator-mxnet/issues/18334 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[incubator-mxnet] branch master updated (fec534a -> 3e676fc)

2020-05-15 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from fec534a Fix missing MKLDNN headers (#18310) add 3e676fc Fix memory leaks in Gluon (#18328) No new

[GitHub] [incubator-mxnet] ptrendx commented on a change in pull request #18325: Change SGD with momentum to include momentum correction by default

2020-05-15 Thread GitBox
ptrendx commented on a change in pull request #18325: URL: https://github.com/apache/incubator-mxnet/pull/18325#discussion_r425969856 ## File path: python/mxnet/optimizer/lars.py ## @@ -252,6 +253,13 @@ def fused_step(self, indices, weights, grads, states): wd

[GitHub] [incubator-mxnet] ChaiBapchya edited a comment on issue #17980: When compiled with MKL, fully_connected calls DNNL while dot and batch_dot call MKL

2020-05-15 Thread GitBox
ChaiBapchya edited a comment on issue #17980: URL: https://github.com/apache/incubator-mxnet/issues/17980#issuecomment-623803913 > In case somebody finds this issue and wants their optimized build, here is a different workaround that removes the need for `LD_PRELOAD`. Just do this before

[GitHub] [incubator-mxnet] kpu commented on issue #17980: When compiled with MKL, fully_connected calls DNNL while dot and batch_dot call MKL

2020-05-15 Thread GitBox
kpu commented on issue #17980: URL: https://github.com/apache/incubator-mxnet/issues/17980#issuecomment-629400657 `dot` hasn't been taken over by MKLDNN = DNNL = oneAPI yet, so you shouldn't expect to see a difference there because it never calls MKLDNN's GEMM anyway. FullyConnected has

[GitHub] [incubator-mxnet] D-Roberts commented on a change in pull request #18197: [Numpy] Add qr backward part 2 for wide matrices with m < n

2020-05-15 Thread GitBox
D-Roberts commented on a change in pull request #18197: URL: https://github.com/apache/incubator-mxnet/pull/18197#discussion_r425903506 ## File path: src/operator/numpy/linalg/np_qr-inl.h ## @@ -542,36 +591,119 @@ void QrBackwardImpl(const TBlob& grad_a,

[GitHub] [incubator-mxnet] szha opened a new issue #18334: test_numpy_ndarray.py::test_np_ndarray_boolean_indexing

2020-05-15 Thread GitBox
szha opened a new issue #18334: URL: https://github.com/apache/incubator-mxnet/issues/18334 ## Description test_numpy_ndarray.py::test_np_ndarray_boolean_indexing ## Occurrences

[GitHub] [incubator-mxnet] leezu merged pull request #18328: Fix memory leaks in Gluon

2020-05-15 Thread GitBox
leezu merged pull request #18328: URL: https://github.com/apache/incubator-mxnet/pull/18328 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #17980: When compiled with MKL, fully_connected calls DNNL while dot and batch_dot call MKL

2020-05-15 Thread GitBox
ChaiBapchya commented on issue #17980: URL: https://github.com/apache/incubator-mxnet/issues/17980#issuecomment-629385677 > Tested with MXNet [cfb474b](https://github.com/apache/incubator-mxnet/commit/cfb474ba743d5ea85161bf19875488f4cb409d3c). Compiled with mostly-default cmake settings:

[GitHub] [incubator-mxnet] ChaiBapchya edited a comment on issue #17980: When compiled with MKL, fully_connected calls DNNL while dot and batch_dot call MKL

2020-05-15 Thread GitBox
ChaiBapchya edited a comment on issue #17980: URL: https://github.com/apache/incubator-mxnet/issues/17980#issuecomment-629385677 > Tested with MXNet [cfb474b](https://github.com/apache/incubator-mxnet/commit/cfb474ba743d5ea85161bf19875488f4cb409d3c). Compiled with mostly-default cmake

[GitHub] [incubator-mxnet] marcoabreu commented on pull request #18252: [CI] run operator tests with naive engine

2020-05-15 Thread GitBox
marcoabreu commented on pull request #18252: URL: https://github.com/apache/incubator-mxnet/pull/18252#issuecomment-629411739 What's our strategy to test multi threaded engine? This is an automated message from the Apache

[GitHub] [incubator-mxnet] ChaiBapchya closed pull request #18130: Set cache_intermediate to True

2020-05-15 Thread GitBox
ChaiBapchya closed pull request #18130: URL: https://github.com/apache/incubator-mxnet/pull/18130 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [incubator-mxnet] ChaiBapchya commented on pull request #18326: [R] Fix incorrect copyto usage & incorrect website title for Symbol API in R

2020-05-15 Thread GitBox
ChaiBapchya commented on pull request #18326: URL: https://github.com/apache/incubator-mxnet/pull/18326#issuecomment-629442039 @mxnet-label-bot add [pr-awaiting-review] This is an automated message from the Apache Git

[GitHub] [incubator-mxnet] ChaiBapchya commented on pull request #18130: Set cache_intermediate to True

2020-05-15 Thread GitBox
ChaiBapchya commented on pull request #18130: URL: https://github.com/apache/incubator-mxnet/pull/18130#issuecomment-629442396 Closing since this one is useful for local development & not on CI. This is an automated message

[GitHub] [incubator-mxnet-ci] mseth10 commented on a change in pull request #25: Serverless implementation for jenkins pipeline monitor lambda

2020-05-15 Thread GitBox
mseth10 commented on a change in pull request #25: URL: https://github.com/apache/incubator-mxnet-ci/pull/25#discussion_r425988017 ## File path: services/jenkins-pipeline-monitor/handler.py ## @@ -0,0 +1,134 @@ +import os +import boto3 +import json +import logging +import

[GitHub] [incubator-mxnet-ci] mseth10 commented on a change in pull request #25: Serverless implementation for jenkins pipeline monitor lambda

2020-05-15 Thread GitBox
mseth10 commented on a change in pull request #25: URL: https://github.com/apache/incubator-mxnet-ci/pull/25#discussion_r425988017 ## File path: services/jenkins-pipeline-monitor/handler.py ## @@ -0,0 +1,134 @@ +import os +import boto3 +import json +import logging +import

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

2020-05-15 Thread aaronmarkham
This is an automated email from the ASF dual-hosted git repository. aaronmarkham 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 e3cceb2 Bump the

[GitHub] [incubator-mxnet] sandeep-krishnamurthy commented on pull request #18288: Website global search feature

2020-05-15 Thread GitBox
sandeep-krishnamurthy commented on pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#issuecomment-629425112 @mxnet-bot run ci [unix-cpu] This is an automated message from the Apache Git Service.

[GitHub] [incubator-mxnet] sandeep-krishnamurthy commented on a change in pull request #18288: Website global search feature

2020-05-15 Thread GitBox
sandeep-krishnamurthy commented on a change in pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#discussion_r425990851 ## File path: docs/static_site/src/assets/js/globalSearch.js ## @@ -0,0 +1,152 @@ +/*! + * Licensed to the Apache Software

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18288: Website global search feature

2020-05-15 Thread GitBox
mxnet-bot commented on pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#issuecomment-629425161 Jenkins CI successfully triggered : [unix-cpu] This is an automated message from the Apache Git

[GitHub] [incubator-mxnet] ChaiBapchya commented on pull request #18331: Remove test metric perf

2020-05-15 Thread GitBox
ChaiBapchya commented on pull request #18331: URL: https://github.com/apache/incubator-mxnet/pull/18331#issuecomment-629444500 Quick history check reveals this file was added by @safrooze in #9705 This is an automated

[GitHub] [incubator-mxnet] ChaiBapchya edited a comment on pull request #18331: Remove test metric perf

2020-05-15 Thread GitBox
ChaiBapchya edited a comment on pull request #18331: URL: https://github.com/apache/incubator-mxnet/pull/18331#issuecomment-629444500 Quick history check reveals this file was added by @safrooze in #9705 Can we move this to nightly since that was planned? or do we get rid of it

[GitHub] [incubator-mxnet] ChaiBapchya edited a comment on pull request #18331: Remove test metric perf

2020-05-15 Thread GitBox
ChaiBapchya edited a comment on pull request #18331: URL: https://github.com/apache/incubator-mxnet/pull/18331#issuecomment-629444500 Quick history check reveals this file was added by @safrooze in #9705 Can we move this to nightly since that was planned? or do we get rid of it

[GitHub] [incubator-mxnet] ys2843 edited a comment on pull request #18288: Website global search feature

2020-05-15 Thread GitBox
ys2843 edited a comment on pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#issuecomment-628990327 > Looks good. > > * Should we use offsite js files or host locally? > * Should the API key be in a js file are can it be in a config file so we keep

[GitHub] [incubator-mxnet] leezu commented on pull request #18331: Remove test metric perf

2020-05-15 Thread GitBox
leezu commented on pull request #18331: URL: https://github.com/apache/incubator-mxnet/pull/18331#issuecomment-629477586 I think we should integrate it in the performance benchmark. Currently there is not mechanism to monitor the results

[GitHub] [incubator-mxnet] leezu opened a new pull request #18335: Skip test_metric_performance on MKL builds

2020-05-15 Thread GitBox
leezu opened a new pull request #18335: URL: https://github.com/apache/incubator-mxnet/pull/18335 https://github.com/apache/incubator-mxnet/issues/18244 metrics now use mxnet operators Fixes https://github.com/apache/incubator-mxnet/issues/18330

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18335: Skip test_metric_performance on MKL builds

2020-05-15 Thread GitBox
mxnet-bot commented on pull request #18335: URL: https://github.com/apache/incubator-mxnet/pull/18335#issuecomment-629504113 Hey @leezu , Thanks for submitting the PR All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following

[GitHub] [incubator-mxnet] leezu commented on pull request #18284: Fix parse operator attributes in new FFI

2020-05-15 Thread GitBox
leezu commented on pull request #18284: URL: https://github.com/apache/incubator-mxnet/pull/18284#issuecomment-629428687 @mxnet-bot run ci [unix-cpu] This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18284: Fix parse operator attributes in new FFI

2020-05-15 Thread GitBox
mxnet-bot commented on pull request #18284: URL: https://github.com/apache/incubator-mxnet/pull/18284#issuecomment-629428729 Jenkins CI successfully triggered : [unix-cpu] This is an automated message from the Apache Git

[GitHub] [incubator-mxnet] ChaiBapchya commented on pull request #18324: [OpPerf] Add example of using opperf with internal op locally

2020-05-15 Thread GitBox
ChaiBapchya commented on pull request #18324: URL: https://github.com/apache/incubator-mxnet/pull/18324#issuecomment-629445905 @mxnet-bot run ci [unix-cpu] This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-mxnet] ChaiBapchya edited a comment on issue #18330: test_metric_performance

2020-05-15 Thread GitBox
ChaiBapchya edited a comment on issue #18330: URL: https://github.com/apache/incubator-mxnet/issues/18330#issuecomment-629044532 http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Funix-cpu/detail/PR-18326/1/pipeline

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18324: [OpPerf] Add example of using opperf with internal op locally

2020-05-15 Thread GitBox
mxnet-bot commented on pull request #18324: URL: https://github.com/apache/incubator-mxnet/pull/18324#issuecomment-629445934 Jenkins CI successfully triggered : [unix-cpu] This is an automated message from the Apache Git

[GitHub] [incubator-mxnet-ci] ChaiBapchya commented on a change in pull request #25: Serverless implementation for jenkins pipeline monitor lambda

2020-05-15 Thread GitBox
ChaiBapchya commented on a change in pull request #25: URL: https://github.com/apache/incubator-mxnet-ci/pull/25#discussion_r426024021 ## File path: services/jenkins-pipeline-monitor/handler.py ## @@ -0,0 +1,134 @@ +import os +import boto3 +import json +import logging +import

[GitHub] [incubator-mxnet-ci] ChaiBapchya commented on a change in pull request #25: Serverless implementation for jenkins pipeline monitor lambda

2020-05-15 Thread GitBox
ChaiBapchya commented on a change in pull request #25: URL: https://github.com/apache/incubator-mxnet-ci/pull/25#discussion_r425978610 ## File path: services/jenkins-pipeline-monitor/handler.py ## @@ -0,0 +1,134 @@ +import os +import boto3 +import json +import logging +import

[GitHub] [incubator-mxnet] ys2843 commented on pull request #18288: Website global search feature

2020-05-15 Thread GitBox
ys2843 commented on pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#issuecomment-629441115 > Very nice. Thank you @ys2843 This will be very helpful for the community. > Please update the release manager checklist for changing the defaults. > > Should

[GitHub] [incubator-mxnet] zhreshold commented on pull request #18325: Change SGD with momentum to include momentum correction by default

2020-05-15 Thread GitBox
zhreshold commented on pull request #18325: URL: https://github.com/apache/incubator-mxnet/pull/18325#issuecomment-629504587 @szhengac Can you provide feedback on this? This is an automated message from the Apache Git

[GitHub] [incubator-mxnet-ci] ChaiBapchya commented on a change in pull request #25: Serverless implementation for jenkins pipeline monitor lambda

2020-05-15 Thread GitBox
ChaiBapchya commented on a change in pull request #25: URL: https://github.com/apache/incubator-mxnet-ci/pull/25#discussion_r425978610 ## File path: services/jenkins-pipeline-monitor/handler.py ## @@ -0,0 +1,134 @@ +import os +import boto3 +import json +import logging +import

[GitHub] [incubator-mxnet] sandeep-krishnamurthy commented on pull request #18288: Website global search feature

2020-05-15 Thread GitBox
sandeep-krishnamurthy commented on pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#issuecomment-629426892 @szha / @leezu - FYI. This is an automated message from the Apache Git Service. To

[GitHub] [incubator-mxnet] ys2843 edited a comment on pull request #18288: Website global search feature

2020-05-15 Thread GitBox
ys2843 edited a comment on pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#issuecomment-629441115 > Very nice. Thank you @ys2843 This will be very helpful for the community. > Please update the release manager checklist for changing the defaults. > >

[GitHub] [incubator-mxnet] ys2843 edited a comment on pull request #18288: Website global search feature

2020-05-15 Thread GitBox
ys2843 edited a comment on pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#issuecomment-628990327 > Looks good. > > * Should we use offsite js files or host locally? > * Should the API key be in a js file are can it be in a config file so we keep

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #17980: When compiled with MKL, fully_connected calls DNNL while dot and batch_dot call MKL

2020-05-15 Thread GitBox
ChaiBapchya commented on issue #17980: URL: https://github.com/apache/incubator-mxnet/issues/17980#issuecomment-629477511 Yes logs agree with these statements. But the perf difference isn't visible via opperf. My bad, lhs, rhs wrongly interpreted. But that's for dot,batch_dot. FC

[GitHub] [incubator-mxnet] bill10 opened a new issue #18336: topk operation is very memory intensive

2020-05-15 Thread GitBox
bill10 opened a new issue #18336: URL: https://github.com/apache/incubator-mxnet/issues/18336 ## Description I am not sure if this is a bug, but the topk operation has a surprisingly high memory cost compared to, e.g., pytorch. ### Error Message This resulted in out-of-memory

[GitHub] [incubator-mxnet-ci] mseth10 commented on a change in pull request #25: Serverless implementation for jenkins pipeline monitor lambda

2020-05-15 Thread GitBox
mseth10 commented on a change in pull request #25: URL: https://github.com/apache/incubator-mxnet-ci/pull/25#discussion_r425988017 ## File path: services/jenkins-pipeline-monitor/handler.py ## @@ -0,0 +1,134 @@ +import os +import boto3 +import json +import logging +import

[GitHub] [incubator-mxnet] leezu merged pull request #18284: Fix parse operator attributes in new FFI

2020-05-15 Thread GitBox
leezu merged pull request #18284: URL: https://github.com/apache/incubator-mxnet/pull/18284 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [incubator-mxnet] leezu closed issue #18004: Wrong result when using new numpy ffi in deferred compute

2020-05-15 Thread GitBox
leezu closed issue #18004: URL: https://github.com/apache/incubator-mxnet/issues/18004 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18315: Fix coredumps

2020-05-15 Thread GitBox
mxnet-bot commented on pull request #18315: URL: https://github.com/apache/incubator-mxnet/pull/18315#issuecomment-629534560 Jenkins CI successfully triggered : [centos-cpu, unix-cpu] This is an automated message from the

[GitHub] [incubator-mxnet] leezu commented on pull request #18315: Fix coredumps

2020-05-15 Thread GitBox
leezu commented on pull request #18315: URL: https://github.com/apache/incubator-mxnet/pull/18315#issuecomment-629534528 @mxnet-bot run ci [unix-cpu, centos-cpu] This is an automated message from the Apache Git Service. To

[GitHub] [incubator-mxnet] szha commented on pull request #18252: [CI] run operator tests with naive engine

2020-05-15 Thread GitBox
szha commented on pull request #18252: URL: https://github.com/apache/incubator-mxnet/pull/18252#issuecomment-629545644 @marcoabreu all but the operator unit tests still run on threaded engine. This is an automated message

[GitHub] [incubator-mxnet] waytrue17 commented on pull request #18339: update dockerfile for jetson

2020-05-15 Thread GitBox
waytrue17 commented on pull request #18339: URL: https://github.com/apache/incubator-mxnet/pull/18339#issuecomment-629563544 @ciyongch This is an automated message from the Apache Git Service. To respond to the message,

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

2020-05-15 Thread aaronmarkham
This is an automated email from the ASF dual-hosted git repository. aaronmarkham 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 4bdada6 Bump the

[GitHub] [incubator-mxnet] connorgoggins commented on pull request #18340: [Website 2.0] Artifact URL Adjustment

2020-05-15 Thread GitBox
connorgoggins commented on pull request #18340: URL: https://github.com/apache/incubator-mxnet/pull/18340#issuecomment-629583448 @mxnet-label-bot add [pr-awaiting-review] This is an automated message from the Apache Git

[GitHub] [incubator-mxnet] connorgoggins commented on pull request #18340: [Website 2.0] Artifact URL Adjustment

2020-05-15 Thread GitBox
connorgoggins commented on pull request #18340: URL: https://github.com/apache/incubator-mxnet/pull/18340#issuecomment-629583487 @mxnet-label-bot add [Website] This is an automated message from the Apache Git Service. To

[incubator-mxnet] branch master updated (3e676fc -> 37280e4)

2020-05-15 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 3e676fc Fix memory leaks in Gluon (#18328) add 37280e4 Fix deferred compute mode for operators using

[incubator-mxnet] branch master updated: Fix deferred compute mode for operators using new FFI (#18284)

2020-05-15 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen 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 37280e4 Fix deferred compute mode for

[GitHub] [incubator-mxnet] szha commented on issue #18244: unix-cpu MKL/MKL-DNN Test Time

2020-05-15 Thread GitBox
szha commented on issue #18244: URL: https://github.com/apache/incubator-mxnet/issues/18244#issuecomment-629566327 Any update on this? This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18337: fix native cd builds

2020-05-15 Thread GitBox
mxnet-bot commented on pull request #18337: URL: https://github.com/apache/incubator-mxnet/pull/18337#issuecomment-629530814 Hey @mseth10 , Thanks for submitting the PR All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following

[GitHub] [incubator-mxnet] mseth10 opened a new pull request #18337: fix native cd builds

2020-05-15 Thread GitBox
mseth10 opened a new pull request #18337: URL: https://github.com/apache/incubator-mxnet/pull/18337 ## Description ## Fix CD builds for 'native' flavor This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18288: Website global search feature

2020-05-15 Thread GitBox
mxnet-bot commented on pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#issuecomment-629557848 Jenkins CI successfully triggered : [unix-cpu] This is an automated message from the Apache Git

[GitHub] [incubator-mxnet] ys2843 commented on pull request #18288: Website global search feature

2020-05-15 Thread GitBox
ys2843 commented on pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#issuecomment-629557829 @mxnet-bot run ci [unix-cpu] This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-mxnet] leezu commented on pull request #18315: Fix coredumps

2020-05-15 Thread GitBox
leezu commented on pull request #18315: URL: https://github.com/apache/incubator-mxnet/pull/18315#issuecomment-629573080 @mxnet-bot run ci [unix-cpu] This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18315: Fix coredumps

2020-05-15 Thread GitBox
mxnet-bot commented on pull request #18315: URL: https://github.com/apache/incubator-mxnet/pull/18315#issuecomment-629573096 Jenkins CI successfully triggered : [unix-cpu] This is an automated message from the Apache Git

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18340: [Website 2.0] Artifact URL Adjustment

2020-05-15 Thread GitBox
mxnet-bot commented on pull request #18340: URL: https://github.com/apache/incubator-mxnet/pull/18340#issuecomment-629583305 Hey @connorgoggins , Thanks for submitting the PR All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the

[GitHub] [incubator-mxnet] connorgoggins opened a new pull request #18340: [Website 2.0] Artifact URL Adjustment

2020-05-15 Thread GitBox
connorgoggins opened a new pull request #18340: URL: https://github.com/apache/incubator-mxnet/pull/18340 ## Description ## This PR officially transfers ownership of code version control and compressed artifact storage for the static artifacts for the MXNet website from Connor Goggins

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #17980: When compiled with MKL, fully_connected calls DNNL while dot and batch_dot call MKL

2020-05-15 Thread GitBox
ChaiBapchya commented on issue #17980: URL: https://github.com/apache/incubator-mxnet/issues/17980#issuecomment-629526785 Can confirm that this issue is specific to AVX512 kernels. Tried this on c5.xl $ lscpu ``` Intel(R) Xeon(R) Platinum 8124M CPU @ 3.00GHz Flags:

[GitHub] [incubator-mxnet] szha commented on pull request #18335: Skip test_metric_performance on MKL builds

2020-05-15 Thread GitBox
szha commented on pull request #18335: URL: https://github.com/apache/incubator-mxnet/pull/18335#issuecomment-629552229 I'm moving this to benchmark scripts in #18252 This is an automated message from the Apache Git Service.

[GitHub] [incubator-mxnet] ChaiBapchya opened a new pull request #18338: [DO NOT MERGE] Use mkl cmake flag to force DNNL to delegate FC op to MKL

2020-05-15 Thread GitBox
ChaiBapchya opened a new pull request #18338: URL: https://github.com/apache/incubator-mxnet/pull/18338 ## Description ## Currently, for GEMM ops [DNNL delegates that op to MKL] However, for FC DNNL doesn't. Workaround : export USE_MKL & pass location of MKL installation. This

[GitHub] [incubator-mxnet] ChaiBapchya commented on pull request #18338: [DO NOT MERGE] Use mkl cmake flag to force DNNL to delegate FC op to MKL

2020-05-15 Thread GitBox
ChaiBapchya commented on pull request #18338: URL: https://github.com/apache/incubator-mxnet/pull/18338#issuecomment-629557360 Related issue : https://github.com/apache/incubator-mxnet/issues/17980 This is an automated

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18338: [DO NOT MERGE] Use mkl cmake flag to force DNNL to delegate FC op to MKL

2020-05-15 Thread GitBox
mxnet-bot commented on pull request #18338: URL: https://github.com/apache/incubator-mxnet/pull/18338#issuecomment-629557180 Hey @ChaiBapchya , Thanks for submitting the PR All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the

[GitHub] [incubator-mxnet] ChaiBapchya commented on pull request #18326: [R] Fix incorrect copyto usage & incorrect website title for Symbol API in R

2020-05-15 Thread GitBox
ChaiBapchya commented on pull request #18326: URL: https://github.com/apache/incubator-mxnet/pull/18326#issuecomment-629564086 @leezu @access2rohit plz review/merge This is an automated message from the Apache Git Service.

[GitHub] [incubator-mxnet] aaraujom commented on issue #17980: When compiled with MKL, fully_connected calls DNNL while dot and batch_dot call MKL

2020-05-15 Thread GitBox
aaraujom commented on issue #17980: URL: https://github.com/apache/incubator-mxnet/issues/17980#issuecomment-629542494 Hi thanks for checking. Let me try to reproduce this in my end and fix it. This is an automated message

[GitHub] [incubator-mxnet] szha commented on pull request #18331: Remove test metric perf

2020-05-15 Thread GitBox
szha commented on pull request #18331: URL: https://github.com/apache/incubator-mxnet/pull/18331#issuecomment-629552076 I'm moving it to benchmark scripts in #18252 This is an automated message from the Apache Git Service.

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18339: update dockerfile for jetson

2020-05-15 Thread GitBox
mxnet-bot commented on pull request #18339: URL: https://github.com/apache/incubator-mxnet/pull/18339#issuecomment-629562168 Hey @waytrue17 , Thanks for submitting the PR All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following

[GitHub] [incubator-mxnet] waytrue17 opened a new pull request #18339: update dockerfile for jetson

2020-05-15 Thread GitBox
waytrue17 opened a new pull request #18339: URL: https://github.com/apache/incubator-mxnet/pull/18339 ## Description ## Trying to fix #18311 on v1.7.x by copying Dockerfile.build.jetson from master to v1.7.x ### Essentials ### - [ ] Changes are complete (i.e. I finished coding

[GitHub] [incubator-mxnet] rondogency commented on pull request #18331: Remove test metric perf

2020-05-15 Thread GitBox
rondogency commented on pull request #18331: URL: https://github.com/apache/incubator-mxnet/pull/18331#issuecomment-629566211 @leezu the performance benchmark we are running is focusing on model level performance on end to end training/inference, this one is more related to opperf to test

[GitHub] [incubator-mxnet] ys2843 commented on pull request #18288: Website global search feature

2020-05-15 Thread GitBox
ys2843 commented on pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#issuecomment-629589289 @mxnet-bot run ci [unix-cpu] This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18288: Website global search feature

2020-05-15 Thread GitBox
mxnet-bot commented on pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#issuecomment-629589299 Jenkins CI successfully triggered : [unix-cpu] This is an automated message from the Apache Git

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

2020-05-15 Thread aaronmarkham
This is an automated email from the ASF dual-hosted git repository. aaronmarkham 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 e61c5b3 Bump the

[GitHub] [incubator-mxnet-ci] marcoabreu commented on a change in pull request #25: Serverless implementation for jenkins pipeline monitor lambda

2020-05-15 Thread GitBox
marcoabreu commented on a change in pull request #25: URL: https://github.com/apache/incubator-mxnet-ci/pull/25#discussion_r425606759 ## File path: services/jenkins-pipeline-monitor/handler.py ## @@ -0,0 +1,134 @@ +import os +import boto3 +import json +import logging +import

[GitHub] [incubator-mxnet] leezu commented on pull request #9705: [MXNET-91] Added unittest for benchmarking metric performance

2020-05-15 Thread GitBox
leezu commented on pull request #9705: URL: https://github.com/apache/incubator-mxnet/pull/9705#issuecomment-629076060 Why does this test only print numbers but doesn't actually enforce anything? This is an automated

[GitHub] [incubator-mxnet] ChaiBapchya edited a comment on issue #18330: test_metric_performance

2020-05-15 Thread GitBox
ChaiBapchya edited a comment on issue #18330: URL: https://github.com/apache/incubator-mxnet/issues/18330#issuecomment-629044532 http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Funix-cpu/detail/PR-18326/1/pipeline

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18327: update init.py with appropriate comments

2020-05-15 Thread GitBox
mxnet-bot commented on pull request #18327: URL: https://github.com/apache/incubator-mxnet/pull/18327#issuecomment-629046009 Jenkins CI successfully triggered : [unix-cpu] This is an automated message from the Apache Git

[GitHub] [incubator-mxnet] ChaiBapchya commented on pull request #18327: update init.py with appropriate comments

2020-05-15 Thread GitBox
ChaiBapchya commented on pull request #18327: URL: https://github.com/apache/incubator-mxnet/pull/18327#issuecomment-629045984 @mxnet-bot run ci [unix-cpu] This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-mxnet] ciyongch commented on pull request #17672: [Numpy] more support for boolean indexing/assign and change on implementation

2020-05-15 Thread GitBox
ciyongch commented on pull request #17672: URL: https://github.com/apache/incubator-mxnet/pull/17672#issuecomment-629046780 Thanks @Alicia1529 , please also tag me when the new PRs are created. This is an automated message

[GitHub] [incubator-mxnet] leezu commented on issue #18330: test_metric_performance

2020-05-15 Thread GitBox
leezu commented on issue #18330: URL: https://github.com/apache/incubator-mxnet/issues/18330#issuecomment-629062928 Notice that this timeout only happens on Python3: MKL-CPU jobs. The job there times out after 1200 seconds (global timeout), whereas it finishes in under 300 seconds on the

[GitHub] [incubator-mxnet] leezu commented on issue #18330: test_metric_performance

2020-05-15 Thread GitBox
leezu commented on issue #18330: URL: https://github.com/apache/incubator-mxnet/issues/18330#issuecomment-629063001 Suggest we mark this test as xfail for MKL builds for now This is an automated message from the Apache Git

[GitHub] [incubator-mxnet] leezu commented on issue #18330: test_metric_performance

2020-05-15 Thread GitBox
leezu commented on issue #18330: URL: https://github.com/apache/incubator-mxnet/issues/18330#issuecomment-629077891 The test doesn't actually enforce anything. It's output is not monitored (AFAIK) so it may be best to simply delete this test.

[GitHub] [incubator-mxnet-ci] mseth10 commented on a change in pull request #25: Serverless implementation for jenkins pipeline monitor lambda

2020-05-15 Thread GitBox
mseth10 commented on a change in pull request #25: URL: https://github.com/apache/incubator-mxnet-ci/pull/25#discussion_r425608613 ## File path: services/jenkins-pipeline-monitor/handler.py ## @@ -0,0 +1,134 @@ +import os +import boto3 +import json +import logging +import

[GitHub] [incubator-mxnet] sxjscience commented on pull request #18328: Fix memory leaks in Gluon

2020-05-15 Thread GitBox
sxjscience commented on pull request #18328: URL: https://github.com/apache/incubator-mxnet/pull/18328#issuecomment-629079777 @Jerryzcn I think this is also related to your previous benchmark. This is an automated message

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18331: No test metric perf

2020-05-15 Thread GitBox
mxnet-bot commented on pull request #18331: URL: https://github.com/apache/incubator-mxnet/pull/18331#issuecomment-629086999 Hey @acphile , Thanks for submitting the PR All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following

  1   2   >