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
[SPARK-3701][MLLIB] update python linalg api and small fixes #2548
Conversation
QA tests have started for PR 2548 at commit
|
QA tests have finished for PR 2548 at commit
|
Test FAILed. |
131ee3a
to
ef853f9
Compare
QA tests have started for PR 2548 at commit
|
Matrix.__init__(self, nRow, nCol) | ||
assert len(values) == nRow * nCol | ||
""" | ||
Column-majored dense matrix. |
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.
I'm pretty sure "major" is more common than "majored" (I think this typo exists elsewhere in MLlib too.)
LGTM (other than the one typo) |
QA tests have finished for PR 2548 at commit
|
Test FAILed. |
Build failure is my fault, due to merging #2351. I've reverted it and re-triggered your unit tests. |
QA tests have started for PR 2548 at commit
|
QA tests have finished for PR 2548 at commit
|
QA tests have started for PR 2548 at commit
|
QA tests have finished for PR 2548 at commit
|
Test FAILed. |
This new failure looks like it might be legitimate. |
QA tests have started for PR 2548 at commit
|
QA tests have finished for PR 2548 at commit
|
Test FAILed. |
QA tests have started for PR 2548 at commit
|
QA tests have finished for PR 2548 at commit
|
Test FAILed. |
QA tests have started for PR 2548 at commit
|
QA tests have finished for PR 2548 at commit
|
Test FAILed. |
test this please |
QA tests have started for PR 2548 at commit
|
QA tests have finished for PR 2548 at commit
|
Test PASSed. |
else: | ||
raise Exception("Cannot call dot with %d-dimensional array" % other.ndim) | ||
elif other.ndim > 2: | ||
raise ValueError("Cannot call dot with %d-dimentaional array" % other.ndim) |
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.
typo "dimentaional"
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.
done.
@mengxr LGTM, except for the typo and question above |
QA tests have started for PR 2548 at commit
|
QA tests have finished for PR 2548 at commit
|
Test FAILed. |
test this please |
QA tests have started for PR 2548 at commit
|
QA tests have finished for PR 2548 at commit
|
Test PASSed. |
Merged into master. Thanks @jkbradley for review! |
@davies @jkbradley