When a track with old BPM data is rescanned after it is loaded, the auto-sync value can get crazy

[mixxxbot]

Reported by: xeruf
Date: 2018-12-16T15:32:04Z
Status: Fix Released
Importance: High
Launchpad Issue: lp1808698
Attachments: [Mixxx Wrong BPM.zip](https://bugs.launchpad.net/bugs/1808698/+attachment/5225389/+files/Mixxx Wrong BPM.zip)

---

Easily reproducible:

1. Choose two songs with the same BPM
2. Load the first one into a deck and enable auto-sync on both decks
3. For the second one, reset "BPM and Beatgrid" and then manually edit the BPM to be half of what it was before
4. Load the second one into the other deck and watch auto-sync going insane.

-> Auto-sync should recalibrate itself whenever the BPM of a track is changed in any way, instead of blindly changing the speed.

Where this happens in real-life scenarios: When I have the option to "Re-analyze beats when settings change or data is outdated" option enabled and the BPM was embedded and not scanned by Mixxx, it will first assume the embedded BPM, then rescan and go mad if auto-sync is enabled. The worst thing is that THIS WILL SOMETIMES EVEN AFFECT THE TRACK PLAYING IN ANOTHER DECK! Its speed is doubled or halved, and a whole mix can be ruined by this error!

I use Mixxx-2.2.0-rc on Linux

[mixxxbot]

Commented by: daschuer
Date: 2018-12-16T21:49:21Z

---

I cannot reproduce any unexpected behavior.

What means: "watch auto-sync going insane"
What exactly happens with the tack, bpm display and rate slider. What should happen instead?

[mixxxbot]

Commented by: xeruf
Date: 2018-12-16T22:31:08Z

---

Oh, did you have the option "Re-analyze beats when settings change or data is outdated" enabled?

The Tracks will be sped up to surreal speeds, up to 720 BPM, depending on how exactly you execute it.

[mixxxbot]

Commented by: daschuer
Date: 2018-12-16T23:00:03Z

---

Does it (still) happen with the latest 2.2 Version? We have merged a related PR #1924
Does this change the situation?

[mixxxbot]

Commented by: xeruf
Date: 2018-12-17T15:53:19Z

---

I use the mixxxbetas ppa and update my computer daily. Yes, it still happens.

I have now created an unlisted video on YouTube, showing all steps to reproduce this issue: https://youtu.be/1U43L02-vNE

[mixxxbot]

Commented by: daschuer
Date: 2018-12-17T20:56:03Z

---

The somehow random jump on the first sync press on the right deck is caused by syncing to a paused sampler. This is fixed here #1818 waiting for a review.

[mixxxbot]

Commented by: xeruf
Date: 2018-12-18T07:16:10Z

---

Great! But obviously, the other bug is much more important, since it can completely mess up the playing track simply by loading another one.

[mixxxbot]

Commented by: daschuer
Date: 2018-12-22T15:36:43Z

---

I think I got the original issue.
Unfortunately the fix breakers test around sync master.
How can we solve this.
If you set a deck explicite as sync master (no skins has controls for it yet, so the question is only theory) and you change the beatgrid of this master track. What should happen?
I consider this a user error. But Mixxx should respond reasonable. Currently the new speed instantly adopted leading to a speed jump on all folowers. This is probably undesired behaviour.

I propose to disable the master state in this case. The user can enable master middle again afterwards.

[mixxxbot]

Commented by: xeruf
Date: 2018-12-22T19:49:36Z

---

I think it should keep master BPM as is and instead behave as if it was reloading the track with the new BPM.

[mixxxbot]

Commented by: daschuer
Date: 2018-12-23T00:39:47Z

---

a fix can be tested here: #1955

[mixxxbot]

Commented by: daschuer
Date: 2018-12-23T00:42:52Z

---

Now the tempo of the edited Deck changes to match the tempo of the other deck.
It the edited Deck is "sync_mode" = 2 (Master, hidden feature) the edited deck plays with unchanged Tempo but the followers are adjusted.
I hope that is reasonable.

[mixxxbot]

Commented by: geozubuntu
Date: 2018-12-27T19:42:48Z

---

In win 7 x86 and
Linux Mint 19.1 x64
with the 2.3.0-alpha-pre (build master r6662)

It happens to me even if I don't change the BPM. If I load the track a 2nd or 3rd time it behaves like that sometimes higher sometimes lower BPM. Check the zip file below.
I tested it with the same track from different albums to be able to check the waveforms as well and compare the sound. See following zip file with pictures from Linux Mint 19 you can check settings and screensots. Check the 2 first tracks and both decks. It changed the deck one to 136 from 170 with no reason.

Keep up the good work.
Best regards

[mixxxbot]

Commented by: geozubuntu
Date: 2018-12-27T19:43:16Z
Attachments: [Mixxx Wrong BPM.zip](https://bugs.launchpad.net/mixxx/+bug/1808698/+attachment/5225389/+files/Mixxx Wrong BPM.zip)

[mixxxbot]

Commented by: daschuer
Date: 2018-12-27T20:36:40Z

---

#1955 should fix the issue.
Are you able to compile the PR branch?

[mixxxbot]

Commented by: geozubuntu
Date: 2018-12-28T00:12:29Z

---

@daniel Schurman
Unfortunately I don't know how to compile and I don't have the environment installed. I am a tech person not a programmer. I just download the latest master build and install. Is there somewhere an installer containing these fixes so I can install and check ? If yes I can check it.

[mixxxbot]

Issue closed with status Fix Released.