Skip to content
  • Categories
  • Recent
  • Tags
  • Popular
  • Unsolved
Collapse
Discussion Forum to share and further the development of home control and automation, independent of platforms.

Global Moderators

Forum wide moderators

Private

Posts


  • [Reactor] Variables not updating correctly in latest-25201-2aa18550
    toggledbitsT toggledbits

    @3rdStng said in [Reactor] Variables not updating correctly in latest-25201-2aa18550:

    EDIT: Updated to 25260 and I'm still not seeing the HumidityRise expression change. I kept my global expressions since they update and refresh on the page, where the local expressons don't appear to do so within the rule. Watching these two expressions, as soon as the prevHumidity sensor updates, the expression turns green. But the Rise expression didn't and still showed 0. In this screen grab, the humidity went from 46 to 45. I would have expected the top expression to show a -1.

    OK. So let's start with the Global Expressions. The rise is always going to be zero, because it will evaluate prevHumidity to the current value of the sensor whenever it changes, and then cause HumidityRise to be re-evaluated subtracting the current value of the sensor taken from the sensor via getEntity() with the current value of the sensor stored in prevHumidity, and that will always be zero. This is as it ever was.

    Rule-based expressions have changed and now operate the same way on the recent builds, so the result is going to be the same. The difference will always be zero. And when I look at your "old" way, I'm kind of surprised that worked on the older builds, too. The fact that it may have is, to me, an indicator of the kind of problems reported on this thread, a problem which you just happened to use to your advantage previously.

    In working on a new approach for you, I also found another use case not handled correctly, so you'll need to update to 25261 to continue with the instructions below.

    Here's a new approach going forward (requires build 25261) :

    First, tie the Rule's trigger condition to any change in the device entity's humidity_sensor.value attribute. Like this:

    bfa7a03f-f9ac-4512-9ad8-2c9c37c56ab0-image.png

    Tip: because we're using the changes operator, when the entity's humidity value changes, the trigger condition (and therefore the Rule) will issue a very quick pulse that activates the SET reaction. That pulse is often too quick to see in the UI (its milliseconds), so setting the condition (in condition options) for pulse output with a delay of a few seconds (e.g. 5) will delay the reset so you can more clearly see the distinct phases of handling in the UI. In this case, that delay makes no impact operationally, it's just for your convenience visually.

    Then, the new set of local expressions:

    45720c19-889d-42c1-9799-049fd33bb6bf-image.png

    Only humidity_rise has an expression. In text (for copy-paste), it is:

        getEntity( "mqtt>shelly_handt3" ).attributes.humidity_sensor.value - humidity_last
    

    The humidity_last and rise_last variables are expressionless, used as storage for the Rule's logic.

    First operating principle: when an entity referred to by a Rule-based expression or trigger condition is modified, the rule is re-evaluated, and that process begins with three steps: (1) re-evaluate its Rule-based expressions (recompute them); (2) evaluate and update the state of all trigger conditions; (3) set the Rule state from trigger conditions result. This is how Rules have always worked, and they still work this way.

    Finally, the SET reaction for this rule would look like this:

    b7e5f40d-6f53-434d-b507-683e86e697ab-image.png

    That script in text for copy-paste is:

    humidity_last = getEntity( "mqtt>shelly_handt3" ).attributes.humidity_sensor.value,
    rise_last = humidity_rise
    

    The conditional group in that Reaction is where you turn on your dehumidifer/exhaust fan, etc. The Script Action updates the local humidity_last and rise_last variables. The rise_last variable is new, and I'll explain why I chose to have two rise variables below.

    Theory of operation: whenever the device reports a new humidity value, the difference variable humidity_rise is calculated and then the trigger condition is evaluated and will be true (because the change in humidity caused the Rule to self-evaluate), so the SET reaction runs. In the SET reaction, if the rise is >= 3%, the actions inside the conditional group will be run. To finish, the Script action stores the last humidity and rise in their respective variables. Because it does this unconditionally (outside the conditional group), the last humidity is always recorded and the rule will therefore handle rising and falling humidity values correctly.

    Why add rise_last? When the trigger condition's pulse time expires (either from your condition options or the native timing of the changes pulse), the Rule is re-evaluated, and again, the first step of that is to evaluate its rule-based variables (i.e. first operating principle in effect). This is going to cause humidity_rise to go to 0, because the SET reaction made humidity_last match the entity's current humidity value (therefore the difference is now zero). I added the rise_last variable, the value of which is set only during the SET reaction, to store the humidity rise from that phase of evaluation, so you can see what it was. It is normal and you should expect humidity_rise to be 0 except when the SET reaction is running; it's now a scratch variable. The rise_last variable is your go-to reference for the last difference that was or was not acted upon by the Rule.

    Also... your original expressions wrapped attribute values in int(), which truncates them (e.g. int(2.99) results in 2, but it's really almost 3). You might want to use round() instead (e.g. round(2.2) is 2, and round(2.99) is 3).

    And finally, you might consider looking at the time series capabilities of VirtualEntityController. That can directly produce a change rate value (using the rate aggregator) that makes it pretty easy to determine when there's a sudden upward change in humidity.

    Multi-System Reactor

  • [Reactor] Variables not updating correctly in latest-25201-2aa18550
    toggledbitsT toggledbits

    Build 25261 now available for 64-bit docker installs only. Fixes an issue with Rule expressions being evaluated too frequently (but the pendulum seems to be settling toward the middle).

    Multi-System Reactor

  • [Reactor] Variables not updating correctly in latest-25201-2aa18550
    toggledbitsT toggledbits

    New build 25260. Also included fixes and enhancements for the expression editor.

    Again, this build is intended only for the limited audience of this thread to test this specific issue. While the build at this point has had considerable testing, it's still not quite ready for consumption by all (but will be soon, if this goes well).

    Multi-System Reactor

  • [Reactor] Variables not updating correctly in latest-25201-2aa18550
    toggledbitsT toggledbits

    OK. More testing done. I've hit a condition where the dependency scan in the last build still isn't hitting absolutely everything. Another build coming soon...

    Multi-System Reactor

  • [Reactor] Variables not updating correctly in latest-25201-2aa18550
    therealdbT therealdb

    So far, so good. Everything seems to be OK atm. I'll report in case of problems.

    Multi-System Reactor

  • [Reactor] Variables not updating correctly in latest-25201-2aa18550
    toggledbitsT toggledbits

    @therealdb said in [Reactor] Variables not updating correctly in latest-25201-2aa18550:

    OMG, very old rules, isn't it? Thanks, I fixed it.

    This particular effort has made me look at rules that I haven't touched since the earliest days as well, many of them almost untouched copies from the Vera importer. It's a trip down memory lane, for sure, and a follow-on "wow, there's such a better way to do that now".

    How's the latest working out so far? @tunnus have you had a chance to play with it as well?

    Multi-System Reactor

  • [Reactor] Variables not updating correctly in latest-25201-2aa18550
    therealdbT therealdb

    @toggledbits said in [Reactor] Variables not updating correctly in latest-25201-2aa18550:

    Just a tip, a more efficient way to write this:

    OMG, very old rules, isn't it? Thanks, I fixed it.

    Multi-System Reactor

  • [Reactor] Variables not updating correctly in latest-25201-2aa18550
    toggledbitsT toggledbits

    @therealdb said in [Reactor] Variables not updating correctly in latest-25201-2aa18550:

    [latest-25248]2025-09-11T07:19:37.603Z Rule:ERR Full expression: t = time(), m = dateparts(t).month, d = dateparts(t).day,
    (m == 12 && d == 24) ||
    (m == 12 && d == 25) ||
    (m == 12 && d == 31) ||
    (m == 1 && d == 1) ||
    (m == 1 && d == 5) ||
    (m == 1 && d == 6)

    Just a tip, a more efficient way to write this:

    parts = dateparts(), m=parts.month, d=parts.day,
        (m == 12 && d == 24) ||
        (m == 12 && d == 25) ||
        (m == 12 && d == 31) ||
        (m == 1 && d == 1) ||
        (m == 1 && d == 5) ||
        (m == 1 && d == 6)
    

    The default time for dateparts() is the current time, so you don't need d, and by saving dateparts() to parts, the function is only invoked once. We old-timers love to save every cycle we squeeze out of the processor! 🙂

    Multi-System Reactor

  • [Reactor] Variables not updating correctly in latest-25201-2aa18550
    toggledbitsT toggledbits

    New build latest-25254 posted. This should address the local variable in rule-based expressions at startup issue. Docker 64-bit platforms only.

    Multi-System Reactor

  • [Reactor] Variables not updating correctly in latest-25201-2aa18550
    therealdbT therealdb

    yep, it seems to be related (in general) to variables not being evaluated at startup. Here's a screenshot from a daily rule that it's setting a couple of things, including a virtual binary sensor for Christmas from a local variable:

    image.png After re-setting the rule, the expression got re-evaluated and the error went away. @toggledbits

    EDIT. a similar one + logs

    This one, instead, every time is evaluated gives the error.

    e1f0057d-0f11-48c6-89a4-0760325798af-image.png

    [latest-25248]2025-09-11T07:19:37.602Z <Rule:ERR> Lights-Christmas-Off (rule-kwv6wdpw in Lights-Outdoor): error evaluating expression specialdays: [Error] Object does not exist [-]
    [latest-25248]2025-09-11T07:19:37.603Z <Rule:ERR> Full expression: t = time(), m = dateparts(t).month, d = dateparts(t).day,
    (m == 12 && d == 24) ||
    (m == 12 && d == 25) ||
    (m == 12 && d == 31) ||
    (m == 1 && d == 1) ||
    (m == 1 && d == 5) ||
    (m == 1 && d == 6)
    [latest-25248]2025-09-11T07:19:37.604Z <Rule:CRIT> Error: Object does not exist [-]
    Error: Object does not exist
        at GlobalExpression.getInstance (/opt/reactor/server/lib/GlobalExpression.js:160:27)
        at _0x1ec9e7._func.<computed> [as _ref] (/opt/reactor/server/lib/Rule.js:848:69)
        at _resolve (/opt/reactor/common/lexp.js:1385:27)
        at _run (/opt/reactor/common/lexp.js:1429:31)
        at /opt/reactor/common/lexp.js:1697:33
        at Array.forEach (<anonymous>)
        at _run (/opt/reactor/common/lexp.js:1696:28)
        at _run (/opt/reactor/common/lexp.js:1604:33)
        at _run (/opt/reactor/common/lexp.js:1438:34)
        at /opt/reactor/common/lexp.js:1425:29
    [latest-25248]2025-09-11T07:19:37.627Z <Rule:INFO> Lights-Christmas-Off (rule-kwv6wdpw in Lights-Outdoor) evaluated; rule state transition from NULL to RESET!
    [latest-25248]2025-09-11T07:19:37.638Z <Rule:INFO> Lights-Christmas-Off (rule-kwv6wdpw in Lights-Outdoor) started
    
    Multi-System Reactor

Member List

CatmanV2C CatmanV2
therealdbT therealdb
toggledbitsT toggledbits
akbooerA akbooer
DesTD DesT
rafale77R rafale77
  • Login

  • Don't have an account? Register

  • Login or register to search.
  • First post
    Last post
0
  • Categories
  • Recent
  • Tags
  • Popular
  • Unsolved