[GitHub] madlib issue #75: SVM: Implement c++ functions for training multi-class svm ...

2018-02-05 Thread njayaram2
Github user njayaram2 commented on the issue: https://github.com/apache/madlib/pull/75 The mini-batch related code in this PR was merged with https://github.com/apache/madlib/pull/229. Multi-class SVM related code is yet to be merged. ---

[GitHub] madlib issue #75: SVM: Implement c++ functions for training multi-class svm ...

2018-02-05 Thread njayaram2
Github user njayaram2 commented on the issue: https://github.com/apache/madlib/pull/75 @mktal can you please close this PR? This code is now merged with https://github.com/apache/madlib/pull/229. ---

[GitHub] madlib pull request #225: Added option for weighted average for both classif...

2018-01-24 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/225#discussion_r163653414 --- Diff: src/ports/postgres/modules/knn/knn.py_in --- @@ -212,22 +244,27 @@ def knn(schema_madlib, point_source, point_column_name, point_id

[GitHub] madlib pull request #225: Added option for weighted average for both classif...

2018-01-24 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/225#discussion_r163657952 --- Diff: src/ports/postgres/modules/knn/test/knn.sql_in --- @@ -72,43 +72,55 @@ copy knn_test_data (id, data) from stdin delimiter

[GitHub] madlib pull request #229: SVM: Add minibatch as a new solver

2018-01-22 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/229#discussion_r163048733 --- Diff: src/modules/convex/algo/igd.hpp --- @@ -56,6 +59,62 @@ IGD<State, ConstState, Task>::transition(stat

[GitHub] madlib pull request #229: SVM: Add minibatch as a new solver

2018-01-22 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/229#discussion_r163012952 --- Diff: src/modules/convex/algo/igd.hpp --- @@ -34,7 +34,10 @@ class IGD { typedef typename Task::model_type model_type; static

[GitHub] madlib issue #223: Balance datasets : re-sampling technique

2018-01-18 Thread njayaram2
Github user njayaram2 commented on the issue: https://github.com/apache/madlib/pull/223 Thank you for all the comments @fmcquillan99 and @orhankislal . We will push the changes addressing the comments soon. ---

[GitHub] madlib issue #223: Balance datasets : re-sampling technique

2018-01-18 Thread njayaram2
Github user njayaram2 commented on the issue: https://github.com/apache/madlib/pull/223 @fmcquillan99 thank you for all the test cases. There were quite a few good catches. We have made multiple changes to address them. ---

[GitHub] madlib pull request #223: Balance datasets : re-sampling technique

2018-01-18 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/223#discussion_r162515086 --- Diff: src/ports/postgres/modules/sample/balance_sample.py_in --- @@ -0,0 +1,994 @@ +# coding=utf-8 +# +# Licensed to the Apache Software

[GitHub] madlib pull request #223: Balance datasets : re-sampling technique

2018-01-18 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/223#discussion_r162515050 --- Diff: src/ports/postgres/modules/sample/balance_sample.py_in --- @@ -0,0 +1,994 @@ +# coding=utf-8 +# +# Licensed to the Apache Software

[GitHub] madlib pull request #227: Add docker file for postgres 9.6 and 10

2018-01-17 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/227#discussion_r162219833 --- Diff: tool/docker/base/Dockerfile_postgres_9_6 --- @@ -1,71 +0,0 @@ -# -# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] madlib pull request #225: Added option for weighted average for both classif...

2018-01-16 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/225#discussion_r161918108 --- Diff: src/ports/postgres/modules/knn/knn.sql_in --- @@ -326,6 +331,39 @@ Result, with neighbors sorted from closest to furthest: (6 rows

[GitHub] madlib pull request #225: Added option for weighted average for both classif...

2018-01-16 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/225#discussion_r161917948 --- Diff: src/ports/postgres/modules/knn/knn.sql_in --- @@ -412,7 +451,8 @@ CREATE OR REPLACE FUNCTION MADLIB_SCHEMA.knn( output_table

