MidiFX Plugin issues in Logic Pro X

ulrik

@Christoph-Hart Ok, well something strange is really happening.
Take a look and a listen to what is happening here when I change selected track.... It seems to add a lot of garbage events, in this recording it lower the pitch as well.
And notice that only the recorded midi is sounding (you can see it in the logg as well)

video

Here's also the snippet I used, very very simple, it only adds an interval to the original played note.

HiseSnippet 1022.3oc6W0saaaCElx1bn1acnEcO.Z8J4sLC4tt1BLTL033LXrlDiorfcWGqHsMajHEnnbq2PA5i1dW1KvdC1NT+XI2pz3YjLfNLegg4gme9NGxyGOdpRFvRRjJjU2SWEyPVeB1ekPuXzBBWflb.x5l3QgLh3HNke3Oi1eULIIgQQVVs+diJVc6fx97me29jPhHfUIBgNSxCXOkGw0URm58C7vvCIT1o7nZZeeuIARwHYnLEfSarKJlDbNYN6XhQsVXj0GMlx0RkulnYIHqN6Koq7WHeoHW+y3I7mGxLKFh7AGkK9PYH0fXiTznE7P5zxzNAgrvSqJBsyKBeF1jtqkWULtU1F1UVTudX0ZS30dC3MrN7bqAuFfjUMH0IGR2F6Gn3w5pcL34iwSDZlZFAJ60gRttnVyaiGIAMD5AQjyYGpfEqsv4Att6YCe0+a60CJ8IZ6kDk8SnzLUVRBserco4yY5QxnXo.V3b2Z5b2JqifZyS4vOdr8XwbtfMHPwfZwQExcLpNKUDn4RgsTbrTyNQ3zu2u0q6QPRAm0C3yEREa7RSXzpTFXR2dcM3RH.+VpF.Gi0GmF8blx32LUVxB2TmyXgx.td0ZM3UYF3uurdxloOILk4jGyxrYPRobgXO6rtiADJs.7C2y1HFh7d1t8ysjOy1Ycf9bSj52qaC9qTmF8Z0lGQf8TxTA0wjfegs6f60uHZutm8aWPmM6Jrh9OndkmChbHr+pr.Ng5rNsmWUFeu0os1Okl9t0AykVkLLzjHWRoXSCMLRE1zrKcDY0G3.2.g0JBsoa16iu3d+5TSA4cW0TTJlH35ShYhKhvBUzRZ3IJPEnpNik3SKXI7C4TlBwoF96ZmYnLXWw4dKOzOM4.hlT5LvuPrhYJM2jFVGvVBL34TPcwGvRNWKiyzsfK.Dukg9UqCq2K7VsdwaHdujS0Kp18W7Vv3yWT6IiIdIZVrO+WY0euIp9yON+dD4U0V5gdWNTf8VRSCI5MozMuiUrAbVuAOpA7hDf.odbux342VHda7TtNXQyXrUCXDNMuNvXwqi2DOd1LVftBfcvu0nAWKg+FWvDJ2I6Q4SR0lKvfPlZ3UwbJwa8bJVsuNlKoyGvykzs7HoooRr9Cr8kLEPBSPOUV3Cms6ctFrYKdR3RhzGVOHrSj4+OQ4+IHJ+QYplKlCCKp3vCgXXPNefgJfAQWHXgl6fVsL2VxW6ZVap.9PGP1h+B9Tr4PyZqhMGVt4+JwHhDnjOKHu6xP5biLIPdKx9ef.0hYs8vxYYvX2AtHy3gOKHvTh+JnIoYat2NXyWuC1b+cvluYGr4A6fMObGr4QuWaLjmOIUKixa4.ASGmQCYYMVPfauY2zQ+M3CeZFN

ulrik

@Christoph-Hart It seems as soon as another track, than the plugin track, is selected, the plugin stops modifying the recorded events, and also jumps in to a "midi through" state.

ulrik

@Christoph-Hart I found this thread in Juce forum:
https://forum.juce.com/t/garbage-midi-data-midi-fx-logic-pro-mac-m1/51774/13

The garbage seems to come out only when Logic runs in Apple silicon processors, I switched my logic to run under Rosetta and that made the garbage go away.

I also found a fix patch from Februari this year, in the Juce code, for this behavior under silicon.
https://github.com/juce-framework/JUCE/commit/d5ad26a162ebde406f685ab36738b878c9b2d4ef

Could we have that fix in Hise as well (the juce code in the Hise code is older than Februari this year I suspect?)

ulrik

@Christoph-Hart I downloaded Juce 7 and then built the ArppegiatorDemo project from the Juce examples, with these settings in Projucer:

Plugin Charasteristics:

Plugin Midi Input
Plugin Midi Output
Midi Effect Plugin

Valid Architectures:

X86_64
arm64
arm64e

