Skip to content
This repository has been archived by the owner on Oct 4, 2023. It is now read-only.

Onset Detector 2 is currently disabled because not so good #1

Open
mikelange49 opened this issue Nov 3, 2020 · 8 comments
Open

Onset Detector 2 is currently disabled because not so good #1

mikelange49 opened this issue Nov 3, 2020 · 8 comments

Comments

@mikelange49
Copy link

Hi Eleonore.

Congrats for your hard work, looks very impressive !

I'm a plugin developer and I'm interessed in the time-based method from Luca Turchet's "Hard Real-Time Onset Detection of Percussive Sounds".

I'd like to know why you disabled it ("because not so good").

Thanks.
Mickael

@EleonoreMizo
Copy link
Owner

Hi mikelange49,

Actually I’m quite disappointed with all the onset detectors I tried so far (there are a few other ones I didn’t commit). They miss a stroke here and there, or emit false positives. I haven’t tried much the spectral-based ones because of the additional latency and much higher CPU load. Only the first OnsetDetect in the Pedale Vite effect list gives almost usable results. Maybe the other algorithms require an accurate tuning to give their best, but this is problematic as I want to use them with a few different instruments (guitars and bass). Or perhaps they are designed for drums or general-purpose use, and not specifically for plucked string instruments. I tried the time-based Luca Truchet’s algorithm (not the spectral part, I don’t need the exact onset location at the moment) with the numerical constants given in the article and tried to refine the settings, but couldn’t find something working really better than the first OnsetDetect. Maybe I should give it another try.

@mikelange49
Copy link
Author

mikelange49 commented Nov 8, 2020 via email

@mikelange49
Copy link
Author

mikelange49 commented Nov 20, 2020 via email

@EleonoreMizo
Copy link
Owner

Hmm… I think it comes from the VST wrapper. I think I haven’t tested it with effects using only control signals as output yet. I bet it will give you the same issue with pv_osdet. I’ll check this and try to fix it soon.

@EleonoreMizo
Copy link
Owner

Fixed in 672fb47

@mikelange49
Copy link
Author

mikelange49 commented Nov 22, 2020 via email

@EleonoreMizo
Copy link
Owner

Oui je le suis :) Cet article, que je n’ai que survolé pour l’instant, m’a l’air intéressant, merci beaucoup. Cependant je ne sais pas du tout à quel point il est adapté à du temps réel puisqu’il semble faire la détection dans le domaine spectral. En tout cas, je mets ça sur le coin de la table et je m’y plongerai en détail un peu plus tard, quand j’aurai un moment à y consacrer.

@mikelange49
Copy link
Author

mikelange49 commented Dec 1, 2020 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants