-
Notifications
You must be signed in to change notification settings - Fork 28.9k
[SPARK-5726] [MLLIB] Elementwise (Hadamard) Vector Product Transformer #4580
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Can one of the admins verify this patch? |
|
add to whitelist |
|
ok to test |
|
Test build #27419 has started for PR 4580 at commit
|
|
Test build #27419 has finished for PR 4580 at commit
|
|
Test PASSed. |
|
Test build #27446 has started for PR 4580 at commit
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does TF mean here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, just realized the TF in HashingTF stands for 'term frequencies', not 'transformer'. Will remove. Thanks for pointing out.
|
Test build #27449 has started for PR 4580 at commit
|
|
Test build #27446 has finished for PR 4580 at commit
|
|
Test FAILed. |
|
Test build #27449 has finished for PR 4580 at commit
|
|
Test FAILed. |
|
@mengxr the unit tests and style checks for this feature pass locally, and the jenkins build actually shows the test suite also passing, but there is a compilation failure, any suggestions to get the build green? edit: rebased on master and still occurring the error is return exit status 1 in run-tests, this line: final edit: passing after another rebase |
cafe8c9 to
453a502
Compare
|
Test build #27453 has started for PR 4580 at commit
|
|
Test build #27453 has finished for PR 4580 at commit
|
|
Test FAILed. |
453a502 to
a7823e5
Compare
|
Test build #27525 has started for PR 4580 at commit
|
|
Test build #27525 has finished for PR 4580 at commit
|
|
Test PASSed. |
a7823e5 to
7eb8f0f
Compare
|
Test build #28597 has started for PR 4580 at commit
|
|
Test build #28597 has finished for PR 4580 at commit
|
|
Test PASSed. |
7eb8f0f to
e7ff5f2
Compare
|
Test build #29243 has started for PR 4580 at commit
|
|
Test build #29243 has finished for PR 4580 at commit
|
3ea9b71 to
90f7e39
Compare
|
Rebased. |
|
Merged build triggered. |
|
Merged build started. |
|
Test build #32131 has started for PR 4580 at commit |
|
Test build #32131 has finished for PR 4580 at commit
|
|
Merged build finished. Test FAILed. |
|
Test FAILed. |
|
@ogeagla Please look at the updated |
|
Merged build triggered. |
|
Merged build started. |
|
Test build #32140 has started for PR 4580 at commit |
|
Test build #32140 has finished for PR 4580 at commit
|
|
Merged build finished. Test PASSed. |
|
Test PASSed. |
|
LGTM Merging into master and branch-1.4 Thanks very much! |
See https://issues.apache.org/jira/browse/SPARK-5726 Author: Octavian Geagla <[email protected]> Author: Joseph K. Bradley <[email protected]> Closes #4580 from ogeagla/spark-mllib-weighting and squashes the following commits: fac12ad [Octavian Geagla] [SPARK-5726] [MLLIB] Use new createTransformFunc. 90f7e39 [Joseph K. Bradley] small cleanups 4595165 [Octavian Geagla] [SPARK-5726] [MLLIB] Remove erroneous test case. ded3ac6 [Octavian Geagla] [SPARK-5726] [MLLIB] Pass style checks. 37d4705 [Octavian Geagla] [SPARK-5726] [MLLIB] Incorporated feedback. 1dffeee [Octavian Geagla] [SPARK-5726] [MLLIB] Pass style checks. e436896 [Octavian Geagla] [SPARK-5726] [MLLIB] Remove 'TF' from 'ElementwiseProductTF' cb520e6 [Octavian Geagla] [SPARK-5726] [MLLIB] Rename HadamardProduct to ElementwiseProduct 4922722 [Octavian Geagla] [SPARK-5726] [MLLIB] Hadamard Vector Product Transformer (cherry picked from commit 658a478) Signed-off-by: Joseph K. Bradley <[email protected]>
See https://issues.apache.org/jira/browse/SPARK-5726 Author: Octavian Geagla <[email protected]> Author: Joseph K. Bradley <[email protected]> Closes apache#4580 from ogeagla/spark-mllib-weighting and squashes the following commits: fac12ad [Octavian Geagla] [SPARK-5726] [MLLIB] Use new createTransformFunc. 90f7e39 [Joseph K. Bradley] small cleanups 4595165 [Octavian Geagla] [SPARK-5726] [MLLIB] Remove erroneous test case. ded3ac6 [Octavian Geagla] [SPARK-5726] [MLLIB] Pass style checks. 37d4705 [Octavian Geagla] [SPARK-5726] [MLLIB] Incorporated feedback. 1dffeee [Octavian Geagla] [SPARK-5726] [MLLIB] Pass style checks. e436896 [Octavian Geagla] [SPARK-5726] [MLLIB] Remove 'TF' from 'ElementwiseProductTF' cb520e6 [Octavian Geagla] [SPARK-5726] [MLLIB] Rename HadamardProduct to ElementwiseProduct 4922722 [Octavian Geagla] [SPARK-5726] [MLLIB] Hadamard Vector Product Transformer
See https://issues.apache.org/jira/browse/SPARK-5726 Author: Octavian Geagla <[email protected]> Author: Joseph K. Bradley <[email protected]> Closes apache#4580 from ogeagla/spark-mllib-weighting and squashes the following commits: fac12ad [Octavian Geagla] [SPARK-5726] [MLLIB] Use new createTransformFunc. 90f7e39 [Joseph K. Bradley] small cleanups 4595165 [Octavian Geagla] [SPARK-5726] [MLLIB] Remove erroneous test case. ded3ac6 [Octavian Geagla] [SPARK-5726] [MLLIB] Pass style checks. 37d4705 [Octavian Geagla] [SPARK-5726] [MLLIB] Incorporated feedback. 1dffeee [Octavian Geagla] [SPARK-5726] [MLLIB] Pass style checks. e436896 [Octavian Geagla] [SPARK-5726] [MLLIB] Remove 'TF' from 'ElementwiseProductTF' cb520e6 [Octavian Geagla] [SPARK-5726] [MLLIB] Rename HadamardProduct to ElementwiseProduct 4922722 [Octavian Geagla] [SPARK-5726] [MLLIB] Hadamard Vector Product Transformer
See https://issues.apache.org/jira/browse/SPARK-5726 Author: Octavian Geagla <[email protected]> Author: Joseph K. Bradley <[email protected]> Closes apache#4580 from ogeagla/spark-mllib-weighting and squashes the following commits: fac12ad [Octavian Geagla] [SPARK-5726] [MLLIB] Use new createTransformFunc. 90f7e39 [Joseph K. Bradley] small cleanups 4595165 [Octavian Geagla] [SPARK-5726] [MLLIB] Remove erroneous test case. ded3ac6 [Octavian Geagla] [SPARK-5726] [MLLIB] Pass style checks. 37d4705 [Octavian Geagla] [SPARK-5726] [MLLIB] Incorporated feedback. 1dffeee [Octavian Geagla] [SPARK-5726] [MLLIB] Pass style checks. e436896 [Octavian Geagla] [SPARK-5726] [MLLIB] Remove 'TF' from 'ElementwiseProductTF' cb520e6 [Octavian Geagla] [SPARK-5726] [MLLIB] Rename HadamardProduct to ElementwiseProduct 4922722 [Octavian Geagla] [SPARK-5726] [MLLIB] Hadamard Vector Product Transformer
See https://issues.apache.org/jira/browse/SPARK-5726