Skip to content
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

Evaluation of unseen user and Incremental modeling issues in BPRMF #427

Merged
merged 4 commits into from
Jul 21, 2013

Conversation

ummae
Copy link
Contributor

@ummae ummae commented Jul 18, 2013

Hi, here is trivial one 79686af

Passing null parameter to HashSet when training_user_matrix[user_id] == null
===> ERROR: Argument cannot be null.
Parameter name: collection at System.Collections.Generic.HashSet1[System.Int32]..ctor (IEnumerable1 collection, IEqualityComparer1 comparer) [0x0006d] in /data1/daum/kskim/program/mono-2.10.8/mcs/class/System.Core/System.Collections.Generic/HashSet.cs:123
at System.Collections.Generic.HashSet1[System.Int32]..ctor (IEnumerable`1 collection) [0x00000] in :0
at MyMediaLite.Eval.Items+c__AnonStoreyA.<>m__14 (Int32 user_id) [0x0002e] in /data1/daum/kskim/mymedialite.git/src/MyMediaLite/Eval/Items.cs:146

…user is empty

fixed: random instance need to be initialized to perform incremental modeling(AddFeedback) in BPRMF
To perform incremental modeling on the loaded model(LoadModel), random
instance need to be instantiated on LoadModel().
@ummae
Copy link
Contributor Author

ummae commented Jul 18, 2013

Another one,

Incremental modeling cannot be performed after LoadModel() with Feedback data because random instance of BPRMF class is instantiated on Train().

I think random instance need to be instantiated on LoadModel() too.

commit: a056d45

zenogantner added a commit that referenced this pull request Jul 21, 2013
Evaluation of unseen user and Incremental modeling issues in BPRMF
@zenogantner zenogantner merged commit 0b1c719 into zenogantner:master Jul 21, 2013
@zenogantner
Copy link
Owner

Thank you for the pull request!
Another remark: MyMediaLite uses tabs for indentation, so if you send me a pull request that is tab-indented it would save me one step of work -- otherwise everything was fine.

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.

2 participants