Kara-Moon Forum

Developers & Technology => Musical MIDI Accompaniment (MMA) => Topic started by: sciurius on July 17, 2019, 07:10:31 AM



Title: Distortion
Post by: sciurius on July 17, 2019, 07:10:31 AM
MIDI voice 30 is DistortonGuitar. I think the right name is DistortionGuitar  ;) .


Title: Re: Distortion
Post by: bvdp on July 17, 2019, 10:00:14 PM
You are 100% correct. I'm not about to change it at this time though, just too many files around with the wrong spelling. Can we agree that it's a MMA quirk :)

If it really bothers you, add the following line to your initialization file:

  Patch Rename DistortonGuitar=DistortionGuitar

You could also do the same thing with "Patch Set 30=DistortonGuitar" but that will generate a warning.

If anyone has a suggestion for a "better" solution ... do let me know!!! BTW, this bug has been in MMA for about 10 or 15 years :) I, for one, never type in the names ... I just copy/paste from the HTML docs. Yup, I'm a crappy typist.


Title: Re: Distortion
Post by: sciurius on July 18, 2019, 07:32:30 AM
I think it could be as easy as:

In miditables.py, change the name to 'DistortionGuitar' (as it should be).

Add a fix after initializing voiceInx:

   voiceInx['DISTORTONGUITAR'] = voiceInx['DISTORTIONGUITAR'];

Now MMA will recognize DistortonGuitar as an alias for DistortionGuitar, but never show the wrong spelling in Patch List.

Patch attached.


Title: Re: Distortion
Post by: folderol on July 19, 2019, 08:13:57 AM
Will this still work if someone enters the wrong spelling?
People get used to things - even wrong things!


Title: Re: Distortion
Post by: sciurius on July 19, 2019, 08:42:09 AM
Yes, the patch I added in the previous message allows both spellings to be used.


Title: Re: Distortion
Post by: bvdp on July 19, 2019, 04:26:41 PM
Great. I will look at this and implement.


Title: Re: Distortion
Post by: bvdp on July 20, 2019, 12:28:44 AM
This patch has been committed. Will show in the next devel release.

It took a bit of work to figure out the documentation generator for this, but "DistortonGuitar" is gone. I've left it in the library files as an interesting historical artifact :)