You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mahout.apache.org by sr...@apache.org on 2009/04/16 11:22:21 UTC
svn commit: r765534 - in
/lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model:
BooleanPrefUser.java file/FileDataModel.java
Author: srowen
Date: Thu Apr 16 09:22:18 2009
New Revision: 765534
URL: http://svn.apache.org/viewvc?rev=765534&view=rev
Log:
... and the rest of the fix for last regression involving change to FileDataModel
Modified:
lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/BooleanPrefUser.java
lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/file/FileDataModel.java
Modified: lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/BooleanPrefUser.java
URL: http://svn.apache.org/viewvc/lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/BooleanPrefUser.java?rev=765534&r1=765533&r2=765534&view=diff
==============================================================================
--- lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/BooleanPrefUser.java (original)
+++ lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/BooleanPrefUser.java Thu Apr 16 09:22:18 2009
@@ -23,6 +23,7 @@
import org.apache.mahout.cf.taste.impl.common.ArrayIterator;
import java.io.Serializable;
+import java.util.Arrays;
/**
* A variant of {@link GenericUser} which is appropriate when users express only a "yes" preference for
@@ -64,6 +65,7 @@
result[i] = buildPreference(itemID);
i++;
}
+ Arrays.sort(result, ByItemPreferenceComparator.getInstance());
return result;
}
Modified: lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/file/FileDataModel.java
URL: http://svn.apache.org/viewvc/lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/file/FileDataModel.java?rev=765534&r1=765533&r2=765534&view=diff
==============================================================================
--- lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/file/FileDataModel.java (original)
+++ lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/file/FileDataModel.java Thu Apr 16 09:22:18 2009
@@ -360,7 +360,7 @@
* are in fact {@link BooleanPreference}s
*/
protected User buildUser(String id, List<Preference> prefs) {
- if (!prefs.isEmpty() || prefs.get(0) instanceof BooleanPreference) {
+ if (!prefs.isEmpty() && prefs.get(0) instanceof BooleanPreference) {
// If first is a BooleanPreference, assuming all are, so, want to use BooleanPrefUser
FastSet<Object> itemIDs = new FastSet<Object>(prefs.size());
for (Preference pref : prefs) {