Documentation typo for initialization of CP-ALS. (Minor) #111

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.

Sign up for GitHub

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

Jump to bottom

Closed

tgkolda opened this issue May 23, 2023 · 0 comments · Fixed by #117

Assignees

Labels

documentation doing good first issue

Collaborator

tgkolda commented May 23, 2023

In: https://pyttb.readthedocs.io/en/latest/cpals.html#pyttb.cp_als.cp_als, it says...

“random”: initialize using a pyttb.ktensor with values chosen from a Normal distribution with mean 1 and standard deviation 0

It should be mean 0 and standard deviation 1.

ntjohnson1 added documentation good first issue labels

DeepBlockDeepak self-assigned this

DeepBlockDeepak added the doing label

DeepBlockDeepak linked a pull request

that will close this issue

Fixed the mean and stdev typo for cp_als #117

Merged

dmdunla closed this as completed in #117

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment