MIDI file generated by MidiFileGenerator is unusable

[kyledecot]

Is there an existing issue for this?

• I have searched the existing issues

Current Behavior

Attempting to load the .mid file generated by code similar that found at Generating Midi Files via MidiFileGenerator into a 3rd party synthesizer yields an error.

For example, attempting to load the midi file into FluidSynth yields fluidsynth: error: Unrecognized MIDI event:

Attempting to load the midi file into Online Sequencer yields Could not parse MIDI file: Invalid event: first byte must be greater than or equal to 0x80:

Finally, attempting to load the midi file into Guitar Pro will load the file however the notes are incorrect and the instrument has changed from Guitar to Piano.

Expected Behavior

Generated .mid file should play in 3rd party synthesizers without issue.

Steps To Reproduce

# NodeJS Application

export class MidiService implements IMidiService {
  async generate(buffer: Buffer): Promise<Uint8Array> {
    const settings = await this.createSettings();
    const score = await this.createScore(buffer, settings);
    const midiFile = await this.createMidiFile();
    const handler = await this.createMidiFileHandler(midiFile);
    const generator = await this.createMidiGenerator(score, null, handler);

    generator.generate();

    return midiFile.toBinary();
  }

const midiService = new MidiService();
const response = await fetch(URL);
const arrayBuffer = await response.arrayBuffer();
const buffer = Buffer.from(arrayBuffer);
const generatedMidi = await this.midiService.generate(buffer);

writeFileSync(PATH, generatedMidi);

Link to jsFiddle, CodePen, Project

No response

Found in Version

1.3-alpha

Platform

Node.js

Environment

- **OS**: MacOS 12.5
- **Browser**: Chrome
- **.net Version**: N/A

Anything else?

sample.zip

[Danielku15]

Generally the writing seems to be fine beside one key problem. I think the bytes in this method have to be reversed:

alphaTab/src/midi/MidiEvent.ts

Lines 128 to 132 in 43cbd1f

	let b: Uint8Array = new Uint8Array([
	(this.message >> 24) & 0xff,
	(this.message >> 16) & 0xff,
	(this.message >> 8) & 0xff,
	this.message & 0xff

[Danielku15]

I had a further look. I think alphaTab cannot provide a midi file export to external software at this point. Reason behind this is: alphaTab already uses the "4.2.11 MIDI 2.0 Pitch Bend Message" from the MIDI 2.0 specification to provide bends on an individual note level instead of midi channel wide pitch bends.

Unfortunately the MIDI Associateion did not publish any proper specification on how to write MIDI 2.0 files (SMF2) leaving us with a gap there. Due to this also hardly any software supports midi 2.0 events.

https://www.midi.org/midi/forum/7958-midi-files-and-midi-2-0

I could try to restore at least some basics on the SMF1 export to work as expected, but I am not sure if this would actually help you because it would not be the file you likely need to bring it further e.g. into Fluidsynth.

[Danielku15]

Seems there is a bit progress on the TMA side and hopefully soon this month there will be a first presentation about the SMF2 fileformat https://www.midi.org/midi/forum/16368-midi-2-0-specs

[allandiego]

I am using this lib to import/export/edit midi files, it may help to solve this issue, it has a option to import a midi from a json, I am thinking of converting the alphatab Score object to this json format to export the midi, the opposite should work too to import midis
#150

https://github.com/Tonejs/Midi

[bisubus]

Generally the writing seems to be fine beside one key problem. I think the bytes in this method have to be reversed:

alphaTab/src/midi/MidiEvent.ts

Lines 128 to 132 in 43cbd1f

	let b: Uint8Array = new Uint8Array([
	(this.message >> 24) & 0xff,
	(this.message >> 16) & 0xff,
	(this.message >> 8) & 0xff,
	this.message & 0xff

I quickly checked the output and reversing the bytes in this place alone doesn't help, this still results in malformed midi. I'm interested in mid file IO and would be happy to collaborate on the issue. Was it working at some point, can this be tracked to the earlier commits? I guess a fallback for per-note bends is essential, SMF1 will be there for a long time. This is how GP midi export works after all.