I launched it in Logic Pro X under silicon processor M1, and all works as it should.

No garbage midi events logged
The Midi FX Plugin runs and outputs plugin generated events even if I have another track selected than the Midi FX plugin track

See/hear the result here:

VIDEO

The Juce plugin code for this project

/*
  ==============================================================================

   This file is part of the JUCE examples.
   Copyright (c) 2022 - Raw Material Software Limited

   The code included in this file is provided under the terms of the ISC license
   http://www.isc.org/downloads/software-support-policy/isc-license. Permission
   To use, copy, modify, and/or distribute this software for any purpose with or
   without fee is hereby granted provided that the above copyright notice and
   this permission notice appear in all copies.

   THE SOFTWARE IS PROVIDED "AS IS" WITHOUT ANY WARRANTY, AND ALL WARRANTIES,
   WHETHER EXPRESSED OR IMPLIED, INCLUDING MERCHANTABILITY AND FITNESS FOR
   PURPOSE, ARE DISCLAIMED.

  ==============================================================================
*/

/*******************************************************************************
 The block below describes the properties of this PIP. A PIP is a short snippet
 of code that can be read by the Projucer and used to generate a JUCE project.

 BEGIN_JUCE_PIP_METADATA

 name:                  ArpeggiatorPlugin
 version:               1.0.0
 vendor:                JUCE
 website:               http://juce.com
 description:           Arpeggiator audio plugin.

 dependencies:          juce_audio_basics, juce_audio_devices, juce_audio_formats,
                        juce_audio_plugin_client, juce_audio_processors,
                        juce_audio_utils, juce_core, juce_data_structures,
                        juce_events, juce_graphics, juce_gui_basics, juce_gui_extra
 exporters:             xcode_mac, vs2022

 moduleFlags:           JUCE_STRICT_REFCOUNTEDPOINTER=1

 type:                  AudioProcessor
 mainClass:             Arpeggiator

 useLocalCopy:          1

 pluginCharacteristics: pluginWantsMidiIn, pluginProducesMidiOut, pluginIsMidiEffectPlugin

 END_JUCE_PIP_METADATA

*******************************************************************************/

#pragma once


//==============================================================================
class Arpeggiator  : public AudioProcessor
{
public:

    //==============================================================================
    Arpeggiator()
        : AudioProcessor (BusesProperties()) // add no audio buses at all
    {
        addParameter (speed = new AudioParameterFloat ({ "speed", 1 }, "Arpeggiator Speed", 0.0, 1.0, 0.5));
    }

    //==============================================================================
    void prepareToPlay (double sampleRate, int samplesPerBlock) override
    {
        ignoreUnused (samplesPerBlock);

        notes.clear();
        currentNote = 0;
        lastNoteValue = -1;
        time = 0;
        rate = static_cast<float> (sampleRate);
    }

    void releaseResources() override {}

    void processBlock (AudioBuffer<float>& buffer, MidiBuffer& midi) override
    {
        // A pure MIDI plugin shouldn't be provided any audio data
        jassert (buffer.getNumChannels() == 0);

        // however we use the buffer to get timing information
        auto numSamples = buffer.getNumSamples();

        // get note duration
        auto noteDuration = static_cast<int> (std::ceil (rate * 0.25f * (0.1f + (1.0f - (*speed)))));

        for (const auto metadata : midi)
        {
            const auto msg = metadata.getMessage();
            if      (msg.isNoteOn())  notes.add (msg.getNoteNumber());
            else if (msg.isNoteOff()) notes.removeValue (msg.getNoteNumber());
        }

        midi.clear();

        if ((time + numSamples) >= noteDuration)
        {
            auto offset = jmax (0, jmin ((int) (noteDuration - time), numSamples - 1));

            if (lastNoteValue > 0)
            {
                midi.addEvent (MidiMessage::noteOff (1, lastNoteValue), offset);
                lastNoteValue = -1;
            }

            if (notes.size() > 0)
            {
                currentNote = (currentNote + 1) % notes.size();
                lastNoteValue = notes[currentNote];
                midi.addEvent (MidiMessage::noteOn  (1, lastNoteValue, (uint8) 127), offset);
            }

        }

        time = (time + numSamples) % noteDuration;
    }

    using AudioProcessor::processBlock;

    //==============================================================================
    bool isMidiEffect() const override                     { return true; }

    //==============================================================================
    AudioProcessorEditor* createEditor() override          { return new GenericAudioProcessorEditor (*this); }
    bool hasEditor() const override                        { return true; }

    //==============================================================================
    const String getName() const override                  { return "Arpeggiator"; }

    bool acceptsMidi() const override                      { return true; }
    bool producesMidi() const override                     { return true; }
    double getTailLengthSeconds() const override           { return 0; }

