2.1.1.114 RM 3 not evaluating conditions properly


#1

@bravenel

Bruce,

EDIT! Bruce, I think the problem is with the evaluation of simple conditions in my actions, not the rule evaluation as I initially suspected, please disregard for now. Thanks, @Ryan780 and @eric.dalquist

While 2.1.1.114 does appear to fix the problem I reported in 2.1.1.112 about RM 3 not subscribing to presence states correctly, there does appear to be a new issue with this build.

I have created an example rule and attached screenshots of logging and application states. Jist of the problem is that it appears conditions are only evaluating a single event even if there are supposed to be several events considered (AND). I haven't confirmed it yet, but it appears this may affect more than just conditions but also conditional actions.

As you can see from the logging, the rule is only evaluating the state or first variable and not the mode, motion or other variable states. Excuse the error in the second condition of the rule, but I don't think it makes any difference for what I'm trying to show (the third condition should have been mode is sleep not sleep mode deactivate vMotion active).







#2

But it doesn't have to evaluate the second. If the first statement of an AND expression if false, then the whole thing is false, correct? Why bother even checking the rest of the conditions? It appears to me that the overall rule is always evaluated correctly as false, is it not?


#3

No in this example the rule isn't even considering the state of the motion detector, mode, or other variables, it seems to only be evaluating on the state of the variable "Time-Of-Day". Since I'm using "AND" in the rule it should be using all the states within the sub rule as part of the condition.

I have MANY rules that are no longer working because they are evaluating incorrectly after installing 2.1.1.114.


#4

Might be related to [BUG] Simple Conditions always executing


#5

That does sound very similar. Good find Eric!


#6

Okay...but do you have an example of one that wasn't evaluated correctly? Because this rule appears to be evaluating correctly.

because that isn't the first part of any of your statements. If the first part of an AND is false, the whole thing is false, correct? Why would it need to even consider the rest? Move the motion sensor up in the statement to be the first one and see how the logs change.


#7

When thinking if I should respond.......It's NOT COMPLICATED to understand when you see this....

tag


#8

Ryan I think you are correct, the heart of the matter is the bug that @eric.dalquist linked to, it's not the rule evaluation that's the problem, but rather the actions I have setup which do use simple conditions which are executing regardless of their truth.

The way my rules are setup it appeared my rules weren't evaluating correctly and I guess I haven't enabled the logging recently to see how the rules are evaluated.

So yes, if any condition of a group of conditions is false there is no reason for the rest of the conditions to be evaluated!


#9

Really? @waynespringer79, just making sure you heard that one.


#10

A broken clock is right twice a day.....

douche


#11

You two should get a room...


#12

LMAO....at least I admit when I'm wrong.


#13

I have similar problems with Rule Machine. Also, I cannot add any new Zone Motion Controllers, and no new zwave devices.

dev:28492019-06-09 02:16:47.895 pm [error]
(http://192.168.5.181/device/edit/2849)java.lang.NullPointerException: null (configure)

I see a lot of Java Null errors in the logs