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

[toggledbits]

Sounds like you might have dimming.minimum on the entity set to 0.1? And what of dimming.maximum? Failing that, does the device itself have min/max set in its Z-Wave configuration?

[vezinpi]

Hi,

The dimming.maximum/minimum are set to 1 and 0

@toggledbits said in Gradually turn on lights.:

Failing that, does the device itself have min/max set in its Z-Wave configuration?

Not sure exactely how to check this. The device is a Zooz ZEN 72. I look in the Z-Wave JS UI and could not find any parameter that would limit the dimming min/max.

I might be wrong here but it is my understanding that the Z-Wave JS UI sits between HA and the devices. Since the Z-Wave JS UI log shows that any level change request above 60 from MSR, stays at 60, it can only be coming from MSR or HA. In addition the dimming.set works without any issues.

[toggledbits]

OK. There are some rounding issues with JavaScript, and I think I can tune things up to work around them a little better. Stay tuned.

Also, tip: use the Copy Attributes button instead of screen-grabbing the attributes. Gives more data.

[toggledbits]

Yup, a lot going on here. ZWave and ZWave-JS produce values for brightness in the range 0-99. HA maps that into its 0-255 scale, so precision oddities are introduced there. HassController then attempts to scale HA's 0-255 into Reactor's 0-1, which introduces more precision oddities.

I've made an interim build with a handful of adjustments. It is published only for docker aarch64/arm64 (RPi 64-bit and similar) right now, because I don't have time to repackage the whole thing for all platforms right now (I'll be unavailable from 11th through 20th), and it's too risky. For others reading, I don't recommend going to this build unless you need this specific fix immediately.

[vezinpi]

@toggledbits Thank you for looking into this.

There is absolutely no rush on any of this on my side. When I read the post I really liked the approach you suggested and made a test rule to see how I could use it and got into some issues.

Since you took the time to make an interim built, I will take the time to test it and report back here with the result. This way, you will know the status when you return.

Thanks again

[vezinpi]

Loaded the new built and as expected, the dimming.up/down are working for all levels. I also noticed that the dimming.level reported are nicely round up to one digit after the "." except for 50% where it displays "0.51" (not really an isssue just reporting what I see)

However, the other issue mentionned my opening post (that got kind of forgotten) is still present. Here is the refference:

said in Gradually turn on lights.:

1- The "dimming.step" when changed does not stick after a dimming.up/down

In the "Entities" menu I set the attribute "dimming.step" to 0.05. Still in the etities menu, I then perform a "dimming.up" and can confirm that indeed the "dimming.level" went up by 0.05 however at the same time the dimming.step goes back to 0.1 (default). From that point, any subsequent "dimming.up" uses the default.

Again, this can wait until your next release if a fix is indeed needed.

[toggledbits]

Have you done a hard refresh on the browser? If that's not it, check file permissions on your storage directory, and also check the logs to see if any errors are emitted when you change the attribute value and within the window of about two minutes after (allowing for the delayed-write strategy caching). The other likely issue is that your storage directory has issues and isn't properly saving the state data for the entity.

Edit: Well, I said:

There is no code that changes dimming.step other than the button you use on the UI.

And that's technically true, there's no code, but... seems there is a stray directive in the entity mapping data table hass_devices.yaml, which is specific to HassController, left over from wayyyyy long ago that should not be there today, and will do what you are observing. Simple fix. I'll roll a new docker image shortly.

Edit 2: Updated container is up and ready.

[vezinpi]

As usual, you nailed it!

With a dimming.step of 0.03, it does a very nice gradation.

Many thanks

[tbully]

I'm having this same issue with a bare metal install. I assume this fix will be included in a future release? (No hurry. This is just a nice-to-have thing...)

[toggledbits]

@tbully said in Gradually turn on lights.:

I'm having this same issue with a bare metal install. I assume this fix will be included in a future release? (No hurry. This is just a nice-to-have thing...)

The 25208 build released yesterday includes the fixes in the temporary containers built for @vezinpi and others on this thread.