Skip to content

Conversation

@iRateD1703
Copy link

Breaking change

Proposed change

New version of Modbus TCP register version 3.60 which introduces new datatypes for previously overflowing variables. The const file is updated accordingly.

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New integration (thank you!)
  • New feature (which adds functionality to an existing integration)
  • Deprecation (breaking change to happen in the future)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Additional information

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • The code has been formatted using Ruff (ruff format custom_components/victron)

Updated data types for solarcharger_yield_power and solarcharger_yield_user to UINT32 with MOdbus TCP register list 3.60
Updated to Modbus TCP register v 3.60 which includes appropriate datatypes for user yield and user power
Updated to Modbus TCP register version 3.60 which now includes appropriate datetypes for previously overflowing unit16 variables
Updated to Modbus TCP register version 3.60 which now includes appropriate datetypes for previously overflowing variables
@sfstar
Copy link
Owner

sfstar commented Nov 2, 2025

Thank you for submitting a pr.
I went for merging #386 my stale branch in which registers up to 3.66 were included (This is now part of the beta-3 pre-release for this integration).
On that branch I went for a slightly different approach (adding new uint32/int32 registers as new entities as not to break older installation).
Feel free to check it the pre-release

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.

Wrong yield total amount

2 participants