-
Notifications
You must be signed in to change notification settings - Fork 3
Decoding of Battery Type and Error Code; small fixes; CT Info and Settings #4
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
base: main
Are you sure you want to change the base?
Changes from 1 commit
d281dd5
9ef7bf7
9332284
4b0b520
8d9232d
b62c00f
d51f8a4
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,6 @@ | ||
binary_sensor: | ||
- platform: modbus_controller | ||
modbus_controller_id: ${modbus_controller_id} | ||
modbus_controller_id: ${modbus_controller_id_hf} | ||
skip_updates: 2 | ||
name: "${entities_name_prefix}-AC INV Relay" # bit 0 | ||
id: "${entities_id_prefix}_AC_INV_Relay" | ||
|
@@ -18,7 +18,7 @@ binary_sensor: | |
bitmask: 0x2 | ||
|
||
- platform: modbus_controller | ||
modbus_controller_id: ${modbus_controller_id} | ||
modbus_controller_id: ${modbus_controller_id_hf} | ||
|
||
skip_updates: 2 | ||
name: "${entities_name_prefix}-General - AC Grid Relay" # bit 2 | ||
id: "${entities_id_prefix}_General_AC_Grid_Relay" | ||
|
@@ -27,7 +27,7 @@ binary_sensor: | |
bitmask: 0x4 | ||
|
||
- platform: modbus_controller | ||
modbus_controller_id: ${modbus_controller_id} | ||
modbus_controller_id: ${modbus_controller_id_hf} | ||
skip_updates: 2 | ||
name: "${entities_name_prefix}-General - AC Generator Relay" # bit 3 | ||
id: "${entities_id_prefix}_General_AC_Generator_Relay" | ||
|
@@ -36,7 +36,7 @@ binary_sensor: | |
bitmask: 0x8 | ||
|
||
- platform: modbus_controller | ||
modbus_controller_id: ${modbus_controller_id} | ||
modbus_controller_id: ${modbus_controller_id_hf} | ||
|
||
skip_updates: 2 | ||
name: "${entities_name_prefix}-General - Turn off/on status" | ||
id: "${entities_id_prefix}_General_Turn_off_on_status" | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -47,9 +47,7 @@ sensor: | |
unit_of_measurement: "W" | ||
state_class: "measurement" | ||
accuracy_decimals: 1 | ||
filters: | ||
- multiply: 0.1 | ||
value_type: U_WORD | ||
value_type: S_WORD | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Did I get the Data Type wrong ? The Reference Guide didn't specify neither U16 nor S16 and I never tested the Generator, so it could very well be. Did you check your Results ? |
||
|
||
- platform: modbus_controller | ||
modbus_controller_id: ${modbus_controller_id} | ||
|
@@ -60,7 +58,7 @@ sensor: | |
unit_of_measurement: "W" | ||
state_class: "measurement" | ||
accuracy_decimals: 1 | ||
value_type: U_WORD | ||
value_type: S_WORD | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Did I get the Data Type wrong ? The Reference Guide didn't specify neither U16 nor S16 and I never tested the Generator, so it could very well be. Did you check your Results ? |
||
|
||
- platform: modbus_controller | ||
modbus_controller_id: ${modbus_controller_id} | ||
|
@@ -71,15 +69,27 @@ sensor: | |
unit_of_measurement: "W" | ||
state_class: "measurement" | ||
accuracy_decimals: 1 | ||
value_type: U_WORD | ||
value_type: S_WORD | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Did I get the Data Type wrong ? The Reference Guide didn't specify neither U16 nor S16 and I never tested the Generator, so it could very well be. Did you check your Results ? |
||
|
||
- platform: modbus_controller | ||
modbus_controller_id: ${modbus_controller_id} | ||
modbus_controller_id: ${modbus_controller_id_hf} | ||
|
||
name: "${entities_name_prefix}-Generator Power Total" | ||
id: "${entities_id_prefix}_Generator_Power_Total" | ||
register_type: holding | ||
address: 667 | ||
unit_of_measurement: "W" | ||
state_class: "measurement" | ||
accuracy_decimals: 1 | ||
value_type: U_WORD | ||
value_type: S_WORD | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Did you test and check your Results of this new Parameter ? |
||
|
||
number: | ||
- platform: modbus_controller | ||
use_write_multiple: true | ||
modbus_controller_id: ${modbus_controller_id} | ||
id: "${entities_id_prefix}_Generator_Minimum_Solar_Power" | ||
name: "${entities_name_prefix}-Generator Minimum Solar Power to Enable" | ||
address: 139 | ||
unit_of_measurement: W | ||
value_type: U_WORD | ||
min_value: 0 | ||
max_value: 8000 |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -39,7 +39,7 @@ sensor: | |
value_type: S_WORD | ||
|
||
- platform: modbus_controller | ||
modbus_controller_id: ${modbus_controller_id} | ||
modbus_controller_id: ${modbus_controller_id_hf} | ||
name: "${entities_name_prefix}-Out-of-Grid Active Power L1" | ||
id: "${entities_id_prefix}_Out_of_Grid_Active_Power_L1" | ||
register_type: holding | ||
|
@@ -50,7 +50,7 @@ sensor: | |
value_type: S_WORD | ||
|
||
- platform: modbus_controller | ||
modbus_controller_id: ${modbus_controller_id} | ||
modbus_controller_id: ${modbus_controller_id_hf} | ||
name: "${entities_name_prefix}-Out-of-Grid Active Power L2" | ||
id: "${entities_id_prefix}_Out_of_Grid_Active_Power_L2" | ||
register_type: holding | ||
|
@@ -61,7 +61,7 @@ sensor: | |
value_type: S_WORD | ||
|
||
- platform: modbus_controller | ||
modbus_controller_id: ${modbus_controller_id} | ||
modbus_controller_id: ${modbus_controller_id_hf} | ||
name: "${entities_name_prefix}-Out-of-Grid Active_Power L3" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why did you remove the High Refresh Rate Controller for this Signal ? |
||
id: "${entities_id_prefix}_Out_of_Grid_Active_Power_L3" | ||
register_type: holding | ||
|
@@ -72,7 +72,7 @@ sensor: | |
value_type: S_WORD | ||
|
||
- platform: modbus_controller | ||
modbus_controller_id: ${modbus_controller_id} | ||
modbus_controller_id: ${modbus_controller_id_hf} | ||
name: "${entities_name_prefix}-Out-of-Grid Active Power Total" | ||
id: "${entities_id_prefix}_Out_of_Grid_Active_Power_Total" | ||
register_type: holding | ||
|
@@ -83,7 +83,7 @@ sensor: | |
value_type: S_WORD | ||
|
||
- platform: modbus_controller | ||
modbus_controller_id: ${modbus_controller_id} | ||
modbus_controller_id: ${modbus_controller_id_hf} | ||
name: "${entities_name_prefix}-Out-of-Grid Apparent Power Total" | ||
id: "${entities_id_prefix}_Out_of_Grid_Apparent_Power_Total" | ||
register_type: holding | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,6 @@ | ||
text_sensor: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why was this removed ? |
||
- platform: modbus_controller | ||
modbus_controller_id: ${modbus_controller_id} | ||
modbus_controller_id: ${modbus_controller_id_hf} | ||
|
||
skip_updates: 2 | ||
bitmask: 0 | ||
register_type: holding | ||
|
@@ -22,7 +22,7 @@ text_sensor: | |
|
||
sensor: | ||
- platform: modbus_controller | ||
modbus_controller_id: ${modbus_controller_id} | ||
modbus_controller_id: ${modbus_controller_id_hf} | ||
|
||
skip_updates: 2 | ||
name: "${entities_name_prefix}-Status - Warning1" | ||
id: "${entities_id_prefix}_Status_Warning1" | ||
|
@@ -32,7 +32,7 @@ sensor: | |
value_type: U_WORD | ||
|
||
- platform: modbus_controller | ||
modbus_controller_id: ${modbus_controller_id} | ||
modbus_controller_id: ${modbus_controller_id_hf} | ||
skip_updates: 2 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Does this really need to have a high Refresh Rate ? |
||
name: "${entities_name_prefix}-Status - Warning2" | ||
id: "${entities_id_prefix}_Status_Warning2" | ||
|
@@ -42,7 +42,7 @@ sensor: | |
value_type: U_WORD | ||
|
||
- platform: modbus_controller | ||
modbus_controller_id: ${modbus_controller_id} | ||
modbus_controller_id: ${modbus_controller_id_hf} | ||
|
||
skip_updates: 2 | ||
name: "${entities_name_prefix}-Status - Warning3" | ||
id: "${entities_id_prefix}_Status_Warning3" | ||
|
@@ -52,7 +52,7 @@ sensor: | |
value_type: U_WORD | ||
|
||
- platform: modbus_controller | ||
modbus_controller_id: ${modbus_controller_id} | ||
modbus_controller_id: ${modbus_controller_id_hf} | ||
|
||
skip_updates: 2 | ||
name: "${entities_name_prefix}-Status - Error1" | ||
id: "${entities_id_prefix}_Status_Error1" | ||
|
@@ -62,7 +62,7 @@ sensor: | |
value_type: U_WORD | ||
|
||
- platform: modbus_controller | ||
modbus_controller_id: ${modbus_controller_id} | ||
modbus_controller_id: ${modbus_controller_id_hf} | ||
|
||
skip_updates: 2 | ||
name: "${entities_name_prefix}-Status - Error2" | ||
id: "${entities_id_prefix}_Status_Error2" | ||
|
@@ -72,7 +72,7 @@ sensor: | |
value_type: U_WORD | ||
|
||
- platform: modbus_controller | ||
modbus_controller_id: ${modbus_controller_id} | ||
modbus_controller_id: ${modbus_controller_id_hf} | ||
|
||
skip_updates: 2 | ||
name: "${entities_name_prefix}-Status - Error3" | ||
id: "${entities_id_prefix}_Status_Error3" | ||
|
@@ -82,7 +82,7 @@ sensor: | |
value_type: U_WORD | ||
|
||
- platform: modbus_controller | ||
modbus_controller_id: ${modbus_controller_id} | ||
modbus_controller_id: ${modbus_controller_id_hf} | ||
|
||
name: "${entities_name_prefix}-Status - Failure Status of Communication Board" | ||
id: "${entities_id_prefix}_Status_Failure_Status_of_Communication_Board" | ||
register_type: holding | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why does this need a high refresh Rate ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is outdated, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@KastB: I think this was one of your earlier Commits. Most of the Stuff I feel like you implemented the
modbus_controller_id_hf
Controller, then a few Commits later you removed it (and switched back to themodbus_controller_id
), but originally I expected that you removed it only for a few Signals.But I think you once again removed it for all of them. I kinda got lost there already.
Basically you tried to change to a high Refresh Rate / dual rate Controller, then I guess you saw it didn't work and preferred to change the Modbus Baud Rate to 115200 instead in Advanced Menu ?