[GitHub] madlib-site issue #12: Update old releases download URLs

2018-01-02 Thread njayaram2
Github user njayaram2 commented on the issue: https://github.com/apache/madlib-site/pull/12 Thank you for the catch @edespino. Have made changes. ---

[GitHub] madlib pull request #212: update PyXB version in README.md

2017-12-07 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/212#discussion_r155684500 --- Diff: README.md --- @@ -11,9 +11,11 @@ Installation and Contribution == See the project website [`MADlib Home

[GitHub] madlib pull request #210: Regularized Regression: Change cross validation st...

2017-12-06 Thread njayaram2
GitHub user njayaram2 opened a pull request: https://github.com/apache/madlib/pull/210 Regularized Regression: Change cross validation stats JIRA:MADLIB-1169 Cross Validation seems to be supported by Elastic Net, SVM, and Decision Trees. If a module is run with cross

[GitHub] madlib issue #207: Infra: Install MADlib on non-default schema in Jenkins

2017-12-01 Thread njayaram2
Github user njayaram2 commented on the issue: https://github.com/apache/madlib/pull/207 Jenkins ok to test ---

[GitHub] madlib pull request #199: Bugfix: Hard coded schema name in WCC install chec...

2017-11-13 Thread njayaram2
GitHub user njayaram2 opened a pull request: https://github.com/apache/madlib/pull/199 Bugfix: Hard coded schema name in WCC install check JIRA: MADLIB-1177 WCC install check sql file had madlib hard coded as the schema name. This commit removes the schema name

[GitHub] madlib pull request #181: Add a script to start docker image

2017-09-22 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/181#discussion_r140573035 --- Diff: README.md --- @@ -18,14 +18,22 @@ Development with Docker === We provide a Docker image with necessary dependencies

[GitHub] madlib issue #182: MADLIB-1154. PostgreSQL 10 (beta 4) support.

2017-09-19 Thread njayaram2
Github user njayaram2 commented on the issue: https://github.com/apache/madlib/pull/182 Will go ahead and merge this since @jingyimei has approved. ---

[GitHub] madlib pull request #187: Docs: MLP docs to be corrected post grouping suppo...

2017-09-19 Thread njayaram2
GitHub user njayaram2 opened a pull request: https://github.com/apache/madlib/pull/187 Docs: MLP docs to be corrected post grouping support The MLP docs created during grouping support had some inadvertant changes. This commit fixes those issues. Closes #187 You can

[GitHub] madlib pull request #181: Add a script to start docker image

2017-09-18 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/181#discussion_r139574130 --- Diff: README.md --- @@ -18,7 +18,11 @@ Development with Docker === We provide a Docker image with necessary dependencies

[GitHub] madlib pull request #181: Add a script to start docker image

2017-09-18 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/181#discussion_r139573813 --- Diff: tool/docker_start.sh --- @@ -0,0 +1,98 @@ +#!/bin/bash +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] madlib pull request #181: Add a script to start docker image

2017-09-18 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/181#discussion_r139573759 --- Diff: tool/docker_start.sh --- @@ -0,0 +1,98 @@ +#!/bin/bash +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] madlib pull request #181: Add a script to start docker image

2017-09-18 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/181#discussion_r139573551 --- Diff: tool/docker_start.sh --- @@ -0,0 +1,98 @@ +#!/bin/bash +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] madlib pull request #181: Add a script to start docker image

2017-09-18 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/181#discussion_r139573426 --- Diff: README.md --- @@ -18,7 +18,11 @@ Development with Docker === We provide a Docker image with necessary dependencies

[GitHub] madlib pull request #178: Featur: HITS

2017-09-11 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/178#discussion_r138208835 --- Diff: src/ports/postgres/modules/graph/hits.sql_in --- @@ -0,0 +1,297

