Logic Conditions framework

[DzikuVx]

No description provided.

[rb1205]

Is this intended for convertiplanes / VTOL flight?

[DzikuVx]

@rb1205 VTOL is only one reason for it. This will solve the problem of swapped control axises after conversion from vertical to horizontal flight, but only this one

[hali9]

Do you plan to do the same for mmix?
It would be necessary to do it for mmix for VTOL.

[DzikuVx]

@hali9 yes

[shellixyz]

Otherwise LGTM

[stronnag]

small fyi,

%% logic_condition.c 
./src/main/common/logic_condition.c: In function 'logicConditionUpdateTask':
./src/main/common/logic_condition.c:237:40: warning: unused parameter 'currentTimeUs' [-Wunused-parameter]
 void logicConditionUpdateTask(timeUs_t currentTimeUs) {
                               ~~~~~~~~~^~~~~~~~~~~~~

[stronnag]

I currently (505f44a) have to set

logic 0 1 0 0 0 0 0 0

(whatever that does :) to have working servos. Is this expected ?
Shouldn't the default be "work as legacy".
.. smix .... s/0/-1/ , aha.

[DzikuVx]

yes, -1 with smix. I will check defaults tho

[stronnag]

On March 19, 2019 7:45:04 AM UTC, "Paweł Spychalski" ***@***.***> wrote: yes, `-1` with `smix`. I will check defaults tho -- You are receiving this because you commented. Reply to this email directly or view it on GitHub: #4144 (comment)

Thanks. As I'd just restored a 0,0 diff, user error not checking what had changed. Probably saved me from flying a light, floaty wing in far too much wind.