[Clam-devel] Re: test changes after refactoring of InstantTunningEstimator

Roman Goj roman.goj at gmail.com
Tue Jul 10 06:22:15 PDT 2007


Step 3:

I thought this could be a seperate patch, but no - it still includes
the two previous ones :( (I changed 1.5/3.0 I did in the last patch to
0.5, obviously should have done this from the beginning... didn't
notice 1.5 isn't like 0.1 produces something nice when divided by 3 ;)
)

but this patch addresses also:

the last two tests in InstantTunningEstimator have been changed to use
the std::vector interface of InstantTunningEstimator's doIt and also
to use binsPerSemitone=1 (I decided against doing it in two seperate
patches ;) )

Maybe I'll wait now for some feedback from David (maybe I should've
sent these to your e-mail, David, rather then the list? Sorry if
anyone feels offended by the e-mail flood in this topic!), before
doing the next three steps.

cheers,
roman
-------------- next part --------------
A non-text attachment was scrubbed...
Name: InstantTunningEstimatorTest_step3_stdvector_in_last_two_tests.patch
Type: text/x-diff
Size: 5577 bytes
Desc: not available
URL: <http://lists.clam-project.org/pipermail/clam-devel-clam-project.org/attachments/20070710/09ba6e25/attachment-0005.patch>


More information about the clam-devel mailing list