global namespace with requirejs about midi HOT 6 CLOSED

MidiConvert has a UMD header on it, so it should work with AMD and CMD. If you are using requirejs, MidiConvert won't be on the global namespace, you'll have to 'require' it.

How are you currently using or importing MidiConvert into your project?

from midi.

Uncaught ReferenceError: MidiConvert is not defined

requirejs.config({
    //By default load any module IDs from js/lib
    baseUrl: 'js/lib',
    //except, if the module ID starts with "app",
    //load it from the js/app directory. paths
    //config is relative to the baseUrl, and
    //never includes a ".js" extension since
    //the paths config could be for a directory.
    paths: {
        app: '../app',
	jquery: 'jquery-3.1.1.min',
	knob: 'jquery.knob.min',
	tone: 'Tone',
	MidiConvert: 'MidiConvert'
    }
});


// Start the main app logic.
requirejs(['jquery', 'knob', 'tone', 'MidiConvert', 'app/control'],
function   ($, MidiConvert) {});

from midi.

Uncaught TypeError: Cannot read property 'parse' of undefined

from midi.

https://github.com/subunits/Instrument/tree/master

from midi.

Just tested out MidiConvert with requirejs myself and the UMD header worked fine for me.

I think your issue is that your function parameter names don't line up with your include array

requirejs(['jquery', 'knob', 'tone', 'MidiConvert', 'app/control'],
function   ($, knob, Tone, MidiConvert, control) {});

from midi.

Thank you for your time 👍

from midi.