GitHub user ArvindSridhar opened a pull request:
https://github.com/apache/madlib/pull/309
Elastic Net: Fix bug related to grouping by columns of non-numeric types
This PR fixes a bug in the elastic_net module pertaining to grouping.
Previously, if one tried to group by a column that was not of a numeric type
(ex. TEXT, boolean, etc.), the program would error out. This PR fixes the error
for all supported versions of Postgres and Greenplum.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/madlib/madlib bugfix/elastic_net_grouping
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/madlib/pull/309.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #309
----
commit 1e860c7c7be9beb178be4bf4368ab96083370783
Author: Arvind Sridhar <arvindsridhar@...>
Date: 2018-08-08T23:16:57Z
Elastic Net: Made change in python code to fix issue with grouping on
non-numeric columns
commit 550454e04fcdb480f7dc08fc84798894115b780b
Author: Arvind Sridhar <arvindsridhar@...>
Date: 2018-08-08T23:24:21Z
Elastic Net: Added dev-check test for grouping on non-numeric column
commit 75229d23c2e28809c8ed4c642454a7faaa8d53c8
Author: Arvind Sridhar <arvindsridhar@...>
Date: 2018-08-09T00:47:41Z
Elastic Net: Added dev-check test for grouping on col of type boolean
----
---
