Skip to content

Is this a typo? #1

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

Open
santisy wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Is this a typo? #1

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
031cd1d
Is this a typo?
santisy Nov 30, 2016
42d4ddf
self_use_init
santisy Dec 1, 2016
1758321
dec-1-success
santisy Dec 1, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion MFNov29Helper.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ def userGradientUpdate(RU, P, Uu, lambda1):
# this function updates the ith row of item feature matrix P, for the vectorized gradient update
def itemGradientUpdate(RI, U, Pi, lambda2):

RI_til = np.dot(Pi,U)*np.sign(RI)
RI_hat = np.dot(Pi,U)*np.sign(RI)

@cheng-01037 cheng-01037 Nov 30, 2016

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the correction. This function is still unused based on the way updating the gradient.

grad = np.dot((RI_hat - RI),U) + lambda2*Pi
return grad

Expand Down