    //==============================================================================
    int getNumPrograms() override                          { return 1; }
    int getCurrentProgram() override                       { return 0; }
    void setCurrentProgram (int) override                  {}
    const String getProgramName (int) override             { return "None"; }
    void changeProgramName (int, const String&) override   {}

    //==============================================================================
    void getStateInformation (MemoryBlock& destData) override
    {
        MemoryOutputStream (destData, true).writeFloat (*speed);
    }

    void setStateInformation (const void* data, int sizeInBytes) override
    {
        speed->setValueNotifyingHost (MemoryInputStream (data, static_cast<size_t> (sizeInBytes), false).readFloat());
    }

private:
    //==============================================================================
    AudioParameterFloat* speed;
    int currentNote, lastNoteValue;
    int time;
    float rate;
    SortedSet<int> notes;

    //==============================================================================
    JUCE_DECLARE_NON_COPYABLE_WITH_LEAK_DETECTOR (Arpeggiator)
};

Would it be possible to make Hise compile like this?

ulrik

@Christoph-Hart do you think it's possible to fix this in Hise?
Or do we need to integrate Juce v.7 for this to work?

ulrik

@Christoph-Hart bump...

ulrik

@Christoph-Hart this is a minimal MidiFX example, when any key is pressed a timer will spit out that key every 100ms

NotePnl.setTimerCallback(function()
{
	var ev = Synth.addNoteOn(1, this.data.nn, this.data.vel, 0);
	Synth.noteOffDelayedByEventId(ev, 1000);
});

Still I get all the crazy data from it when using Logic, and if a different track, than the plugin track, is selected, it will not work.
Do you think you have the time to take look and see what is wrong?

HiseSnippet 925.3oc4V0saiSDEdljZfDnRrR7.XsW4JEhRfkEjPUTZRJJZosQjREb0po1GGOpimwxdbVrP7.v6FOL7F.mi+Yi8RVZUD6UDoVkyOey7Mm47MmrJ03CYYlTFevMEI.i+QNqKz1nYQBolsbNieryZYbhBbuAxrryKRDYYP.iy6+cTJ7AGwJ+7mey4BkP6C6bwX2Zj9v2Kik1cdWc1KjJ0Eh.3FYbqre1YK8M5YFkIGoSemIrDg+8hMvUBJsdNL96sHPZMoqsBKjw3GctInXcj4U5p7uUlIuSAjwT1ZbgpbegQEPLl7xlEIUAqZN1YLbUVsqHzupH7INWJCju1+thwGWFvcGh10CdutzqeG5MsM8lzhd6gR8ZQoipnzSbV6mJSr6hP74CcVpsPZn.K6soRUtrd+dOmYFLCscbr3d3hTz30H7d9jIibw+cxWObHV5yrtaEotWYrvJsx8T2Fna.6LSbhQiFdOsN9SIT0eebFXoKyzYBk5N7VyKLW6akFs2IC+0gCnkE1hqXYu0XQP.A7Zs2zQt1HY13.gULVqaasETibIpMnBjlPDFNGThBH37hEaQ1rLvC1Nxc5jxCwug+0rwtFc8dTxfKwZF1JMVtQaRgRrd1zbfV+gCZNF0z.IZS93QmBdUd7cPpGkcmbQR1M4aAkwWZK5jZlUjVUe7lVQzgtuIOCCeLDUF55UUNHhkGuJEnq7W.EYdm3d5oXAa3fV6qIoZa+G6Ic0lZTJJ1CrscAVudDl8ujd5xZ0HrWRg3aRDaj6pNbd6pi1hW+pdvVIZzK0R60If9sIoY0Mt329wkyw6ITRwq8g4k.oVIQA9bXK99Tk.afybH6drhUlac2Ni+91xnG2H+DZPwjAT90kY1uz9Euh1FuRFXi143mOKBjahrs8rs4Xy6ydrp9AkuBcctceZd9e339.hfLPGbiodM7db8iO.l2re5+ecM641CmVXBxUBa2QHzby5.XMnycH81rNCe+n8b0+ylq7Xo3SbVIs9Q6mi81CGoQVuC3X8z3icVDFB91cD7HmK9o2MidY+fI2J0atTXSknv1AejcM9CR7Ab20n1m5A48ntkJ6IjMUAViBjRi+B+TGbJYyqCNsIHKV3mZdoeUmOMu+CJ8fbRW9ScP8MY6NkUpFPbNSFOgEihuW56SG+OEaf2OlO6.v74G.lmc.X9hC.yyO.Le4Af4q9WwPu68s4VSbkb.crZQ4SDb9Bs.6rJ6BY+M2dkKcB

d.healey

@ulrik you should use the synth timer. In a non deferred script.

ulrik

@d-healey yeah but this will make no difference to the issue, still the same garbage events and will still not work if another track is selected

d.healey

@ulrik You've tried the synth timer? A panel timer isn't accurate so I would expect problems with that.

