[Biopython-dev] [Bug 2629] Updated Bio.NaiveBayes to listfns import

[bugzilla-daemon at portal.open-bio.org]

http://bugzilla.open-bio.org/show_bug.cgi?id=2629





------- Comment #7 from bsouthey at gmail.com  2008-11-04 17:47 EST -------
(In reply to comment #6)
> I've committed Peter's fix for the set import to CVS.
> 
> About the replacement for listfns.contents in the modified NaiveBayes code: Did
> you do any timings to compare the new code to the old code? Since
> listfns.contents is implemented in C, it may be (much) faster than the
> replacement code.
> 

(Hopefully I created a patch correctly.)

The purpose of listfns.contents() is to compute the frequency of each class and
return it as a dictionary. There is a difference but it is very small between
the different versions (1/100ths of second) for what I have looked at (which is
more than the actual listfns.contents function).  


-- 
Configure bugmail: http://bugzilla.open-bio.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.