Re: [DISCUSS] Use modernized C++11 range loops uniformly throughout the project

2018-10-04 Thread Chris Olivier
-0.5 if keeping it as a warning. On Thu, Oct 4, 2018 at 1:23 AM kellen sunderland < kellen.sunderl...@gmail.com> wrote: > I agree that active C++ developers should be the ones making these > choices. The downside to that is that many of these people are already > pretty busy. To make the best

Re: Which merge option to use on the Import Julia binding PR?

2018-10-04 Thread Michael Wall
Great, glad it worked. I learned something too. On Thu, Oct 4, 2018, 22:09 Marco de Abreu wrote: > Oh nice, great catch Michael and Carin! I just learned something new, > thanks :) > > I guess we're all set then, right? Thanks a lot, everyone! > > -Marco > > Carin Meier schrieb am Fr., 5. Okt.

Jira and Github

2018-10-04 Thread Chris Olivier
Jira has new tight integration with github. I think we should look into enabling it. https://blog.github.com/2018-10-04-announcing-the-new-github-and-jira-software-cloud-integration/

Re: Which merge option to use on the Import Julia binding PR?

2018-10-04 Thread Marco de Abreu
Oh nice, great catch Michael and Carin! I just learned something new, thanks :) I guess we're all set then, right? Thanks a lot, everyone! -Marco Carin Meier schrieb am Fr., 5. Okt. 2018, 02:47: > Micheal, > > Thanks. You were right! I could merge. > > The PR shows up now as merged > https://g

Re: Maturity Model and Graduation

2018-10-04 Thread Steffen Rochel
I started a draft assessment - https://cwiki.apache.org/confluence/display/MXNET/Apache+Maturity+Model+Assessment+for+MXNet based on my personal view. Please keep in mind I'm new to the project and Apache (just attended my first ApacheCon!!). The items I was not sure myself I marked as "???". Jim

Re: Which merge option to use on the Import Julia binding PR?

2018-10-04 Thread Carin Meier
Micheal, Thanks. You were right! I could merge. The PR shows up now as merged https://github.com/apache/incubator-mxnet/pull/10149 My merge commit is here https://github.com/apache/incubator-mxnet/commits/master Thanks again for the help. - Carin On Thu, Oct 4, 2018 at 8:09 PM Michael Wall

Re: MXNet Podling Report - October

2018-10-04 Thread Haibin Lin
Hi Justin and Michael, I updated the report with the links to the tutorial summaries: June - https://lists.apache.org/thread.html/52f88e9dc7a6a2a1dfa5ad41c469fe2cdd1209a0be2eb345bc2f9a96@%3Cuser.mxnet.apache.org%3E July - https://lists.apache.org/thread.html/dea9184350f2fe87ce450722ead28072f763196

Re: Which merge option to use on the Import Julia binding PR?

2018-10-04 Thread Michael Wall
I would try the merge locally and then inspect the result closely to make sure it looks like what you want. If it looks good, you could try pushing to master. If you can't push, then we know but I "think" protected just means you can't force push in this case based on https://issues.apache.org/ji

Re: Which merge option to use on the Import Julia binding PR?

2018-10-04 Thread Marco de Abreu
You won't be able to push to master. The commit will be declined automatically, so command line is not an option. -Marco Carin Meier schrieb am Fr., 5. Okt. 2018, 01:29: > Micheal, > > Thanks for catching up and helping us with this. > I do see the "view command line instructions". I just assum

Re: Which merge option to use on the Import Julia binding PR?

2018-10-04 Thread Carin Meier
Micheal, Thanks for catching up and helping us with this. I do see the "view command line instructions". I just assumed that master was a protected branch and I would not be able to push to it. Honestly, I'm a bit scared if it isn't :) What do you suggest? Should I try to merge and push to master

Re: Which merge option to use on the Import Julia binding PR?

2018-10-04 Thread Michael Wall
Just now looking at this. The button is disabled for merge commit as you have mentioned. Before I go to INFRA, is the command line an option? Do you see "or view command line instructions" beside the green squash and merge button? On Thu, Oct 4, 2018 at 9:09 AM Carin Meier wrote: > Thank you

