GitHub user njayaram2 opened a pull request:
https://github.com/apache/incubator-madlib/pull/98
Bugfix/knn validation
knn: Fix input validation issues
- Some input validation was missing which is included now.
- there was still a bug wrt ambiguous references to column names,
which was only partly handled previously.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/njayaram2/incubator-madlib
bugfix/knn-validation
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-madlib/pull/98.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 #98
----
commit 3110c40a58d2b132f6d060abbc123f5febb1ad67
Author: Nandish Jayaram <[email protected]>
Date: 2017-02-03T23:30:36Z
knn: Fix input validation issues
- Some input validation was missing which is included now.
- there was still a bug wrt ambiguious references to column names,
which was only partly handled previously.
- In this version, the classification label column must be an integer,
1 and 0 for true and false. Having a boolen is not working out well
since madlib's mode function does not work with booleans. This
issue remains to be fixed, most likely will be fixed after 1.10.0
commit e96cae58d6c2b1899a8f5dea48d12458bfed354b
Author: Nandish Jayaram <[email protected]>
Date: 2017-02-04T00:04:30Z
handle boolean values for classification
commit 0d9657d534f08fc2f35aa55977b0d6c28975cb0e
Author: Nandish Jayaram <[email protected]>
Date: 2017-02-04T00:54:53Z
minor edits
----
---
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 feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
