David (& list):<br><br>I just got back from my trip (no internet for 3 days- the horror!!) and while I was working in my sandbox I noticed that the implementations of TonnetzPlugin and KeyspacePlugin within InstantViewPlugin.cxx
 were virtually the same except for the string id of each class... so I set to work on reworking those two classes into default implementations for all the virtual functions instead of two classes with 99% the same code... I figured this would make adding a plugin (such as BarGraph) as simple as inheriting from InstantViewPlugin and simply passing the id string to the constructor (unless you needed to override any of the functions).
<br><br>Anyhow, I made a big mess and overwrote my backed-up copies of InstantViewPlugin.[c/h]xx so I'm updating my source now.  This was after spending some time working on integrating BarGraph so all is not lost but I'd like to spend a bit of time cleaning up what I've done before sending anything out.
<br><br>So I guess the purpose of this email is twofold: 1) seeing if anyone wanted to argue against changing InstantViewPlugin into default implementations for its virtual functions instead of having to implement each function for each plugin class and 2) to say that I'll be working more on rolling BarGraph into Annotator and will have something to send out shortly :)
<br><br>----------<br>Bennett<br><br><div><span class="gmail_quote">On 6/4/07, <b class="gmail_sendername">David García Garzón</b> <<a href="mailto:dgarcia@iua.upf.edu">dgarcia@iua.upf.edu</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Bennett, if you already did any change to Tonnetz don't update your sandbox or<br>you might get some conflicts. I did some changes to get ride of the Dummy<br>data sources for KeySpace and Tonnetz. If you send the patch to the list ill
<br>try to merge it.<br><br>The changes i did consisted on removing the DummyXXXDataSources  classes on<br>the KeySpace and Tonnetz which provided dummy constant data for qt designer.<br>I used instead the generic dummy data source i already used for the BarGraph,
<br>the DummyFloatArrayDataSource class which is 'programable' with the constant<br>data to show. I just added programmable labels to fit KeySpace and Tonnetz<br>needs.<br><br>When you split the monitor class all the common files will get 'almost'
<br>identical.<br><br>Looking forward your changes, it will be very interesting viewing PCP's and<br>MFCC's as a BarGraph on the Annotator.<br><br>--<br>David García Garzón<br>(Work) dgarcia at iua dot upf anotherdot es
<br><a href="http://www.iua.upf.edu/~dgarcia">http://www.iua.upf.edu/~dgarcia</a><br><br></blockquote></div><br>