ulrik

@d-healey yes, same issue

ulrik

@Christoph-Hart
Is it possible to modify Hise to compile working MidiFX plugins?
I know that you have been very busy the last days and I don't expect you to make this happen right away, but what do you think, will it be on the roadmap for Hise in a near future?

ulrik

@Christoph-Hart bump

ulrik

@Christoph-Hart bumpelibump

ulrik

@Christoph-Hart I feel I'm a bit nagging but do you have any plans for making Hise compile decent MidiFX plugins?

ulrik

@Christoph-Hart A friendly bump!

Christoph Hart

@ulrik Have you checked other hosts?

Also, what David said. The example is never supposed to work correctly (and the timer might even be deactivated when the UI isn't visible, which is a feature not a bug.

For this kind of stuff you HAVE to use the synth timer callback, this guarantees sample accurate execution.

ulrik

@Christoph-Hart Here's the same snippet as before but using the synth timer

HiseSnippet 840.3oc4V0saSCCE1osAQJLIlDO.QiaRkpJMv3GooIF8OT0naUzwD2M4kbZq0bsqbbJqBwaHOL7F.Gmjslt0oNUwthbQa742Oe7243zWICfnHohX4bx7o.w5o1ClKziaNlxDjtsHVaYOgExFdoqFhzjFymRihfPhkUwOYLwxoDI442enAkSEAvBQDxoRV.7Y1Dldgz9GbHiy6PCgSXSxY8tGzMPJZJ4xXDNEsqSlRCtfNBNhZLqfMw5QsCYZoZflhfgXUpgLb9fwxuKRs+TVD6bNXV3SFfAJUbGIOzfXiTRywLdX+q11QDLJ8WTDJlVDdtcObOes7EEimknvcgG4qGVEVFdEWBd94gW8bvaEPpPNHUJERaaOHPwlpWnwfmmX2UnA0PJV1yCkTaIEdQA6lRzBgt1D5EPGEt3ZO7da85UcweprW4xJXjqPrWx+y.NJYXrHPyjBWo3HoFNV3Uo7OJ6zCyNdnTiMRHUP6YXr8zpX.ChSYGgvce2qLYDnMddT7jyAkmw.LxKq+TfKCX54IZSXd0hzTk1PMTd9uz2.teV18lfY3v6CZXCc8RCpAJwS5q.SE5PXdjWE2822sdkxNWmV4zzrdqLZJgJImazsljtriYwy3CtMobWXFt8SyGMLLqr5WEq7UMU8pt0WTGDo6yV.mNGBaLOIOcC8fYUc8qmbp4fHKRxgZSULDC6nM4am6B+dhjCBLSTNBVCpLFhTrk4s12MuMeaUPJwJmgRQWASe7TPbWMajL1H91W61hpoHY2JSFZ2TPoYFHX0BlgSNRo9N1sfnKviGb1w8ruvIoO83X8p5Jr9ks6ZH2QfH7DYVL7teTv03yMIQ2MU4+nSObdpLLlS0KOj0byRlBrFrzYnY5kHBmXj+lm+YSduuPba69Lcv3UiwBq.ilg5O.XL69psraObHDnW.vR1c91CykSjuHi0LwndTshcIx2v4pCvqrC.L6BAvMbPqBF1R555l0lJv.rAIYwevmLk9l0VYJ8uRIYBMPIOKHk4atQ7wIRPLIR9X.r+1r10mjzMjuNa9ZkyBBVNT2xwWsoN95M0wc2TGeyl53a2TGe2l536Wuil4ieLVKmj11PH852NYThkUaAEYfIrUxewQXqfq

I still have the same problems in Logic, it's working when the instrument track is selected, both playback and record, as soon as I select another track, the MidiFX function stop working. it seems like it gets "bypassed"
That doesn't happen when I run the Juce compiled MidiFX

Here's 2 videos comparing this simple snippet from Hise, with the arpeggiator project compiled in Juce

HISE:
HISE MidiFX

JUCE:
JUCE MidiFX

As you can see the Hise MidiFX is not fully functional but the Juce MidiFX is.
How can I build a fully functional MidiFX in Hise, is there some code in Juce that is not implemented in Hise?
Is it possible to implement it myself (I doubt that :)) If so, can you give a hint?

d.healey

@ulrik I don't know if it will make a difference since the script isn't deferred in your snippet, but you should put this stuff outside of the UI Midi Processor.

ulrik

@d-healey said in MidiFX Plugin issues in Logic Pro X:

@ulrik I don't know if it will make a difference since the script isn't deferred in your snippet, but you should put this stuff outside of the UI Midi Processor.

It made no difference, I moved the stuff to a different script processor and compiled again, same issue

MidiFX Plugin issues in Logic Pro X

37

1.7k

11.7k

102.1k