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

Playlist Tempo Bug #1779

Closed
sgrace9 opened this issue May 20, 2023 · 4 comments · Fixed by #1781
Closed

Playlist Tempo Bug #1779

sgrace9 opened this issue May 20, 2023 · 4 comments · Fixed by #1781
Labels

Comments

@sgrace9
Copy link

sgrace9 commented May 20, 2023

Hydrogen version * : 1.2.0
Operating system + version : Arch Linux (current)
Audio driver + version : JACK


When recalling a song from a playlist, the song's tempo is sometimes (randomly?) set to 120 BPM instead of the tempo listed in the song file. This problem started with version 1.2.0.

Steps to reproduce:

  1. Create two or more song files with tempos other than 120 BPM. Create a playlist of these files.

  2. With the playlist active, use the F6/F5 keys to move through the playlist. The tempo will sometimes get set to 120 BPM in error. Moving to a different song and back to problem song once or twice will usually correct the tempo.

The problem can also happen when opening a song file from the Playlist Editor but seems to be more common when using the F6/F5 keys.

@trebmuh trebmuh added the bug label May 21, 2023
theGreatWhiteShark added a commit to theGreatWhiteShark/hydrogen that referenced this issue May 21, 2023
tempo changes applied without an explicit user action, like loading a song, did trigger the `PlayerControl::bpmChanged()` slot of the bpm lcd widget as well. This led to spurious and random tempo changes when switching between different tempi in batch assignment in short time, e.g. switching from a song using 140bpm to one with 180bpm (and resetting the audio engine to default 120bpm inbetween).

fixes hydrogen-music#1779
theGreatWhiteShark added a commit to theGreatWhiteShark/hydrogen that referenced this issue May 21, 2023
tempo changes applied without an explicit user action, like loading a song, did trigger the `PlayerControl::bpmChanged()` slot of the bpm lcd widget as well. This led to spurious and random tempo changes when switching between different tempi in batch assignment in short time, e.g. switching from a song using 140bpm to one with 180bpm (and resetting the audio engine to default 120bpm inbetween).

fixes hydrogen-music#1779
@theGreatWhiteShark
Copy link
Contributor

Hey @sgrace9,

Thanks for reporting! Could you check whether #1781 does fix this issue at your end?

@sgrace9
Copy link
Author

sgrace9 commented May 22, 2023

The description of #1781 seems plausible as a cause of my issue. However, using my existing version 1.2.0 I tried moving through a playlist slowly (waiting several seconds between songs) and still had incorrect song tempos.

If you're asking me to build Hydrogen from source, I'm not set up to do that. However, if you can upload a new binary somewhere I will test it.

@theGreatWhiteShark
Copy link
Contributor

However, using my existing version 1.2.0 I tried moving through a playlist slowly (waiting several seconds between songs) and still had incorrect song tempos.

No, you would need the build the code to try the fixed version.

However, if you can upload a new binary somewhere I will test it.

For Linux we do not provide binaries. Most libraries are dynamically linked and Linux distributions differ way too much to make the resulting binary portable. There may be a portable AppImage of the latest Hydrogen version in the future. But we are not there yet.

Alright, then I'll close this issue as completed for now. In case you experience this problem in the upcoming 1.2.1 release, please reopen this ticket.

theGreatWhiteShark added a commit that referenced this issue May 22, 2023
Fix spurious tempo changes to 120 bpm (#1779)
theGreatWhiteShark added a commit that referenced this issue May 22, 2023
tempo changes applied without an explicit user action, like loading a song, did trigger the `PlayerControl::bpmChanged()` slot of the bpm lcd widget as well. This led to spurious and random tempo changes when switching between different tempi in batch assignment in short time, e.g. switching from a song using 140bpm to one with 180bpm (and resetting the audio engine to default 120bpm inbetween).

fixes #1779
@sgrace9
Copy link
Author

sgrace9 commented Jun 13, 2023

I just installed 1.2.1 and the issue now appears to be fixed. Thanks.

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

Successfully merging a pull request may close this issue.

3 participants