Skip to content
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

fix: SG3.6RS missing registers #180

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JorgHendriks
Copy link

SG3.6RS was missing in the model list for particular addresses in the register.
As a result not all available values were exposed, most notably 5031, total_active_power that is part of the config example for mqtt export.

Fix: added SG3.6RS to all entries where SG3.0RS and SG4.0RS were listed (same model family).

Level 2 output after this fix:

+----------------------------------------------------------------------+
| Address | Register                            | Value                |
+---------+-------------------------------------+----------------------+
| ----    | device_type_code                    | SG3.6RS              |
| vr001   | run_state                           | ON                   |
| vr003   | last_reset                          | 2024-5-22 16:56:13   |
| vr006   | daily_export_to_grid                | 0.0 kWh              |
| vr007   | daily_import_from_grid              | 0.0 kWh              |
| 5001    | nominal_active_power                | 3.7 kW               |
| 5002    | output_type                         | 2P                   |
| 5003    | daily_power_yields                  | 7.6 kWh              |
| 5004    | total_power_yields                  | 7.6 kWh              |
| 5006    | total_running_time                  | 6 h                  |
| 5008    | internal_temperature                | 39.6 °C              |
| 5009    | total_apparent_power                | 2655 VA              |
| 5011    | mppt_1_voltage                      | 319.3 V              |
| 5012    | mppt_1_current                      | 8.6 A                |
| 5013    | mppt_2_voltage                      | 40.6 V               |
| 5014    | mppt_2_current                      | 0.0 A                |
| 5017    | total_dc_power                      | 2740 W               |
| 5019    | phase_a_voltage                     | 243.7 V              |
| 5020    | phase_b_voltage                     | 0.0 V                |
| 5021    | phase_c_voltage                     | 0.0 V                |
| 5022    | phase_a_current                     | 11.0 A               |
| 5031    | total_active_power                  | 2606 W               |
| 5033    | total_reactive_power                | -4 Var               |
| 5035    | power_factor                        | 1.0                  |
| 5036    | grid_frequency                      | 50.08 Hz             |
| 5038    | work_state_1                        | Run                  |
| 5049    | nominal_reactive_power              | 2.2 kVar             |
| 5071    | array_insulation_resistance         | 2432 k-ohm           |
| 5077    | active_power_regulation_setpoint    | 3700 W               |
| 5079    | reactive_power_regulation_setpoint  | 0 Var                |
| 5081    | work_state_2                        | Run                  |
| 5113    | daily_running_time                  | 371 min              |
| 5128    | monthly_power_yields                | 7.6 kWh              |
| 5146    | negative_voltage_to_the_ground      | 0.0 V                |
| 5147    | bus_voltage                         | 383.7 V              |
| 5150    | pid_work_state                      | None                 |
| 5151    | pid_alarm_code                      | None                 |
| 5216    | export_power                        | 0 W                  |
| 5218    | power_meter                         | 0 W                  |
| 7013    | string_1_current                    | 0.0 A                |
| 7014    | string_2_current                    | 0.0 A                |
| 7026    | string_14_current                   | 0.0 A                |
| 5006    | start_stop                          | Start                |
| 5007    | power_limitation_switch             | Enable               |
| 5008    | power_limitation_setting            | 100.0 %              |
| 5019    | power_factor_setting                | 0.0                  |
| 5035    | night_svg_switch                    | Disable              |
| 5036    | reactive_power_adjustment_mode      | Off                  |
| 5037    | reactive_power_percentage_setting   | 0.0 %                |
| 5039    | power_limitation_adjustment         | 3.7 kW               |
| 5040    | reactive_power_adjustment           | 0.0 kVar             |
| 5041    | pid_recovery                        | Disable              |
| vr002   | timestamp                           | 2024-5-22 17:01:13   |
| vr004   | export_to_grid                      | 0 W                  |
| vr005   | import_from_grid                    | 0 W                  |
+----------------------------------------------------------------------+
Logged 55 registers to Console

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant