RealTimeSequencer drifts out of sync when tempo events occur

[peppinodoesmidi]

so while i was making deathdeservioli midi from pizza tower i tested it with midis2jam2 but when the bpm slide started it was going so slow and the instruments weren’t playing in the right place or time so i hope you can fix this

[wyskoj]

@peppinodoesmidi Please provide a MIDI file that causes the issue you describe. (You will need to ZIP it before uploading because GitHub does not allow MIDI files to be uploaded.)

[saverofthewos]

This happens with any midi with a BPM slide, for example [Black MIDI] RUSH E (Fully RECREATED).zip plays too fast.

[peppinodoesmidi]

oh ok i'll try to do taht

[peppinodoesmidi]

what i meant was there was a bpm slide glitch deservioli.mid.zip idk if you can download the zip or not but i hope this works

[wyskoj]

I've boiled this down to a bug in the OpenJDK RealTimeSequencer implementation of Sequencer. Although it's an upstream issue, I'm going to attempt to fork it and implement my own fix.

[wyskoj]

This happens with any midi with a BPM slide, for example [Black MIDI] RUSH E (Fully RECREATED).zip plays too fast.

@saverofthewos That MIDI file doesn't have any tempo changes.

[saverofthewos]

This happens with any midi with a BPM slide, for example [Black MIDI] RUSH E (Fully RECREATED).zip plays too fast.

@saverofthewos That MIDI file doesn't have any tempo changes.

It did when I downloaded it

[wyskoj]

@saverofthewos Here's a CSV readout of that MIDI file.

[Black MIDI] RUSH E (Fully RECREATED).csv

If you filter the event column to only show tempo events, you can see that there is only one event.

[saverofthewos]

That's weird. The website I downloaded it from had a tempo slide.

[peppinodoesmidi]

wait but i used fl studio for the tempo slide for the midi

[wyskoj]

wait but i used fl studio for the tempo slide for the midi

The issue doesn't lie with the software package used to create the MIDI file.

I'm working on a fix.

[peppinodoesmidi]

oh ok

[wyskoj]

This required a complete overhaul of the existing sequencer. Fix will be published in the next version.

[peppinodoesmidi]

k