Hi David,<div><br class="webkit-block-placeholder"></div><div>I've attached a patch that incorporates your comments (including a rename of FirstBinFreq to FirstBinOffset and updated the doxygen comments).  Still, though, the BarGraph view of the SpectrumMagnitude data doesn't seem to be working-- now it sometimes displays what looks like a fixed array of data but doesn't move (at least on my system, which can run Annotator excruciatingly slow...).  Any suggestions as to what's going on?
</div><div><br class="webkit-block-placeholder"></div><div>B</div><div><br><div><span class="gmail_quote">On 7/5/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="margin:0;margin-left:0.8ex;border-left:1px #ccc solid;padding-left:1ex">Patch reviewed. Almost there. Just two comments:<br><br>- For composing the labels (getLabel) you still use the parent (that's the
<br>frame time not the frequency!), so use the new members. And add the offset.<br><br>- You call several times to project.GetAttributeScheme(scope,name). Make the<br>code faster and more readable by keeping a reference to such object within
<br>that method.<br>const CLAM_Annotator::SchemaAttribute & attribute =<br>project.GetAttributeScheme(scope,name);<br><br>We should try the labels, try to add a label at some point of the bars, just<br>to test it. And make the screenshot relevant.
<br><br><br><br>On Thursday 05 July 2007 19:25:53 bennett kolasinski wrote:<br>> Patch attached with fixes.  Couple questions:<br>><br>> So many errors should have ended in something noticeable but in fact we are
<br>><br>> > not having any error because we are not using them. They should be used<br>> > on PoolFloatArrayDataSource::getLabel, but now it is computing the label<br>> > by returning the string formatting of NBins, instead the string
<br>> > formatting of offset+i*bingap.<br>><br>> I updated PFADS::getLabel but I'm not sure what you mean by offset-- you<br>> mean the starting frequency (which, the more i think about it, might not be
<br>> a necessary attribute at all since you said it's always 0)?  So getLabel<br>> now returns bin*binGap if the bin does not have a label, which should be<br>> the starting frequency of the bin.<br>><br>
><br>> also- PFADS::setCurrentTime uses binLabels but I'm not sure of the purpose<br>> of the function so I'm not sure what to do about removing them.  The code<br>> shouldn't crash but I think a call to setCurrentTime with no binLabels
<br>> assigned will set the _frameData attribute to 0... hmm maybe that is bad.<br>><br>> and finally-- ok, I'm not sure if it's my mac quirks or me still needing to<br>> work out how to use Annotator well... but I can't figure out how to get rid
<br>> of views in Annotator after I add them.  Is there supposed to be a close<br>> widget on them or am I missing something?  And the first bin of the MFCC<br>> bar view when I've been running it always seems to be at its max value,
<br>> which I think is causing the view of the other bins to be scaled and thus<br>> be pretty small.<br>><br>> OK, hope this patch helps...<br>> B<br><br><br></blockquote></div><br></div>