[GitHub] incubator-madlib pull request #153: Graph: BFS algorithm design docs

2017-08-17 Thread rashmi815
Github user rashmi815 closed the pull request at: https://github.com/apache/incubator-madlib/pull/153 --- 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 if the

[GitHub] incubator-madlib issue #166: Sample: test_train_split

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

[GitHub] incubator-madlib-site issue #8: MLP demo

2017-08-17 Thread fmcquillan99
Github user fmcquillan99 commented on the issue: https://github.com/apache/incubator-madlib-site/pull/8 This is a really great demo workbook, thanks for posting. Suggestions 1) concat does not work for greenplum, perhaps do: SELECT

[GitHub] incubator-madlib pull request #165: Release 1.12: Version numbering and upgr...

2017-08-17 Thread orhankislal
Github user orhankislal commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/165#discussion_r133772221 --- Diff: src/madpack/changelist.yaml --- @@ -9,27 +9,39 @@ # file installed on the upgrade version. All other files (that don't have

[GitHub] incubator-madlib issue #166: Sample: test_train_split

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

[GitHub] incubator-madlib issue #166: Sample: test_train_split

2017-08-17 Thread njayaram2
Github user njayaram2 commented on the issue: https://github.com/apache/incubator-madlib/pull/166 jenkins ok to test --- 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

[GitHub] incubator-madlib issue #169: MLP: Doc Fix

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

Re: Apache MADlib v1.12 status

2017-08-17 Thread Frank McQuillan
Here is the PR https://github.com/apache/incubator-madlib/pull/169 Should be merged today On Wed, Aug 16, 2017 at 10:20 AM, Ed Espino wrote: > Frankie, > > Are there Jiras for the remaining work? This work (minor changes to neural > nets) is currently not visible on the

[GitHub] incubator-madlib issue #169: MLP: Doc Fix

2017-08-17 Thread fmcquillan99
Github user fmcquillan99 commented on the issue: https://github.com/apache/incubator-madlib/pull/169 the user docs part of this PR 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

[GitHub] incubator-madlib issue #166: Sample: test_train_split

2017-08-17 Thread iyerr3
Github user iyerr3 commented on the issue: https://github.com/apache/incubator-madlib/pull/166 +1 --- 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 if

[GitHub] incubator-madlib pull request #168: Code refactoring for KNN

2017-08-17 Thread iyerr3
Github user iyerr3 commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/168#discussion_r133629865 --- Diff: src/ports/postgres/modules/knn/knn.py_in --- @@ -127,5 +124,102 @@ def knn_validate_src(schema_madlib, point_source, point_column_name,

[GitHub] incubator-madlib pull request #168: Code refactoring for KNN

2017-08-17 Thread iyerr3
Github user iyerr3 commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/168#discussion_r133628535 --- Diff: src/ports/postgres/modules/knn/knn.py_in --- @@ -127,5 +124,102 @@ def knn_validate_src(schema_madlib, point_source, point_column_name,

[GitHub] incubator-madlib pull request #168: Code refactoring for KNN

2017-08-17 Thread iyerr3
Github user iyerr3 commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/168#discussion_r133628699 --- Diff: src/ports/postgres/modules/knn/knn.py_in --- @@ -127,5 +124,102 @@ def knn_validate_src(schema_madlib, point_source, point_column_name,