RE: Smart Home: Let the Party Begin

mcb1 — Tue, 22 May 2018 21:53:59 GMT

Looks great.

the actual value sent to the device is 255,0,22. I Am not sure the reason

I'm not the software expert here, but I suspect the error might be here.

[embed:dc8ab71f-3b98-42d9-b0f6-e21e02a0f8e2:3ef06e6b-9b7d-4651-b12d-db8be7f5e991:type=c_cpp&text=String+intval+%3D+msg.substring%28found+%2B1%2C+sizeof%28msg%29%29%3B++%0A++Gval+%3D+intval.substring%280%2C+intval.indexOf%28%22%3A%22%29%29%3B++%0A++Bval+%3D+intval.substring%28intval.indexOf%28%22%3A%22%29+%2B+1%2C+sizeof%28msg%29%29%3B]

You've created a new string (intval) which has a new length, but you're still using the length of the old string (msg) to determine the end point.

I suspect it should read

[embed:dc8ab71f-3b98-42d9-b0f6-e21e02a0f8e2:3ef06e6b-9b7d-4651-b12d-db8be7f5e991:type=c_cpp&text=Bval+%3D+intval.substring%28intval.indexOf%28%22%3A%22%29+%2B+1%2C+sizeof%28intval%29%29%3B]

The values passed to the Pixels.Color() are all multiplied by 2.55 as the RGB values received from Openhab are in the range of 0-100

You may want to look at the command 'map'. https://www.arduino.cc/reference/en/language/functions/math/map/

it allows a smoother adjustment of 0-100 to 0-255 (or any other range you wish)

Mark

RE: Smart Home: Let the Party Begin

genebren — Tue, 22 May 2018 18:06:00 GMT

Nice update on your lighting system. This looks like a fun little project.

Keep up the good work!

Gene