alf45tar / PedalinoMini

Wireless and Bluetooth MIDI Foot Controller
GNU General Public License v3.0
479 stars 84 forks source link

3.0.7 Summary of Firmware Problems #413

Closed bobvc133 closed 1 year ago

bobvc133 commented 1 year ago
  1. Create a new configuration file, the configuration editor will prompt invalid attributes, multiple events will also be confused, and the Press & Release event will be saved as NONE Snipaste_2023-01-07_17-47-01

Snipaste_2023-01-06_22-44-11

  1. After editing the off color of the Release event LED and saving it, the on color will automatically be the same as the off color. Snipaste_2023-01-06_23-01-10 Snipaste_2023-01-06_23-07-02

  2. Except for the Press+Release event and the newly added Press&Release event, other events can only send one signal, and the latch effect cannot be achieved. It is recommended to consider the solution I proposed in https://github.com/alf45tar/PedalinoMini/issues/388#issuecomment-1366016368 Snipaste_2023-01-06_23-08-54 Snipaste_2023-01-06_23-09-03

bobvc133 commented 1 year ago

A little later, I will illustrate the third point of problem improvement suggestions in the form of pictures

bobvc133 commented 1 year ago

With the current features a momentary switch can simulate a latch switch. A latch switch cannot simulate a momentary switch because to RELEASE it you have to press it,Therefore, all functions can be perfectly realized only by using momentary switches. My following instructions are based on the premise that the hardware uses momentary switches.

In daily use, some equipment and software controls require momentary effects, not simple on and off effects.

  1. For example, to switch timbre or bank, sometimes a 0 or 127 Value is needed to trigger, and sometimes two Value of 0 and 127 are needed to trigger at the same time. Although the Press+Release two events and the newly added Press&Release event can achieve momentary effects, they cannot achieve only Send a 0 or 127 Value, always send the Value twice, possibly switch two timbres or BNAK Snipaste_2023-01-07_01-08-42

And other events can only send one signal, not an momentary effect or a latch effect Snipaste_2023-01-07_01-07-59

  1. For example, sending a note signal must be an momentary effect. One event is enough, and two events are not needed to realize it. The NOTE ON and NOTE OFF of the sending list can also be simplified into one NOTE. Snipaste_2023-01-07_01-36-05

  2. For example, sending increasing and decreasing signals must also have an momentary effect, especially in the display state of the LED. When triggered, it should be displayed as flickering, not always on or unable to be set. Snipaste_2023-01-07_01-49-16

Usage scenario: If PedalinoMini needs to control multiple devices, BANK-A01 controls A device, BANK-A02 controls B device, BANK-A03 controls C device... BANK-A0X controls X-device. I need to set the two switches on the right to control BANK+ and BANK-. At this time, the LEDs of the two switches cannot be set. If they can be set, the press or CLICK events before the firmware version 3.0.6, the LED effect is to press Light on, press it again to turn off, the 3.0.7 firmware LED will keep on and cannot be turned off, if you use Press+Release and the newly added Press&Release momentary event, it will switch BNAK twice because of sending two Value, and the LED will also Blinks twice. Snipaste_2023-01-07_03-01-15 Snipaste_2023-01-07_02-59-26

Let me explain how to improve

  1. All events should be changed to the default momentary effect, and then add a HOLD MODE option to simulate the latch effect, so that all events can be freely set whether it is an momentary effect or a latch effect, and the LED display status can also set the effect correctly , or the previous reason, the momentary switch can simulate a latch switch, and the latch switch cannot simulate a momentary switch. The current state completely limits the freedom of setting the momentary switch and most events, but it has no effect on the latch switch.
  2. When sending the Control Change signal under the momentary effect, all events can leave a value blank and only send one value, which solves the problem that different devices or software require sending one or two values when switching timbres or timbre banks. At the same time, the BANK+ and BANK+ of the PedalinoMini pedal can also use the momentary event and LED flashing effect correctly Snipaste_2023-01-07_04-07-50
alf45tar commented 1 year ago

If you want to send multiple MIDI messages for one event use Sequence.

bobvc133 commented 1 year ago

If you want to send multiple MIDI messages for one event use Sequence.

