[Clam-devel] formant extractor [PATCH]es

Xavier Amatriain xavier at create.ucsb.edu
Wed Jul 25 04:34:40 PDT 2007


abe kazemzadeh wrote:
> One thing that I wanted to get feedback on for a possible future 
> refactoring of the root solving functionality.  Right now it's 
> implemented as a function call, but the more I get familiar with clam 
> methodology, it seems like it would be very clam-ish/clammy  to do it 
> as a processing, where the data in would be a polynomial and the 
> output would be an array of roots.  However, maybe this is overkill 
> and it makes more sense to just keep it as a member function.  
Hi Abe, I do think that in the end it will make sense to have this as a 
Processing class. However this is not an urgent refactoring.
I would focus on adding functionalities first (like we have been 
discussing offlist) and then refactor when the whole system is up
and running.

And congrats for the patches!




More information about the clam-devel mailing list