Disable midi learn

[d.healey]

@ustk If you enable saveInPreset and you enable MIDI learn then it will save the MIDI learn in preset anyway, no need for an extra option.

Currently when you click the saveInPreset button it triggers this function s->setCanBeMidiLearned(newValue); which makes no sense to me.

[ustk]

@d-healey Sounds like the soil was ready to seed the midiLearn system but it hasn't been done...
Ok for the saveInPreset link ;) Have you had the green light for a pull request?

[d.healey]

@ustk Waiting for our benevolent leader to confirm that this is not a terrible idea before I do it.

[d.healey]

The deed is done:

It's available in my fork - https://github.com/davidhealey/HISE/tree/scriptnode

I've also made a pull request so will be in the main repo soon I expect @Christoph-Hart.

[Natan]

@d-healey Good Job Sir
Can You Implement This For Fx Plugins, It's Only Available In Instruments As Far As I Know!!!

[d.healey]

@Natan Nope, I'm crap at C++, I just hack at it.

[Casey Kolb]

@d-healey Any issues with this yet? I'm tempted to add it to my build too.

[d.healey]

@Lunacy-Audio Nope seems fine, I got the blessing from Christoph in a pm.

[Casey Kolb]

Woot! Awesome, great to know.

[Tania Ghosh]

@d-healey I am getting this error.. Unable to Compile.

[Natan]

In Case You're Doing FX Plugin, This Learn Midi Does No Work In FX!!!

[ustk]

@Natan Not sure it works with midi learn but have you enabled the midi input for FX in the projucer and in the preferences? (dunno if both are necessary though…)

[Natan]

@ustk I Mean Right Clicking Over The Parameters And Assign To Midi Controllers Knob, That Feature Not Works With FX Plugins, Or Do I Need To Change Someting In Projucer?

Edit: I Never Changed Anything Regarding Midi And Midi Commands No Idea

[d.healey]

@Tania-Ghosh Nothing to do with the new changes. Are you compiling plugin or standalone?

[Tania Ghosh]

@d-healey Yes, Standalone...

[d.healey]

@Tania-Ghosh How many copies of the source code do you have on your system?

[Tania Ghosh]

@d-healey 2... One the regular one (sdcriptnode from Hart's Repo) and one from your repo

[d.healey]

@Tania-Ghosh

2...

I would guess that this is the root of the issue. Do you need the disable MIDI learn feature right now? If not I'd wait for Christoph to merge it into the official repo.

[Tania Ghosh]

@d-healey No problem...then I will wait..

[DanH]

@d-healey is it possible to script enableMidiLearn?