Github user asfgit commented on the issue:
https://github.com/apache/madlib/pull/296
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/577/
---
Github user iyerr3 commented on a diff in the pull request:
https://github.com/apache/madlib/pull/295#discussion_r203811750
--- Diff:
src/ports/postgres/modules/recursive_partitioning/decision_tree.py_in ---
@@ -2327,6 +2328,110 @@ def _tree_error(schema_madlib, source_table,
depe
Github user iyerr3 commented on a diff in the pull request:
https://github.com/apache/madlib/pull/295#discussion_r203816091
--- Diff:
src/ports/postgres/modules/recursive_partitioning/decision_tree.py_in ---
@@ -2327,6 +2328,110 @@ def _tree_error(schema_madlib, source_table,
depe
Github user iyerr3 commented on a diff in the pull request:
https://github.com/apache/madlib/pull/295#discussion_r203812338
--- Diff:
src/ports/postgres/modules/recursive_partitioning/decision_tree.py_in ---
@@ -2327,6 +2328,110 @@ def _tree_error(schema_madlib, source_table,
depe
Github user iyerr3 commented on a diff in the pull request:
https://github.com/apache/madlib/pull/295#discussion_r203811384
--- Diff:
src/ports/postgres/modules/recursive_partitioning/decision_tree.py_in ---
@@ -2327,6 +2328,110 @@ def _tree_error(schema_madlib, source_table,
depe
Github user asfgit commented on the issue:
https://github.com/apache/madlib/pull/296
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/578/
---
Github user fmcquillan99 commented on the issue:
https://github.com/apache/madlib/pull/291
In cols2vec,
For this table:
```
CREATE TABLE golf (
id integer NOT NULL,
"OUTLOOK" text,
temperature double precision,
humidity double precision,
Github user njayaram2 commented on the issue:
https://github.com/apache/madlib/pull/295
Thank you for the comments @iyerr3 , will make necessary changes.
---
Github user asfgit commented on the issue:
https://github.com/apache/madlib/pull/295
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/579/
---
Github user asfgit commented on the issue:
https://github.com/apache/madlib/pull/295
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/580/
---
Github user fmcquillan99 commented on the issue:
https://github.com/apache/madlib/pull/291
In vec2cols,
```
SELECT madlib.vec2cols(
'golf', -- source table
'vec2cols_result',-- output table
'clouds_airquality
Github user fmcquillan99 commented on the issue:
https://github.com/apache/madlib/pull/291
After the above 2 issues I mentioned are fixed, I will have 1 more commit
on user docs to this PR
---
Github user asfgit commented on the issue:
https://github.com/apache/madlib/pull/296
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/581/
---
Github user asfgit commented on the issue:
https://github.com/apache/madlib/pull/291
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/582/
---
Github user fmcquillan99 commented on the issue:
https://github.com/apache/madlib/pull/295
Should impurity_var_importance always add up to 100?
From the regression example in the user docs:
```
DROP TABLE IF EXISTS mt_imp_output;
SELECT madlib.get_var_importance('mt
Github user fmcquillan99 commented on the issue:
https://github.com/apache/madlib/pull/295
Another run I got
```
grp 0 grp1
31.01364943 31.6576
22.85881741
Github user njayaram2 commented on a diff in the pull request:
https://github.com/apache/madlib/pull/291#discussion_r203890181
--- Diff: src/ports/postgres/modules/utilities/transform_vec_cols.py_in ---
@@ -0,0 +1,492 @@
+# Licensed to the Apache Software Foundation (ASF) under
Github user iyerr3 commented on the issue:
https://github.com/apache/madlib/pull/295
Considering the above situation, I suggest the variable importance values
not be scaled to sum to 100. We can make the normalization within
`get_var_importance` just for the reporting (which is the be
Github user fmcquillan commented on the issue:
https://github.com/apache/madlib/pull/295
Would this apply to oob too?
Or just impurity?
---
Github user njayaram2 commented on the issue:
https://github.com/apache/madlib/pull/295
@fmcquillan only impurity, I don't think we scale oob to 100.
---
Github user asfgit commented on the issue:
https://github.com/apache/madlib/pull/291
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/583/
---
21 matches
Mail list logo