[jira] Commented: (MAHOUT-163) Get (better) cluster labels using Log Likelihood Ratio

2009-12-31 Thread Shashikant Kore (JIRA)
[ https://issues.apache.org/jira/browse/MAHOUT-163?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12795570#action_12795570 ] Shashikant Kore commented on MAHOUT-163: Grant, Yes, it should have been

Re: [math] MAHOUT-235/GenericSorting, looking for a clue

2009-12-31 Thread Grant Ingersoll
+1. The JVM provides a mergeSort, right? Lucene has a quickSort implementation that I believe is pretty fast. On Dec 31, 2009, at 1:09 AM, Ted Dunning wrote: Nuke it if there isn't an obvious need for the code. (my opinion anyway) On Wed, Dec 30, 2009 at 6:10 PM, Benson Margulies

Re: [math] MAHOUT-235/GenericSorting, looking for a clue

2009-12-31 Thread Benson Margulies
It is needed, but on the drive in this morning I thought of a perfectly straightforward way to extend the code in Sorting to do what it does. It's got something that nothing else has: sort (startIndex, endIndex, Comparator, Swapper) but I can see how to add that without understanding the code

[jira] Commented: (MAHOUT-220) Mahout Bayes Code cleanup

2009-12-31 Thread Grant Ingersoll (JIRA)
[ https://issues.apache.org/jira/browse/MAHOUT-220?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12795603#action_12795603 ] Grant Ingersoll commented on MAHOUT-220: Yeah, I don't think Utils should need to

[jira] Commented: (MAHOUT-163) Get (better) cluster labels using Log Likelihood Ratio

2009-12-31 Thread Grant Ingersoll (JIRA)
[ https://issues.apache.org/jira/browse/MAHOUT-163?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12795604#action_12795604 ] Grant Ingersoll commented on MAHOUT-163: Yep, I committed a change w/ those things

[jira] Updated: (MAHOUT-235) GenericSorting.java also needs replacing

2009-12-31 Thread Benson Margulies (JIRA)
[ https://issues.apache.org/jira/browse/MAHOUT-235?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Benson Margulies updated MAHOUT-235: Attachment: oops.diff This doesn't work, and I'm a bit stumped. GenericSorting.java also

[jira] Commented: (MAHOUT-230) Replace org.apache.mahout.math.Sorting with code of clear provenance

2009-12-31 Thread Benson Margulies (JIRA)
[ https://issues.apache.org/jira/browse/MAHOUT-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12795666#action_12795666 ] Benson Margulies commented on MAHOUT-230: - Grant, do you resolve or do I? Replace

[jira] Updated: (MAHOUT-230) Replace org.apache.mahout.math.Sorting with code of clear provenance

2009-12-31 Thread Grant Ingersoll (JIRA)
[ https://issues.apache.org/jira/browse/MAHOUT-230?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Grant Ingersoll updated MAHOUT-230: --- Resolution: Fixed Status: Resolved (was: Patch Available) Anyone can. Replace

[math] I need an extra pair of eyeballs

2009-12-31 Thread Benson Margulies
I attached a patch to MAHOUT-235 that fails its unit test. I haven't spend \hours/ staring at it, but I have spent some time, and I feel like all this quicksort code is running together in my head. If someone is feeling charitable, or wants an opportunity to snicker at my expense ...

[jira] Commented: (MAHOUT-235) GenericSorting.java also needs replacing

2009-12-31 Thread Ted Dunning (JIRA)
[ https://issues.apache.org/jira/browse/MAHOUT-235?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12795704#action_12795704 ] Ted Dunning commented on MAHOUT-235: I just looked at this. Didn't find the problem,

[jira] Issue Comment Edited: (MAHOUT-235) GenericSorting.java also needs replacing

2009-12-31 Thread Ted Dunning (JIRA)
[ https://issues.apache.org/jira/browse/MAHOUT-235?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12795704#action_12795704 ] Ted Dunning edited comment on MAHOUT-235 at 12/31/09 11:24 PM:

[jira] Commented: (MAHOUT-235) GenericSorting.java also needs replacing

2009-12-31 Thread Benson Margulies (JIRA)
[ https://issues.apache.org/jira/browse/MAHOUT-235?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12795707#action_12795707 ] Benson Margulies commented on MAHOUT-235: - the odd thing is that I got here by