diff --git a/packages/midi/midi.mjs b/packages/midi/midi.mjs index 29abf21b..fa86f629 100644 --- a/packages/midi/midi.mjs +++ b/packages/midi/midi.mjs @@ -6,9 +6,8 @@ This program is free software: you can redistribute it and/or modify it under th import * as _WebMidi from 'webmidi'; import { Pattern, isPattern, logger } from '@strudel.cycles/core'; -import { getAudioContext } from '@strudel.cycles/webaudio'; import { noteToMidi } from '@strudel.cycles/core'; - +import { Note } from 'webmidi'; // if you use WebMidi from outside of this package, make sure to import that instance: export const { WebMidi } = _WebMidi; @@ -16,12 +15,28 @@ function supportsMidi() { return typeof navigator.requestMIDIAccess === 'function'; } -export function enableWebMidi(options = {}) { - const { onReady, onConnected, onDisconnected } = options; +function getMidiDeviceNamesString(outputs) { + return outputs.map((o) => `'${o.name}'`).join(' | '); +} +export function enableWebMidi(options = {}) { + const { onReady, onConnected, onDisconnected, onEnabled } = options; + if (WebMidi.enabled) { + return; + } if (!supportsMidi()) { throw new Error('Your Browser does not support WebMIDI.'); } + WebMidi.addListener('connected', () => { + onConnected?.(WebMidi); + }); + WebMidi.addListener('enabled', () => { + onEnabled?.(WebMidi); + }); + // Reacting when a device becomes unavailable + WebMidi.addListener('disconnected', (e) => { + onDisconnected?.(WebMidi, e); + }); return new Promise((resolve, reject) => { if (WebMidi.enabled) { // if already enabled, just resolve WebMidi @@ -32,13 +47,6 @@ export function enableWebMidi(options = {}) { if (err) { reject(err); } - WebMidi.addListener('connected', (e) => { - onConnected?.(WebMidi); - }); - // Reacting when a device becomes unavailable - WebMidi.addListener('disconnected', (e) => { - onDisconnected?.(WebMidi, e); - }); onReady?.(WebMidi); resolve(WebMidi); }); @@ -47,8 +55,6 @@ export function enableWebMidi(options = {}) { // const outputByName = (name: string) => WebMidi.getOutputByName(name); const outputByName = (name) => WebMidi.getOutputByName(name); -let midiReady; - // output?: string | number, outputs: typeof WebMidi.outputs function getDevice(output, outputs) { if (!outputs.length) { @@ -60,29 +66,20 @@ function getDevice(output, outputs) { if (typeof output === 'string') { return outputByName(output); } - return outputs[0]; + // attempt to default to first IAC device if none is specified + const IACOutput = outputs.find((output) => output.name.includes('IAC')); + const device = IACOutput ?? outputs[0]; + if (!device) { + throw new Error( + `🔌 MIDI device '${output ? output : ''}' not found. Use one of ${getMidiDeviceNamesString(WebMidi.outputs)}`, + ); + } + + return IACOutput ?? outputs[0]; } // Pattern.prototype.midi = function (output: string | number, channel = 1) { Pattern.prototype.midi = function (output) { - if (!supportsMidi()) { - throw new Error(`🎹 WebMidi is not enabled. Supported Browsers: https://caniuse.com/?search=webmidi`); - } - /* await */ enableWebMidi({ - onConnected: ({ outputs }) => - logger(`Midi device connected! Available: ${outputs.map((o) => `'${o.name}'`).join(', ')}`), - onDisconnected: ({ outputs }) => - logger(`Midi device disconnected! Available: ${outputs.map((o) => `'${o.name}'`).join(', ')}`), - onReady: ({ outputs }) => { - const device = getDevice(output, outputs); - const otherOutputs = outputs - .filter((o) => o.name !== device.name) - .map((o) => `'${o.name}'`) - .join(' | '); - midiReady = true; - logger(`Midi connected! Using "${device.name}". ${otherOutputs ? `Also available: ${otherOutputs}` : ''}`); - }, - }); if (isPattern(output)) { throw new Error( `.midi does not accept Pattern input. Make sure to pass device name with single quotes. Example: .midi('${ @@ -90,35 +87,43 @@ Pattern.prototype.midi = function (output) { }')`, ); } - return this.onTrigger((time, hap) => { - if (!midiReady) { + + enableWebMidi({ + onEnabled: ({ outputs }) => { + const device = getDevice(output, outputs); + const otherOutputs = outputs.filter((o) => o.name !== device.name); + logger( + `Midi enabled! Using "${device.name}". ${ + otherOutputs?.length ? `Also available: ${getMidiDeviceNamesString(otherOutputs)}` : '' + }`, + ); + }, + onDisconnected: ({ outputs }) => + logger(`Midi device disconnected! Available: ${getMidiDeviceNamesString(outputs)}`), + }); + + return this.onTrigger((time, hap, currentTime, cps) => { + if (!WebMidi.enabled) { return; } const device = getDevice(output, WebMidi.outputs); - if (!device) { - throw new Error( - `🔌 MIDI device '${output ? output : ''}' not found. Use one of ${WebMidi.outputs - .map((o) => `'${o.name}'`) - .join(' | ')}`, - ); - } hap.ensureObjectValue(); - // calculate time - const timingOffset = WebMidi.time - getAudioContext().getOutputTimestamp().contextTime * 1000; - time = time * 1000 + timingOffset; + const offset = (time - currentTime) * 1000; + // passing a string with a +num into the webmidi api adds an offset to the current time https://webmidijs.org/api/classes/Output + const timeOffsetString = `+${offset}`; // destructure value const { note, nrpnn, nrpv, ccn, ccv, midichan = 1 } = hap.value; const velocity = hap.context?.velocity ?? 0.9; // TODO: refactor velocity - const duration = hap.duration.valueOf() * 1000 - 5; + // note off messages will often a few ms arrive late, try to prevent glitching by subtracting from the duration length + const duration = Math.floor(hap.duration.valueOf() * 1000 - 10); if (note != null) { const midiNumber = typeof note === 'number' ? note : noteToMidi(note); - device.playNote(midiNumber, midichan, { - time, - duration, - attack: velocity, + const midiNote = new Note(midiNumber, { attack: velocity, duration }); + device.playNote(midiNote, midichan, { + time: timeOffsetString, }); } if (ccv && ccn) { @@ -129,7 +134,7 @@ Pattern.prototype.midi = function (output) { throw new Error('expected ccn to be a number or a string'); } const scaled = Math.round(ccv * 127); - device.sendControlChange(ccn, scaled, midichan, { time }); + device.sendControlChange(ccn, scaled, midichan, { time: timeOffsetString }); } }); };