Re: AbstractFactorizer is not currently refreshable

2011-03-29 Thread Sebastian Schelter
Good point! Hadn't thought about that, than we should make Factorizer implement Refreshable as you proposed. --sebastian On 29.03.2011 14:12, Chris Newell wrote: Sebastian, Would it not be easier to just have SVDRecommender refresh the DataModel first and then just make it recompute the facto

Re: AbstractFactorizer is not currently refreshable

2011-03-29 Thread Chris Newell
Sebastian, Would it not be easier to just have SVDRecommender refresh the DataModel first and then just make it recompute the factorization? The problem is that when the DataModel is refreshed the userIDMapping and itemIDMapping in the Factorizer also need to be refreshed and I don't think th

Re: AbstractFactorizer is not currently refreshable

2011-03-29 Thread Sebastian Schelter
Hi Chris, Nice to see you take that task. Could you open a JIRA issue for it? Would it not be easier to just have SVDRecommender refresh the DataModel first and then just make it recompute the factorization? --sebastian On 29.03.2011 12:23, Chris Newell wrote: AbstractFactorizer (in packag

AbstractFactorizer is not currently refreshable

2011-03-29 Thread Chris Newell
AbstractFactorizer (in package org.apache.mahout.cf.taste.impl.recommender.svd) does not currently implement refreshable. This makes it difficult to implement refresh in SVDRecommender (currently a "ToDo" which I'd like to fix). There are two options I can see: 1) remember which Factorizer Cl