helm icon indicating copy to clipboard operation
helm copied to clipboard

Arpeggiator doesn't scale with sample rate in Ardour5

Open aeonik opened this issue 7 years ago • 10 comments

Hello, The arpeggiator does not scale properly with different sample rates. The faster the sample rate, the faster Arp notes play.

This can be replicated on my system by:

  1. Creating two test projects in Ardour5 1a. Project 1 sample rate is set to 44.1 KHz 1.b Project 2 sample rate is set to 48 KHz
  2. Create the same MIDI track with 4 whole notes.
  3. I configure Helm Arp to 1/4 quarter note frequency. 4a. Project 1 plays an expected 4 notes. 4b. Project 2 plays 5 notes, off time.

This bug scales with sample rate: the faster the sample rate, the more erroneous notes play. This affects both my Arch and Ubuntu systems. Helm v 0.9.0, Ardour v 5.12.0

Regards, Aeonik

aeonik avatar Sep 19 '18 01:09 aeonik

Might be the same as #182?

community/helm 0.9.0-7 on Arch has that issue, but git does not.

gonX avatar Sep 19 '18 08:09 gonX

Sebastian, It looks like the same issue. Great find!

I can build the Git version on my Arch system tonight to confirm this hypothesis.

On Wed, Sep 19, 2018, 04:39 Sebastian Jensen [email protected] wrote:

Might be the same as #182 https://github.com/mtytel/helm/issues/182?

community/helm 0.9.0-7 on Arch has that issue, but git does not.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/mtytel/helm/issues/209#issuecomment-422711093, or mute the thread https://github.com/notifications/unsubscribe-auth/AAVpsKFavUVyxqzBBZup2w6cy2ir77psks5ucgK2gaJpZM4WvRTU .

aeonik avatar Sep 19 '18 16:09 aeonik

Yeah I haven't released a new build in a while.. I've been busy working on Helm's successor!

mtytel avatar Sep 19 '18 16:09 mtytel

@mtytel Will you keep Helm maintained?

Bleuzen avatar Oct 07 '18 13:10 Bleuzen

Matt, Thanks for sharing the news! I can't wait to see what you have in store. Let us know when it becomes available.

-Aeonik

On Sun, Oct 7, 2018 at 9:29 AM Elias [email protected] wrote:

@mtytel https://github.com/mtytel Will you keep Helm maintained?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/mtytel/helm/issues/209#issuecomment-427653683, or mute the thread https://github.com/notifications/unsubscribe-auth/AAVpsE7EzUCvQT0CEXPf2ZYb7eOWo-aWks5uigGwgaJpZM4WvRTU .

aeonik avatar Oct 19 '18 01:10 aeonik

I think I just hit this bug. I'm always running at 48 kHz - and Arpeggiator is too fast for my session's tempo, even though it correctly got 140 BPM.

unfa avatar Oct 21 '18 16:10 unfa

Just noticed this error too. The arp is slightly faster than the track's tempo on Renoise (Linux)

rburgosnavas avatar May 20 '19 08:05 rburgosnavas

I saw the problem on Ableton Live / Windows 10. Using a sample rate of 44100, the arpeggiator was synced with tempo. When I changed it to 48000, the arp was out of sync. Helm's LFOs, stutter, and sequencer were fine, however.

jpriebe avatar Jun 16 '19 14:06 jpriebe

As I experimented with updating JUCE to 5.4.7 I noticed that this bug seems to be gone from the build using the newer JUCE when tested on 48 kHz project in Ardour. Now the arpeggio is in sync as expected.

xard-dev avatar Oct 18 '20 15:10 xard-dev

The patch is in form of pull request now.

xard-dev avatar Oct 18 '20 20:10 xard-dev