Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Array out of bounds nel calcolo della roughness #14

Open
francescocretti opened this issue Jan 8, 2019 · 1 comment
Open

Array out of bounds nel calcolo della roughness #14

francescocretti opened this issue Jan 8, 2019 · 1 comment
Assignees
Labels
audio Audio management, features, ecc... bug Da sistemare...

Comments

@francescocretti
Copy link
Member

francescocretti commented Jan 8, 2019

C'è un errore nel metodo calcRoughness() in class_timbre.pde, riga 500.

Temporaneamente fixato da @Screeen nel commit 50f6a2 aggiungendo il controllo peak < 100 qui:

    if (largerThanPrevious && largerThanNext && largerThanNoiseFloor && peak < 100 ) 
    {
     peak++;
     peakValue[ peak - 1 ] = FFTcoeffs[i];
     freqValue[ peak - 1 ] = centerFreqHz(i);
    }

ma andrebbe verificato qual'è il vero problema e influisce sul calcolo della roughness.

@AndryFish hai scritto tu la funzione.

Any ideas?

@francescocretti francescocretti added bug Da sistemare... audio Audio management, features, ecc... labels Jan 8, 2019
@AndryFish
Copy link
Contributor

AndryFish commented Jan 9, 2019

Ciao a tutti. In merito al metodo calcRoughness e ai suoi problemi riporto quanto discusso con @francescocretti.
Per calcolare la Roughness è necessario estrarre dei valori di picco dal segnale.
Per recuperare i picchi si è scelto di utilizzare la stessa tecnica usata in calcSpectralComplexity() . L'errore che appare si deve al fatto che per qualche motivo i picchi continuano ad essere rilevati fino a superare la capienza dell'array.
Questo problema è sorto un paio di volte ma ponendo 100 come dimensione tutto andava a bene.
Non capisco proprio perchè è venuto fuori quell'errore. I'm sorry.

@francescocretti francescocretti added bug Da sistemare... audio Audio management, features, ecc... and removed audio Audio management, features, ecc... bug Da sistemare... labels Jan 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
audio Audio management, features, ecc... bug Da sistemare...
Projects
None yet
Development

No branches or pull requests

3 participants