-
Notifications
You must be signed in to change notification settings - Fork 411
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
Illegal Data Address in Modbus Task Write for GoodWe.Grid-Meter #2895
Comments
Sn0w3y
added a commit
to Sn0w3y/openems
that referenced
this issue
Dec 2, 2024
This fixes: OpenEMS#2895
sfeilmeier
added a commit
that referenced
this issue
Dec 10, 2024
sfeilmeier
added a commit
that referenced
this issue
Dec 10, 2024
* Edge * ENTSO-E: added preferred resolution config * ModbusTcpApi: fix updating ModbusRecordTable - Clears ModbusList on initializing => components, that are removed in the modbus component_ids cant be accessed by qmodmaster anymore. - Fixed ModbusComponents getting actually removed out of array => resulted in write commands not being executed after esscluster change - High Prio because this bug could lead to serious problems on our systems, especially on Industrial systems. * ToU: resolution property addition in ENTSO-E App + fix ENTSO-E timing - Added resolution property to be able to be configurable through APP. - Fixed the ENTSO-E Bug where prices are mismatched by 15 mins. * Prediction ValueRange: fix possible NPE if value is null - In my refactoring of the Prediction class I missed handling of a situation where a NullPointerException could happen. - This leads to the problem that - if no local historic values are existing, e.g. after system outage or at IBN, the prediction fails with a NullPointerException. * Time-of-Use-Tariff: apply ESS_MAX_SOC in Run-State - Unfortunately this had been lost during splitting of Optimizer and ToU-Controller - Also increasing the value to 94 % to take into account larger storage systems * EVCS Mennekes Read Only implementation - Mennekes Implementation from OpenEMS PR #2082 - Added App for Mennekes - Added new App Category for Read Only Evcs - Added Read only to mennekes config * Core: add Greece to safety country list * Backport GoodWe Grid.Meter fix; relates to #2895 * UI * dynamic scaling for charts and start scales at least at 0 - Redoing feature that yAxis scale has dynamic scale and didnt have to start at 0 * adjust server not accessible page for local monitoring - Adjusting server not accessible page for local monitoring, * update last updated timestamp for empty edge - Updating `lastUpdated` timestamp even if edge config is empty, `_sum/State` still exists * show absolute value for grid current value in modal - Use absolute value for grid modal current, text on the current value shows state `grid-feed-in` or `grid-buy` * Offline-Page Translations * ToU chart bugfix regarding dynamic scaling * Darkmode - some color changes were done for both light and dark mode --------- Co-authored-by: Stefan Feilmeier <[email protected]> Co-authored-by: Sagar Venu <[email protected]> Co-authored-by: Sebastian Asen <[email protected]> Co-authored-by: Fabian Brandtner <[email protected]> Co-authored-by: Lukas Rieger <[email protected]> Co-authored-by: Anas Shetla <[email protected]> Co-authored-by: Johann Kaufmann <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
The implementation of the GoodWeGridMeterImpl component in the OpenEMS project is encountering an Illegal Data Address exception when executing a Modbus Write operation using the FC6WriteRegisterTask. The specific error observed is:
ERROR io.openems.edge.bridge.modbus.api.task.AbstractTask Execute failed FC6WriteRegister [meter0;unitid=247;ref=47456/0xb960;length=1] ModbusSlaveException: Illegal Data Address
This occurs while attempting to write to the register 47456 (0xB960) for the EXTERNAL_METER_RATIO channel. The problem appears to be related to the GoodWe grid meter's Modbus implementation or register access.
Screenshots
No response
Operating System
Ubuntu
How to reproduce the Error?
Set up the GoodWeGridMeterImpl component in an OpenEMS Edge environment.
The text was updated successfully, but these errors were encountered: