Skip to content

Conversation

@LKuemmel
Copy link
Contributor

@LKuemmel LKuemmel commented Nov 7, 2024

No description provided.

ndrsnhs and others added 30 commits October 25, 2024 10:35
Co-authored-by: LKuemmel <lena.kuemmel@openwb.de>
* fix default value if serial number is missing

* comment
* internal cp: don't set heartbeat if one cp is in error state

* fix non-duo

* fix 5 min handler
* fix null in daily log data

* fix
* range offset based on range mid point

* fix test: constant control rage
Co-authored-by: LKuemmel <lena.kuemmel@openwb.de>
update cards display theme dependencies
benderl and others added 3 commits November 7, 2024 08:03
* Solis: fix modbus address for SOC

The Solis support was created based on Fox ESS. It looks like the
creator missed to adjust the modbus address for SOC, it's 33139 based
on a register map I got from solis support.

Total discharge energy is also in kWh, not 1/10kWh units.

* Solis: read all registers as input registers

Every modbus query fails, as all registers need to be read as input
registers (code 0x04), instead of holding register.

* Solis: Fix for unreachable local variables

right now I get the following error:
UnboundLocalError: local variable 'power' referenced before assignment

Fix it by moving some code around.

* Review

---------

Co-authored-by: LKuemmel <76958050+LKuemmel@users.noreply.github.com>
@LKuemmel LKuemmel merged commit 03cc7ec into Beta Nov 7, 2024
1 check 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.

7 participants