[GitHub] madlib pull request #:

2018-08-17 Thread fmcquillan99
Github user fmcquillan99 commented on the pull request: https://github.com/apache/madlib/commit/5e707f745c50343dd7395a3e8f86c04428210977#commitcomment-30142753 Also fixed some spacing issues ---

Re: [VOTE] MADlib v1.15-rc1

2018-08-17 Thread Frank McQuillan
Just to follow up on Rashmi's suggestion from earlier in this thread, the vec2cols function has been changed for 1.15.1 to address the issue of unequal length arrays https://issues.apache.org/jira/browse/MADLIB-1270 Frank On Fri, Aug 10, 2018 at 3:28 PM, Ed Espino wrote: > +1 > > For reference,

Re: Issues compiling MADlib 1.12 for Postgres 10

2018-08-17 Thread Frank McQuillan
gr8, let us know if that goes OK Mauricio, and if u have any other questions Frank On Fri, Aug 17, 2018 at 3:56 AM, Mauricio Scheffer < mauricioschef...@gmail.com> wrote: > Thanks Orhan, Rahul. I was able to get past that error by > including in src/ports/postgres/dbconnector/UDF_impl.hpp > and

[GitHub] madlib issue #313: MLP: Simplify momentum and Nesterov updates

2018-08-17 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/madlib/pull/313 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/madlib-pr-build/656/ ---

[GitHub] madlib pull request #313: MLP: Simplify momentum and Nesterov updates

2018-08-17 Thread iyerr3
GitHub user iyerr3 opened a pull request: https://github.com/apache/madlib/pull/313 MLP: Simplify momentum and Nesterov updates Momentum updates are complicated due to Nesterov requiring an initial update before gradient calculations. There is, however, a different form of the Nest