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 storage modes (and a register) for Solis-FB00 #1207

Merged
merged 2 commits into from
Jan 16, 2025

Conversation

isorin
Copy link
Contributor

@isorin isorin commented Jan 8, 2025

I've upgraded my Solis inverter to the latest firmware and started using FB00 implementation.

The storage control switch is the 1st thing I noticed.

  1. Incorrect settings for some modes (so trying to make it more consistent with the pre FB00 implementation)
  2. The two sets need to match, otherwise setting a mode will result in an empty value.

Also fixed register for "timed_charge_start_hours_6".

One other thing which needs to be fixed (I will send another PR if I find where it is) is the missing prefix for switch entities
e.g. "Timed Charge Slot X Enable" is not prefixed (just the name, the entity_ids are correct)
it should be "Solis Timed Charge Slot 1 Enable" (or whatever the user chooses as prefix in the config)

@isorin isorin changed the title Fix storage modes for Solis-FB00 Fix storage modes (and a register) for Solis-FB00 Jan 9, 2025
@wills106
Copy link
Owner

@fboundy are you able to check this change as you did the initial support for the newer Firmware version.

Just want to check that different Inverter's aren't using a different register for slot 6.

Copy link
Contributor

@fboundy fboundy left a comment

Choose a reason for hiding this comment

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

This looks correct. Must have been a typo in the original.

Copy link
Contributor

@fboundy fboundy left a comment

Choose a reason for hiding this comment

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

Yes, this all look OK. Personally I only use 33 and 49 anyway.

@wills106 wills106 merged commit 84a6b1b into wills106:main Jan 16, 2025
3 checks passed
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.

3 participants