This repository was archived by the owner on May 11, 2025. It is now read-only.
merge main changes #1
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
implemented feature to set static ip instead of dhcp
Handle different library calls for rebooting ESP32 vs ESP8266
Fix null string on conversion
Do not write the attributes to the json buffer if OneATTR is defined
Add a MQTT message at beginnig and end of a loop to be able to use it as trigger channel in e.g. openHAB when OneATTR is being defined. It is meant as dispatcher message when dealing with calcultions of e.g. the cop which is dependent on many different values.
Remove the loop trigger since one can use the main attr message to indicate a loop end.
Some minor improvements
Support for NodeMCUv2/ESP-12E/ESP8266 board
Also changed some code style for consistency and readability
This reverts commit 3af927a.
Restart device if no WiFi is available for 60 secs
Basic implementation of Rotex/Daikin S protocol
Align unsigned char usage, to ensure proper comparison results. Fix #165.
Fix CRC computation
Make ha remember switch state across reboots via mqtt retain flag.
Home assistant remember state
Fix a link to Daikin protocol documentation
Negative numbers will be added to json as numeric type instead of string. And only if minus sign is the first character, since converter might also return "---", which is not numeric. And there is no need to check all of the remaining characters in string, if alpha condition is met. Breaking out of the loop immediately.
Its showing now as a select entity on HA, on the same device as the rest of the entities. The payload needed to trigger each mode is unchanged, the conversion is handled by ha using templates. ToDo: (in a new PR) S4S (energy meter pulse output) signal generation, so smart grid is fully operational.
FIX: Smart grid operating mode MQTT discovery.
Negative values as numeric type
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
8 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
merge main changes