Huawei - DEV - Update of example yaml and readme #281
Merged
Conversation
This file contains 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
Error in formula for house_consumption_power_less_aux_non_essential, removed battery from equation.
Error in equation for house_consumption_power, removed battery from equation.
Add round(2) to house_consumption_power_less_aux_non_essential to stop long number of decimals.
Added rounding(2) to house_consumption_power_less_aux_non_essential to stop high number of decimal place output. Re-ordered first 2 sensors in file.
Updated all_gpo_active_power_less_known to gpo_all_active_power_less_known for following naming format
Updated to reflect Lights All and GPO All sensor names
Added sunsynk_power_flow_aux_connected_status that queries the sensor group sensor.sunsynk_power_flow_aux_active_power and if the ouput of that sensor is >0 (showing power being used) it then outputs 'on', else 'off'.
Updated aux_connected_status: to use new binary_sensor.sunsynk_power_flow_aux_connected_status added to sunsynk_power_flow_card_derived_sensors.yaml
removed 'devices' from sensor.sunsynk_power_flow_card_aux_devices_energy_daily name
Added a Max(0) to house_consumption_power_less_aux_non_essential as occassionally this sensor reports negative values, due to waiting for other sensors to update.
REM'ed out the energy_meter for HVAC example. This is optional and if people want they can remove the #'s and update the source sensors to their HVAC Active Power sensor.
Typo - aligned two lines right 1 space
Saving for safety, as need to take a break.
ReST formatting correction
Change shade of battery colour, was too close to grid import.
Switch to using the Huawei sensor for battery shutdown SoC, instead of needing to enter a figure into card yaml
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
Numerous changes including bug fix, and trying to align sensor naming formats, whilst making configuration easier.
Related issues
Motivation and Context
Bug fix and improvements
How has this been tested
Tested on 2 L1 and M1 inverter sites.
Type of change
Please delete options that are not relevant.
Checklist
package.jsonfollowing semver