-
-
Notifications
You must be signed in to change notification settings - Fork 107
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
custom variables #1423
Comments
I see, That's quite clever. How about instead of using custom entities to store variables we could create a new type called Variables. Maybe limit to 10 max, as strings with a default value that is stored in LittleFS and loaded at boot. |
Something like:
@MichaelDvP thoughts? |
Comments to:
|
excellent, so we only need to implement #1 ! |
@MichaelDvP my idea with Variables was to be able to use them in Values For example if I created a custom entity called 'foo' I could use this in the Scheduler as a value like |
The value is in ram and can be accessed. The typical example to set the boiler flowtemp without thermostat.
|
I didn't realize that, I need to look at the code again. I thought WebSchedulerService::command() just called the command. But cool! |
tested, works great. I'll add some notes to the doc. |
From a discussion with Michael:
The scheduler have entity/command to activate/deactivate the schedule. This was for replace the
forceheatingoff
, sendingselflowtemp 0
every minute to stop the boiler, and disable scheduler to activate the boiler again.Maybe it's a nice addition to also manipulate the schedule command value via command. But difficult to add in the command structure.
I wonder if we can add custom RAM entities only stored in ems-esp and use them for setting a variable value. E.g. in custom use devcie/telegram type 0x0 for storing 256 bytes. Custom entity:
myseltemp, device 0, type 0, offset 0, UINT
, Scheduler:every minute, command: boiler/seltemp, value: custom/myseltemp
. Then the flowtemp can be set via mqtt: custom/myseltemp and is used by the scheduler.Originally posted by @MichaelDvP in #1418 (comment)
The text was updated successfully, but these errors were encountered: