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

Illegal Data Address in Modbus Task Write for GoodWe.Grid-Meter #2895

Closed
Sn0w3y opened this issue Dec 2, 2024 · 0 comments
Closed

Illegal Data Address in Modbus Task Write for GoodWe.Grid-Meter #2895

Sn0w3y opened this issue Dec 2, 2024 · 0 comments

Comments

@Sn0w3y
Copy link
Contributor

Sn0w3y commented Dec 2, 2024

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.

Sn0w3y added a commit to Sn0w3y/openems that referenced this issue Dec 2, 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]>
@Sn0w3y Sn0w3y closed this as completed Dec 17, 2024
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

No branches or pull requests

1 participant