Re: CUDNN algorithm selection failure

2018-10-04 Thread kellen sunderland
"I ran a similar test(test_slice_batchnorm) for 5K times and I couldn't reproduce the issue." One thing to keep in mind is that the SelectAlgo call will cache results in a registry that is in static scope. To repro you'd likely have to create a new process each time you run the test. (Apologies

Re: MXNet Podling Report - October

2018-10-04 Thread Haibin Lin
Hi Justin, Thanks for the notice. I've reformatted the MXNet section to have at most 76 characters per line. Sorry about the last minute update. Best, Haibin On Thu, Oct 4, 2018 at 12:59 PM Justin Mclean wrote: > Hi, > > I noticed you have edited the report after the due date and have broken >

Re: MXNet Podling Report - October

2018-10-04 Thread Justin Mclean
Hi, I noticed you have edited the report after the due date and have broken the formatting a little after I formatted it. Each line must have a maximum of 76 characters, would you mind fixing your section of the report? Thanks, Justin

Re: CUDNN algorithm selection failure

2018-10-04 Thread Marco de Abreu
For GPU, we don't run any tests in parallel. -Marco Naveen Swamy schrieb am Do., 4. Okt. 2018, 19:54: > Looking at the error raised, you can see that the workspace size(GPU mem > size) of 1GB isn't sufficient. I am wondering if it is due to tests running > in parallel on CI, if this is true(tes

Re: CUDNN algorithm selection failure

2018-10-04 Thread Naveen Swamy
Looking at the error raised, you can see that the workspace size(GPU mem size) of 1GB isn't sufficient. I am wondering if it is due to tests running in parallel on CI, if this is true(tests running in parallel) is it possible to reduce the parallelism ? Error: "mxnet.base.MXNetError: [05:40:12] src

Re: CUDNN algorithm selection failure

2018-10-04 Thread Piyush Ghai
Another build where test_slice_batchnorm_reshape_batchnorm fails : http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/PR-12721/7/pipeline — Piyush

Re: MXNet Podling Report - October

2018-10-04 Thread Haibin Lin
Hi Michael, Thanks for reviewing the report! I've so far updated the report based on question 1,2, 4 and 5. 1. None. 2. Links: https://medium.com/apache-mxnet https://twitter.com/ApacheMXNet https://www.youtube.com/apachemxnet https://www.slideshare.net/apachemxnet https://www.reddit.com/r/m

Re: Which merge option to use on the Import Julia binding PR?

2018-10-04 Thread Carin Meier
Thank you Mike! On Thu, Oct 4, 2018 at 8:54 AM Michael Wall wrote: > Hi Carin, > > I will take a look at this tonight. I am not tracking everything, so I > want to go back and make sure I understand what is being asked. Then I am > happy to submit an INFRA ticket. > > Mike > > On Thu, Oct 4, 2

Re: Which merge option to use on the Import Julia binding PR?

2018-10-04 Thread Michael Wall
Hi Carin, I will take a look at this tonight. I am not tracking everything, so I want to go back and make sure I understand what is being asked. Then I am happy to submit an INFRA ticket. Mike On Thu, Oct 4, 2018 at 8:36 AM Carin Meier wrote: > I just found out that since we are a podling, w

Re: Which merge option to use on the Import Julia binding PR?

2018-10-04 Thread Carin Meier
I just found out that since we are a podling, we should route all our Infra tickets through one of our mentors and link the dev list discussion in JIRA. Is there a mentor that is willing to help us navigate this process to get the PR merged? Thanks, Carin On Tue, Oct 2, 2018 at 8:42 AM Carin Mei

Re: [DISCUSS] Use modernized C++11 range loops uniformly throughout the project

2018-10-04 Thread kellen sunderland
I agree that active C++ developers should be the ones making these choices. The downside to that is that many of these people are already pretty busy. To make the best use possible of their time it would probably make sense to create a concise doc with proposed style changes and ETAs rather than