[GitHub] madlib pull request #178: Featur: HITS

2017-09-11 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/178#discussion_r138204869 --- Diff: src/ports/postgres/modules/graph/hits.py_in --- @@ -0,0 +1,417 @@ +# coding=utf-8 +# +# Licensed to the Apache Software Foundation

[GitHub] madlib pull request #179: MLP: Add grouping support to neural nets

2017-09-07 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/179#discussion_r137672182 --- Diff: src/ports/postgres/modules/convex/mlp_igd.py_in --- @@ -158,175 +236,257 @@ def mlp(schema_madlib, source_table, output_table

[GitHub] madlib pull request #178: Featur: HITS

2017-09-01 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/178#discussion_r136633157 --- Diff: src/ports/postgres/modules/graph/hits.py_in --- @@ -0,0 +1,439 @@ +# coding=utf-8 +# +# Licensed to the Apache Software Foundation

[GitHub] madlib pull request #178: Featur: HITS

2017-09-01 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/178#discussion_r136631234 --- Diff: src/ports/postgres/modules/graph/hits.py_in --- @@ -0,0 +1,439 @@ +# coding=utf-8 +# +# Licensed to the Apache Software Foundation

[GitHub] madlib pull request #178: Featur: HITS

2017-09-01 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/178#discussion_r136631039 --- Diff: src/ports/postgres/modules/graph/hits.sql_in --- @@ -0,0 +1,307

[GitHub] madlib pull request #178: Featur: HITS

2017-09-01 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/178#discussion_r136643041 --- Diff: src/ports/postgres/modules/graph/hits.py_in --- @@ -0,0 +1,439 @@ +# coding=utf-8 +# +# Licensed to the Apache Software Foundation

[GitHub] madlib pull request #178: Featur: HITS

2017-09-01 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/178#discussion_r136630786 --- Diff: src/ports/postgres/modules/graph/hits.sql_in --- @@ -0,0 +1,307

[GitHub] madlib pull request #178: Featur: HITS

2017-09-01 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/178#discussion_r136632178 --- Diff: src/ports/postgres/modules/graph/test/hits.sql_in --- @@ -0,0 +1,81

[GitHub] madlib pull request #178: Featur: HITS

2017-09-01 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/madlib/pull/178#discussion_r136630581 --- Diff: src/ports/postgres/modules/graph/hits.sql_in --- @@ -0,0 +1,307

[GitHub] madlib issue #179: MLP: Add grouping support to neural nets

2017-08-31 Thread njayaram2
Github user njayaram2 commented on the issue: https://github.com/apache/madlib/pull/179 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 wishes so

[GitHub] madlib issue #179: MLP: Add grouping support to neural nets

2017-08-31 Thread njayaram2
Github user njayaram2 commented on the issue: https://github.com/apache/madlib/pull/179 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 wishes so

[GitHub] madlib issue #179: MLP: Add grouping support to neural nets

2017-08-31 Thread njayaram2
Github user njayaram2 commented on the issue: https://github.com/apache/madlib/pull/179 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 wishes so

[GitHub] madlib issue #179: MLP: Add grouping support to neural nets

2017-08-31 Thread njayaram2
Github user njayaram2 commented on the issue: https://github.com/apache/madlib/pull/179 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 wishes so

[GitHub] madlib issue #179: MLP: Add grouping support to neural nets

2017-08-31 Thread njayaram2
Github user njayaram2 commented on the issue: https://github.com/apache/madlib/pull/179 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 wishes so

[GitHub] madlib pull request #179: MLP: Add grouping support to neural nets

2017-08-31 Thread njayaram2
GitHub user njayaram2 opened a pull request: https://github.com/apache/madlib/pull/179 MLP: Add grouping support to neural nets JIRA: MADLIB-1149 Changes to support grouping with neural nets. Changes include: - Standardize independent features by group. - Use

<    1   2