[Clam-devel] Re: Turnaround

Sun Aug 10 05:06:15 PDT 2008

Here's a new patch. PolarChromaPeaks displays some data but I don't
think it's correct. The code contains a lot of debug statements so I'm
not committing it yet.

Pawel

On 10/08/2008, Pawel Bartkiewicz <tuuresairon at gmail.com> wrote:
> Hi,
>
> We made some progress in developing Turnaround with David on Friday
> and you can see two new screenshots on the wiki.
>
> Today I tried to add support for PolarChromaPeaks but it doesn't work
> as expected (the dial is always on G and it doesn't move). I haven't
> managed to fix it so I'm attaching the patch if anyone is interested.
> I'll try again in the morning.
>
> Pawel
>
>
> On 08/08/2008, David García Garzón <dgarcia at iua.upf.edu> wrote:
>> I must say i am the guilty one. I just syncronized the qt4.py tool to
>> qt.py
>> original tool in the current version for SCons in ubuntu hardy, so i
>> introduced some SCons modernities in the script. Sorry for the problems
>> this
>> could have caused you.
>>
>> On Divendres 08 Agost 2008, Pawel Bartkiewicz wrote:
>>> Hi,
>>>
>>> Yes, after installing scons 0.98.5 the problem was gone. I haven't
>>> tried any other versions (except 0.97), 0.98.5 was the newest in
>>> Gentoo portage so I tried it first.
>>> Maybe there should be some warning or scons version check?
>>>
>>> Pawel
>>>
>>> On 07/08/2008, Han, Yushen <yushen.han at gmail.com> wrote:
>>> > Hi, David and Pawel
>>> >
>>> > I have the "no attribute 'AddMethod' " problem when I am compiling NE.
>>> > All the .py files listed below did exist in the right dir. But I don't
>>> > understand
>>> > "AttributeError: SConsEnvironment instance has no attribute
>>> > 'AddMethod':"
>>> >
>>> > Pawel, did you say that the problem was gone after you " had to update
>>> > scons 0.97 to 0.98.5 "?
>>> >
>>> > I am still using 0.97 and am going to update it first...
>>> >
>>> > Best regards,
>>> > Han, Yushen
>>> >
>>> > 156-56-219-141:NetworkEditor yushen$ sudo scons install
>>> > prefix=/usr/local clam_prefix=/usr/local QTDIR=/opt/local
>>> > scons: Reading SConscript files ...
>>> >
>>> > scons: warning: QTDIR variable is not defined, using moc executable as
>>> > a hint (QTDIR=/opt/local)
>>> > File "/usr/local/share/clam/sconstools/qt4.py", line 208, in _detect
>>> > AttributeError: SConsEnvironment instance has no attribute 'AddMethod':
>>> >   File "/Users/yushen/CLAM/clam/NetworkEditor/SConstruct", line 44:
>>> >     env.Tool('qt4', toolpath=[clam_sconstoolspath])
>>> >   File "/opt/local/lib/scons-0.97/SCons/Environment.py", line 1340:
>>> >     tool(self)
>>> >   File "/opt/local/lib/scons-0.97/SCons/Tool/__init__.py", line 157:
>>> >     apply(self.generate, ( env, ) + args, kw)
>>> >   File "/usr/local/share/clam/sconstools/qt4.py", line 390:
>>> >     env.AddMethod(enable_modules, "EnableQt4Modules")
>>> >
>>> >
>>> > On Tue, Aug 5, 2008 at 8:23 AM, Pawel Bartkiewicz
>>> > <tuuresairon at gmail.com>
>>> >
>>> > wrote:
>>> >> Ok, it took me some time to understand, but here's a patch. I'm not
>>> >> committing it, because it's not complete yet. It replaces the realtime
>>> >> analysis with a VectorView displaying the time code. I had to change
>>> >> AudioFileMemoryLoader a bit.
>>> >>
>>> >> BTW, I had to update scons 0.97 to 0.98.5 (masked as unstable in
>>> >> Gentoo) because one of the recent commits added the following error in
>>> >> sconstools/qt4.py when building Turnaround:
>>> >> "SConsEnvironment instance has no attribute 'AddMethod'"
>>> >>
>>> >> On 02/08/2008, David García Garzón <dgarcia at iua.upf.edu> wrote:
>>> >>> Pawel, I did some changes on the PoolDataArraysource in order to easy
>>> >>> your
>>> >>> task, so update your repository before going on.
>>> >>>
>>> >>> El Saturday 02 August 2008 00:06:23 David García Garzón va escriure:
>>> >>>> Back to the list.
>>> >>>>
>>> >>>> We'll take it step by step.
>>> >>>>
>>> >>>> First clearance:
>>> >>>> - Do not use toStdString to get file names. Use
>>> >>>> toLocal8Bit().constData()
>>> >>>> in order to support current locale. toStdString uses Latin1 despite
>>> >>>> the current one. :-(
>>> >>>> - Remove the nameProject parameter unless you are using it for
>>> >>>> anything. If
>>> >>>> you want to provide a song as parameter, rename it and don't forget
>>> >>>> to
>>> >>>> use
>>> >>>> the fromLocal8Bit.
>>> >>>>
>>> >>>> Then the actual forward steps. As you have the widgets that make
>>> >>>> more
>>> >>>> noise
>>> >>>> than any other thing if you want the analysis, let's forget about
>>> >>>> the
>>> >>>> analysis and lets concetrate first on the widgets.
>>> >>>>
>>> >>>> The first phase is having a FloatArrayDataSource that allows to
>>> >>>> travel
>>> >>>> on
>>> >>>> an extracted pool of dummy data.
>>> >>>>
>>> >>>> First lets create and fill the dummy data on loading the file.
>>> >>>> * Create an array of floats of nFrames*12 slots. By now let's take
>>> >>>> the
>>> >>>> hop
>>> >>>> size from a constant and the lenght from the audio size.
>>> >>>> * Fill the array with a 12 bits binary time code 0, 1, 10, 11, 100,
>>> >>>> ... loop
>>> >>>>
>>> >>>> Then we should create a view for such data.
>>> >>>> * Substitute the KeySpace with a VectorView, changing the port to
>>> >>>> PCP
>>> >>>> which
>>> >>>> is simpler to debug than ChordCorrelation port (12 bins vs. a
>>> >>>> varying
>>> >>>> number of chords depending on the configuration).
>>> >>>> * Copy annotator's PoolFloatArrayDataSource.
>>> >>>>
>>> >>>> Instead of relying on the pools and schemas we want to provide the
>>> >>>> dummy parameters by hand, so:
>>> >>>> * Comment out the setDataSource and the UpdateData notice that they
>>> >>>> use the
>>> >>>> Schema (structure) and the DataPool (actual data) to fill some
>>> >>>> members. * Provide a setDataSource with nBins, binGap,
>>> >>>> firstBinOffset
>>> >>>> and binLabels
>>> >>>> as parameters  so you can do direct mapping. (those bins are bins in
>>> >>>> a
>>> >>>> frame, pitches, so binGap=firstBinOffset=0 and binLabels are
>>> >>>> G,G#,A,A#,B...
>>> >>>> * Provide an UpdateData with a sampling rate, a frame division, and
>>> >>>> the array of data that you will copy into _data
>>> >>>> * Create a PFADS on the Turnarround
>>> >>>> * Set the PFADS as the VectorView data source
>>> >>>> * Connect the progress control to a Turnaround slot that calls
>>> >>>> PFADS::updateTime with the appropiate parameter.
>>> >>>> * Try to use a Tonnetz instead of a VectorView to check the labels
>>> >>>>
>>> >>>> At this point we'll have the widgets visualizing offline dummy data,
>>> >>>> so you
>>> >>>> can remove the TonalAnalysis and monitors from the real-time
>>> >>>> network.
>>> >>>> Then
>>> >>>> start building an offline network that fills the offline structures,
>>> >>>> our widgets know how to display.
>>> >>>>
>>> >>>> El Friday 01 August 2008 21:17:33 Pawel Bartkiewicz va escriure:
>>> >>>> > Yes, I was reading mail and working on Turnaround from Monday, but
>>> >>>> > I
>>> >>>> > still can't make the offline analysis working, so I haven't sent
>>> >>>> > any
>>> >>>> > new patches.
>>> >>>> >
>>> >>>> > I forgot to commit that old patch, sorry. I've just done it but I
>>> >>>> > had to revert and clean up the new code because it wasn't working.
>>> >>>> >
>>> >>>> > I'm afraid I'll have to write some hack or simply copy the code
>>> >>>> > from
>>> >>>> > Annotator's ChordExtractor, because I'm stuck and otherwise I may
>>> >>>> > not finish the project on time.
>>> >>>> >
>>> >>>> > On 01/08/2008, David García Garzón <dgarcia at iua.upf.edu> wrote:
>>> >>
>>> >> _______________________________________________
>>> >> Clam-devel mailing list
>>> >> Clam-devel at llistes.projectes.lafarga.org
>>> >> https://llistes.projectes.lafarga.org/cgi-bin/mailman/listinfo/clam-deve
>>> >>l
>>> >
>>> > _______________________________________________
>>> > Clam-devel mailing list
>>> > Clam-devel at llistes.projectes.lafarga.org
>>> > https://llistes.projectes.lafarga.org/cgi-bin/mailman/listinfo/clam-devel
>>
>>
>>
>> --
>> David García Garzón
>> (Work) dgarcia at iua dot upf anotherdot es
>> http://www.iua.upf.edu/~dgarcia
>>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Turnaround-ChromaPeaks_2.patch
Type: text/x-diff
Size: 19599 bytes
Desc: not available
URL: <http://lists.clam-project.org/pipermail/clam-devel-clam-project.org/attachments/20080810/c4c7d09e/attachment-0003.patch>