You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@madlib.apache.org by mktal <gi...@git.apache.org> on 2016/03/01 18:32:12 UTC
[GitHub] incubator-madlib pull request: SVM: Using temp table when generati...
GitHub user mktal opened a pull request:
https://github.com/apache/incubator-madlib/pull/28
SVM: Using temp table when generating random matrices
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mktal/incubator-madlib patch/svm_gaussian
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-madlib/pull/28.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 #28
----
commit 5da7a86f4c5a609ea121c2543d11fa18f773d0d7
Author: Xiaocheng Tang <xi...@gmail.com>
Date: 2016-03-01T17:29:58Z
SVM: Using temp table when generating random matrices
----
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-madlib pull request: SVM: Using temp table when generati...
Posted by iyerr3 <gi...@git.apache.org>.
Github user iyerr3 commented on the pull request:
https://github.com/apache/incubator-madlib/pull/28#issuecomment-195482049
Is the random matrix table not needed for prediction?
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-madlib pull request: SVM: Using temp table when generati...
Posted by iyerr3 <gi...@git.apache.org>.
Github user iyerr3 commented on the pull request:
https://github.com/apache/incubator-madlib/pull/28#issuecomment-195573062
Judging by how 'table_type' is used, I would expect 'TEMP' is the only useful/valid value. We should change that parameter to a boolean and add the 'TEMP' text in the called function.
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-madlib pull request: SVM: Using temp table when generati...
Posted by mktal <gi...@git.apache.org>.
Github user mktal commented on the pull request:
https://github.com/apache/incubator-madlib/pull/28#issuecomment-195484124
After training those random matrices are gathered into a single table which is used in prediction. The random matrices then need to be garbage-collected.
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-madlib pull request: SVM: Using temp table when generati...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-madlib/pull/28
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---