Re: [VOTE]: MADlib repo(s) migration

2017-08-09 Thread Rahul Iyer
+0 for either options. I feel the Github workflow doesn't add much and prefer to not use the Github merge button. Keeping the history clean when possible requires merging on local machine, at which point either Github or ASF is just a change in remote URL. On Wed, Aug 9, 2017 at 2:47 PM, Orhan

[GitHub] incubator-madlib issue #162: MLP: Multilayer Perceptron Phase 2

2017-08-09 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/incubator-madlib/pull/162 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/madlib-pr-build/146/ --- If your project is set up for it, you can reply

[GitHub] incubator-madlib issue #162: MLP: Multilayer Perceptron Phase 2

2017-08-09 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/incubator-madlib/pull/162 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/madlib-pr-build/145/ --- If your project is set up for it, you can reply

[GitHub] incubator-madlib pull request #162: MLP: Multilayer Perceptron Phase 2

2017-08-09 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/162#discussion_r132340750 --- Diff: doc/design/modules/neural-network.tex --- @@ -197,3 +207,44 @@ \subsubsection{The $\mathit{Gradient}$ Function} \State \Return

[GitHub] incubator-madlib pull request #162: MLP: Multilayer Perceptron Phase 2

2017-08-09 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/162#discussion_r132340321 --- Diff: doc/design/modules/neural-network.tex --- @@ -46,41 +47,49 @@ \subsection{Formal Description} In the remaining part of this

[GitHub] incubator-madlib pull request #162: MLP: Multilayer Perceptron Phase 2

2017-08-09 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/162#discussion_r132339557 --- Diff: doc/design/modules/neural-network.tex --- @@ -46,41 +47,49 @@ \subsection{Formal Description} In the remaining part of this

[GitHub] incubator-madlib issue #163: MADLIB-1118. Change tolerance to 1e-2 (from 1e-...

2017-08-09 Thread njayaram2
Github user njayaram2 commented on the issue: https://github.com/apache/incubator-madlib/pull/163 LGTM, since we anyway don't assert on `relative_error` on `log_likelihood` in elastic_net. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-madlib issue #162: MLP: Multilayer Perceptron Phase 2

2017-08-09 Thread njayaram2
Github user njayaram2 commented on the issue: https://github.com/apache/incubator-madlib/pull/162 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or

Re: [VOTE]: MADlib repo(s) migration

2017-08-09 Thread Orhan Kislal
1 Orhan Kislal On Wed, Aug 9, 2017 at 2:32 PM, Nandish Jayaram wrote: > Hi All, > > With MADlib's graduation to TLP, it's time to migrate its github > repos from `*incubator-madlib*` to `*madlib*`. We will have to open > an Apache Infrastructure ticket to request this move

Re: [VOTE]: MADlib repo(s) migration

2017-08-09 Thread Nandish Jayaram
1 NJ On Wed, Aug 9, 2017 at 2:40 PM, Ed Espino wrote: > 2 > > Thanks NJ! > > On Wed, Aug 9, 2017 at 2:35 PM, Cooper Sloan wrote: > > > 2 > > > > On Wed, Aug 9, 2017 at 2:32 PM Nandish Jayaram > > wrote: > > > > > Hi All, > > > > > >

[VOTE]: MADlib repo(s) migration

2017-08-09 Thread Nandish Jayaram
Hi All, With MADlib's graduation to TLP, it's time to migrate its github repos from `*incubator-madlib*` to `*madlib*`. We will have to open an Apache Infrastructure ticket to request this move for the following repos (along with other stuff like wiki, jenkins etc):

Re: Migrating MADlib code base out of incubator.

2017-08-09 Thread Ed Espino
NJ, I would probably start another with a clear Subject. Thanks, -=e On Wed, Aug 9, 2017 at 2:08 PM Nandish Jayaram wrote: > > Shall we put the main MADlib repo(s) migration to GitBox to a vote? > Sure. Do we vote on this thread or on a different one? > > NJ > > On Wed,

[GitHub] incubator-madlib issue #162: MLP: Multilayer Perceptron Phase 2

2017-08-09 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/incubator-madlib/pull/162 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/madlib-pr-build/143/ --- If your project is set up for it, you can reply

Re: Migrating MADlib code base out of incubator.

2017-08-09 Thread Nandish Jayaram
> Shall we put the main MADlib repo(s) migration to GitBox to a vote? Sure. Do we vote on this thread or on a different one? NJ On Wed, Aug 9, 2017 at 12:28 PM, Ed Espino wrote: > I believe there is another git repo "incubator-madlib-site" (pair) that > needs to be migrated

MADLIB-1118 - IC is taking too long for elastic net

2017-08-09 Thread Ed Espino
MADlibers, I took a stab at addressing MADLIB-1118 (IC is taking too long for elastic net). I'm not sure if the approach (updating elastic_net_train tolerance value (1e-6 --> 1e-2) is appropriate. Although it does deliver the expected reduction in execution. As the jira suggested, I tried to

[GitHub] incubator-madlib issue #163: MADLIB-1118. Change tolerance to 1e-2 (from 1e-...

2017-08-09 Thread edespino
Github user edespino commented on the issue: https://github.com/apache/incubator-madlib/pull/163 For future reference, this is how I reviewed the elastic_net install-check execution: * Update the following

Re: Migrating MADlib code base out of incubator.

2017-08-09 Thread Ed Espino
I believe there is another git repo "incubator-madlib-site" (pair) that needs to be migrated as well: https://git1-us-west.apache.org/repos/asf?p=incubator-madlib-site.git https://github.com/apache/incubator-madlib-site (GitHub mirror) Personally, I am in favor of Gitbox use. I believe there is

Migrating MADlib code base out of incubator.

2017-08-09 Thread Nandish Jayaram
Hi All, I was planning on opening an Apache Infra JIRA to migrate MADlib's current code repo out of "incubator", following the directions in: https://incubator.apache.org/guides/transferring.html http://www.apache.org/dev/infra-contact#requesting-graduation An example Infra JIRA we can follow:

[GitHub] incubator-madlib pull request #163: MADLIB-1118. Change tolerance to 1e-2 (f...

2017-08-09 Thread edespino
GitHub user edespino opened a pull request: https://github.com/apache/incubator-madlib/pull/163 MADLIB-1118. Change tolerance to 1e-2 (from 1e-6) This changes the execution elapsed time to 2252 milliseconds from 10171 milliseconds on mac with Postgre 9.6 You can merge this pull