The pictures are only for the convenience of screenshots and comparisons, so they are all displayed on one control, not to send multiple signals that may conflict on one control, and the description of the third point is still being revised.

bobvc133 commented 1 year ago

If you want to send multiple MIDI messages for one event use Sequence.

Regarding the questions and suggestions about the momentary effect, although I have explained it many times in several issues and comments, due to translation problems and cultural differences, I may still not express it clearly. After a variety of devices, software and usage scenarios After repeated testing and referring to the conventional settings of similar MIDI pedals and software, I think that the functions of PedalinoMini can be optimized to be more powerful, and the settings are more abundant and free.The purpose of raising so many issues is just to help test and improve PedalinoMini better. Of course, this is just my personal experience and suggestion, rather than asking to improve PedalinoMini according to my personal ideas. This is the last release about momentary effects issues, I don't want to be your burden.

alf45tar commented 1 year ago

Don't worry. I appreciate any suggestion. Regarding the "hold mode" I think it is already implemented in Step by Step action apart for Bank+ and Bank-.

bobvc133 commented 1 year ago

Don't worry. I appreciate any suggestion. Regarding the "hold mode" I think it is already implemented in Step by Step action apart for Bank+ and Bank-.

Can you give an example of the configuration? The requirement is to need two buttons, one to send BANK+, one to send BANK-, the LED lights up and sends the value when pressed, and the LED turns off when released.

alf45tar commented 1 year ago

At the moment it is not possible because Bank+/Bank- cannot be used in Sequences.

bobvc133 commented 1 year ago

At the moment it is not possible because Bank+/Bank- cannot be used in Sequences.

That's why I have been emphasizing that all events of the momentary switch should default to momentary effects. You can only send one value, then leave a value blank, and add a HOLD MODE function, allowing users to choose whether to use momentary effects or latches according to their needs Effect. because the above Bank+/Bank- requirements cannot be perfectly realized by any event.

bobvc133 commented 1 year ago

For example, the effect software TH-U only needs a value of 0 or 127 to switch the timbre bank or timbre, but PedalinoMini still cannot be perfectly realized. The Press+Release event and the newly added Press&Release event send two values at a time, and switch the timbre twice Libraries and timbres,.although other events can send a value, but the LED will remain on when pressed once. Snipaste_2023-01-08_01-07-17 Snipaste_2023-01-08_01-07-29 Snipaste_2023-01-08_01-18-43

alf45tar commented 1 year ago

The logic of Press&Release is different. Press&Release do not send 2 values at a time. It send one value on Press and one value on Release. One value for each event because between Press and Release can pass any amount of time. It has been created to aggregate 2 events in one action but it do not add any new feature.

It looks like that TH-U is fully configurable. I see the Learn button. Why you are not configuring the Press event to one button for "Previous bank", another one for "Next bank", another one for "Previous preset" and another one for "Next preset"?

bobvc133 commented 1 year ago

Learning operation is not a problem, but after the press event is pressed, the LED will remain on and cannot be turned off. TH-U's bank and preset only need a 0 or a 127, but the Press&Release event is to send a 0 and A 127, you can test this software.

alf45tar commented 1 year ago

Do not use led for these type of actions where it doesn't make sense. At the moment the led flashing is not supported. Forget Press&Release. It is not implemented to flashing leds.

bobvc133 commented 1 year ago

I use a lot of MIDI pedals, don't know why the regular settings don't make sense Snipaste_2023-01-08_01-58-02

https://user-images.githubusercontent.com/91867813/211164304-67e76608-6e1f-4ff6-9918-096a94754f46.mp4

bobvc133 commented 1 year ago

Snipaste_2023-01-08_01-58-02

https://user-images.githubusercontent.com/91867813/211164896-bd335e2b-cc76-4d48-8a0a-a97ba524fd3f.mp4

bobvc133 commented 1 year ago

https://user-images.githubusercontent.com/91867813/211165328-72553c40-653c-4ab2-80ec-93f6c86f80d5.mp4

alf45tar commented 1 year ago

I mean a fixed led color has less sense than other scenario and flashing led for press confirmation is not supported.

bobvc133 commented 1 year ago

Well, I close this question, because of translation errors and cultural differences, this question may not be able to communicate clearly