Re: [DISCUSS] About the PR merging policy

2018-12-11 Thread Tianqi Chen
I think it is fine as long as we act on good faith. I will normally respect code review comments from anyone who might be able to give reasonable comments, and beg to differ with good technical reasoning. Normally contributions happen in a way that things won't get blocked in small features. For m

[DISCUSS] About the PR merging policy

2018-12-11 Thread Qing Lan
Hi all, Recently I self-merged my PR without getting approvals from other committers https://github.com/apache/incubator-mxnet/pull/13617 and only contributors approval. I apologize to the community and thank Marco for pointing out the problem. I took a lesson that we should at least have one c

Re: Slack invite hotlink

2018-12-11 Thread Michael Wall
I am not sure honestly. I read that as the token expires in the 30 days and the shortened URL is permanent and will always point to the token even after it expires. There has been some discussion in the #asfinfra channel on slack[1]. If I figure anything out I'll report back. [1] https://the-as

LabelBot Predictions

2018-12-11 Thread Harsh Patel
Hey all, I am looking forward to moving the predictions of labels into production. This an effort to reduce the number of issues that go unlabelled by the community. Labelling these issues provides a better way to serve MXNet users. Currently there are about 10 labels which we can predict with over

Re: Slack invite hotlink

2018-12-11 Thread Marco de Abreu
Hi Mike, thanks for the heads up and linking the ticket. In there, Chris stated that he converted the link to a permanent one: > The link in your shortened URL is now permanent. Does that address your concerns? Best regards, Marco On Tue, Dec 11, 2018 at 6:19 PM Michael Wall wrote: > Marco, >

Re: Include MKLDNN into default mxnet pip package

2018-12-11 Thread Naveen Swamy
Great effort Alex and also folks from Intel. +1 to make MKLDNN default. On Tue, Dec 11, 2018 at 9:10 AM Kumar, Vikas wrote: > +1 > > On 12/10/18, 8:01 PM, "Zhao, Patric" wrote: > > +1, thanks for the efforts, Alex. > > > > > -Original Message- > > From: Alex Zai [mailto:az

Re: Slack invite hotlink

2018-12-11 Thread Michael Wall
Marco, Seems like that link is only good for 30 days. See https://issues.apache.org/jira/browse/INFRA-17152. Mike On Tue, Dec 11, 2018 at 12:01 PM Marco de Abreu wrote: > Hey everyone, > > there's a hotlink [1] that allows people to join Apaches Slack server > directly, I found it here [2].

Re: Include MKLDNN into default mxnet pip package

2018-12-11 Thread Kumar, Vikas
+1 On 12/10/18, 8:01 PM, "Zhao, Patric" wrote: +1, thanks for the efforts, Alex. > -Original Message- > From: Alex Zai [mailto:aza...@gmail.com] > Sent: Tuesday, December 11, 2018 8:00 AM > To: dev@mxnet.incubator.apache.org > Subject: Include MKLD

Slack invite hotlink

2018-12-11 Thread Marco de Abreu
Hey everyone, there's a hotlink [1] that allows people to join Apaches Slack server directly, I found it here [2]. They would then still have to join the #mxnet channel though. Should we maybe add this to the instructions on the website? Best regards, Marco [1]: https://s.apache.org/slack-invite

GitHub CI status update not stable

2018-12-11 Thread Anton Chernov
Dear MXNet community, Currently the CI system is experiencing issues with GitHub verification status updates. If your PR is stuck with a status that looks like: ci/jenkins/mxnet-validation/centos-cpu Expected — Waiting for status to be reported ci/jenkins/mxnet-validation/centos-gpu Expected — Wa