Skip to content

Conversation

@cengelen
Copy link
Contributor

Summary

This pull request introduces a new function, update_classic_inverter_setting, to the GrowattApi class. This function allows users to apply settings to classic inverters, expanding the library's functionality to support more inverter types, like the older MTL-S series.

Additionally, a new example script, settings_example_classic.py, has been added to demonstrate how to use this function. The documentation in shinephone.md and inverter_settings.md has also been updated to include details about the new function.

Motivation

  • The addition of update_classic_inverter_setting addresses the need for my legacy/classic inverter, which was previously not covered by the library. Using this function I can enable/disable the inverter based on dynamic energy rates.

Checklist

  • I've made sure the PR does small incremental changes. (The new function, example script, and documentation updates are isolated changes.)
  • I've added/updated the relevant docs for code changes I've made. (Documentation in shinephone.md and inverter_settings.md has been updated.)

@cengelen
Copy link
Contributor Author

Hi @indykoning,
Please check this small PR so I can use this in the Home Assistant integration.

I think this would be beneficial for a lot of people using older Growatt inverters and dynamic tariffs.
Thank you!

Copy link
Owner

@indykoning indykoning left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@indykoning indykoning merged commit 09466c0 into indykoning:master Dec 27, 2025
@cengelen cengelen deleted the feature/update_classic_inverter branch December 28, 2025 20:09
@cengelen
Copy link
Contributor Author

cengelen commented Jan 1, 2026

@indykoning, Thanks for reviewing and approving! When do you expect this will part of a next 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.

2 participants