tracktion_engine
tracktion_engine copied to clipboard
Tracktion
Fix for misplaced click emphasis of firstBeatOfBar
Summary: Changed bool isFirstBeatOfBar from beatTime getWholeBeats() to beat with a modulus of the time signature numerator.
Reasoning: Periodically I observed the click emphasis on the first beat of a bar being misplaced. ie. the second beat or last beat of a bar were emphasised instead of the expected first beat.
The issue seems to originate from getWholeBeats() not being equal to 0 when expected.
In general this issue presented more often with bpm values with lots of decimal places. eg 157.1396273744468 at 4/4 timing.
As of now I have not managed to reproduce the same problem in testing my adjusted code.
Can you provide a bit more info on why it was failing? What was the value of getWholeBeats() when the problem occurred?
Also, are you able to provide a failing test case? I'd like to add this to catch any potential regressions in the future. Cheers!
I think this has been fixed on the feature/launcher branch as a side effect of some other changes. If you still manage to see it, can you reopen this or open a new issue please? Thanks.