search

alex-berman / sonotopy

Library for perceptually analyzing an acoustic signal in real time
GNU General Public License v3.0
9 stars 1 forks source link

eliminate sonogram concept #12

Closed alex-berman closed 13 years ago

alex-berman commented 13 years ago

For improved code transparency, it might make sense to get rid of the sonogram concept. The only purpose served by the accumulation of spectra into sonograms is to smoothen the map response. This task could instead be addressed by SonogramMap or SonogramMapCircuit, which could then be named SpectralMap or SpectrotopicMap (-Circuit).

alex-berman commented 13 years ago

Renamed it SpectrumMap. The removed smoothing effect in the response does not seem to be a problem; on the contrary, the faster response seems to be only positive. But if that impression changes, one can increase the spectrum integration time to compensate for the removed smoothing.

The performance test reports ~10% decrease in computation time.

It also looks like TwoDimArray can now be removed.

alex-berman commented 13 years ago

Solved by fc151fff90e80a432fea40bf8c5d48ff76fdc727