Skip to content

Conversation

@yinxusen
Copy link
Contributor

If there are no transpose() in self.theta, a

ValueError: matrices are not aligned

is occurring. The former test case just ignore this situation.

@AmplabJenkins
Copy link

Merged build triggered.

@AmplabJenkins
Copy link

Merged build started.

@AmplabJenkins
Copy link

Merged build finished. All automated tests passed.

@AmplabJenkins
Copy link

All automated tests passed.
Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/14282/

@mengxr
Copy link
Contributor

mengxr commented Apr 22, 2014

LGTM. @yinxusen Thanks for the fix!

@pwendell
Copy link
Contributor

Thanks, merged.

asfgit pushed a commit that referenced this pull request Apr 22, 2014
If there are no `transpose()` in `self.theta`, a

*ValueError: matrices are not aligned*

is occurring. The former test case just ignore this situation.

Author: Xusen Yin <[email protected]>

Closes #463 from yinxusen/python-naive-bayes and squashes the following commits:

fcbe3bc [Xusen Yin] fix bugs of dot in python
(cherry picked from commit c919798)

Signed-off-by: Patrick Wendell <[email protected]>
@asfgit asfgit closed this in c919798 Apr 22, 2014
pdeyhim pushed a commit to pdeyhim/spark-1 that referenced this pull request Jun 25, 2014
If there are no `transpose()` in `self.theta`, a

*ValueError: matrices are not aligned*

is occurring. The former test case just ignore this situation.

Author: Xusen Yin <[email protected]>

Closes apache#463 from yinxusen/python-naive-bayes and squashes the following commits:

fcbe3bc [Xusen Yin] fix bugs of dot in python
asfgit pushed a commit that referenced this pull request Jul 17, 2014
This is the same as #463 , which I forgot to merge into branch-0.9.

Author: Xiangrui Meng <[email protected]>

Closes #1453 from mengxr/nb-transpose-0.9 and squashes the following commits:

bc53ce8 [Xiangrui Meng] fix NaiveBayes
bzhaoopenstack pushed a commit to bzhaoopenstack/spark that referenced this pull request Sep 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants