Q Light Controller+

• Flash Override Priority - This adds a higher priority to the generic fader (Universe::Flashing). This priority is between Override and SimpleDesk. So, flash buttons will override sliders and also running chasers. We could also consider adding the possibility to choose the priority instead of toggling it to not override sliders but override running chasers.

• Flash Force LTP - This simply forces the flash scene channels to behave like LTP channels. This will override intensity channels also.

Hi, thanks for this.
Here's my comments:

• can you please point me to the relevant discussions that led you to submit this PR?

• QLC+ does have a "central" place where to store settings (see all QSettings used) but they're not save into projects

• I need to know why this has to be a global setting rather than per-button one. As far as I know, flashing is needed only with buttons. I don't see a reason to use it elsewhere

• I'm not sure a new type of state is needed to stay before Simple Desk. A few years ago I added priorities to faders. Maybe that's enough to achieve priority flashing

• in general, such big changes should be discussed (e.g. in the development section of the forum) before submitting a PR to save time to you to write an already approved code and to me so that I don't have to review ideas not discussed and without the proper knowledge of the code base

Thanks

Hi,

• primarily, this was implemented due to personal needs and i thought it would be helpful for others. That's why I submitted a PR.

• Alright, i did not Knew the QSettings.

• This does not have to be a global setting. I also thought about adding the possibility to the button itself. But that would add a decision to the user, which does not exactly know what this is for. And if a user really needs this, he can easily activate it. For all other users it would stay the same, and they do not have to make any further decision. I can change this with pleasure. I thought, normally, this decision would be a decision for the whole show file and not for each button.

• I also don't think a new state is really needed. But what if later on in the development of QLC+ some functions also have the override priority set but should not override flash scenes. I just added it so that further implementation does not break this functionality. But yeah, the existing states would be enough to achieve this.

• Of course. I literally forgot about the possibility to create a discussion in the forum and thought the discussion could be held here. I will add a forum discussion with pleasure if you want to.

Thanks

Massive thanks for sharing this.

This feature would be invaluable to my club shows and would open up a brand new way to program "bumps."

My thoughts:

• I can see why it would be useful to have buttons with both Override priority on and off for this reason I believe it would be best to have it available to select per button.

• This said, the "global" setting would be nice to have as a default for new buttons added to the console

• For the forum discussion: is there a reason you can think of to have a priority level selector? Would the user like a flash button to be an override over other flashbuttons?

[...]

For the forum discussion: is there a reason you can think of to have a priority level selector? Would the user like a flash button to be an override over other flashbuttons?

Wazzledi wrote: ↑Fri Oct 27, 2023 12:00 pm Alright, I removed the VC properties solution and only left the per-button solution in the implementation.

@Wazzledi What is the difference between override and forceLTP now? I think one option could be enough for users since it's clear they want to override everything else when flashing?

Let me explain the difference.

ForceLTP:
This option just adds the FadeChannel::ForceLTP Flag to the created FadeChannel for the GenericFader.
This results in the new handling inside the write function for Flashing FadeChannels. When a FadeChannel is Flashing then it sets the forceLTP parameter of the Universe::write(int channel, uchar value, bool forceLTP) method according to whether the FadeChannel::ForceLTP flag is set or not.
This results in the universe handling the specific value as a LTP value.

OverridePriority:
As you might know, the universe has a priority list of GenericFaders which values should be written to the output.
For example: It exists GenericFader a and GenericFader b both have the same priority and a is added before b. Then, the value of b would override the value of channel a (assuming the value is LTP), because b is handled after a. This is normal behavior, but for flashing we want to prioritize this value. For that, we add a new priority so that the Flashing channels would be handled after the normal ones.
This happens for example when a chaser is running, and we want to flash a button. When the next step of the chaser is added, this overrides the flash button without this option.
Right now, the Universe::Flashing priority is greater than the Universe::Override priority, which results in Flash buttons overriding virtual console faders.

I could imagine that the override priority configuration might be unnecessary or too much. But I think some users might not want to override some faders with the flash button. Maybe we can change the name and the handling to override faders.
Therefore, we would add a priority before Universe::Override and after. This will result that we would override everything per default except faders, and if we tick the option we also override faders.

In higher level:
ForceLTP forces the channels of the Flash button to be handled as LTP values. With this we can also flash dimmer values which are lower than the current dimmer values.

OverridePriority forces the handling of the channels to be later than other running functions. This will override virtual console faders and functions triggered after triggering the flash button.

GGGss wrote: ↑Tue Nov 07, 2023 11:12 pm I'm willing to misuse a showcase to this purpose... Let us give this version a try?