SMA SunnyBoy - fix phase currents for single phase inverters #2844
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
LKuemmel
requested changes
Oct 22, 2025
| currents = [current_L1 / 1000, current_L2 / 1000, current_L3 / 1000] | ||
|
|
||
| currents = self.tcp_client.read_holding_registers(30977, [ModbusDataType.INT_32]*3, unit=unit) | ||
| currents = [current / -1000 if current != self.SMA_INT32_NAN else 0 for current in currents] |
Contributor
There was a problem hiding this comment.
Im Fall von NaN müsste currents=None gesetzt werden, sonst kann in der nachfolgenden Verarbeitung nicht zwischen 0A und keine Werte vorhanden unterschieden werden.
LKuemmel
pushed a commit
to LKuemmel/core
that referenced
this pull request
Nov 6, 2025
…2844) * support single phase inverters * handle power and currents NaN values
LKuemmel
added a commit
that referenced
this pull request
Nov 10, 2025
* add frequency (#2792) * temp charge templates as option (#2855) * draft * draft * temp charge templates optional * fix temporary settings optional (#2868) * fix temporary charge template: change vehicle (#2869) * improve error handling internal cp (#2814) * fix single phase wallboxes (#2836) * SMA SunnyBoy - fix phase currents for single phase inverters (#2844) * support single phase inverters * handle power and currents NaN values * Revert ValueError for NaN values SMA (#2887) * Handle invalid power_total by resetting values Set power_total to 0 and currents to [0, 0, 0] when power_total is not valid. * Helptext improved Co-authored-by: LKuemmel <76958050+LKuemmel@users.noreply.github.com> --------- Co-authored-by: LKuemmel <76958050+LKuemmel@users.noreply.github.com> * improve register check (#2906) * build all themes (#2886) * build all themes * remove hard coded theme bases --------- Co-authored-by: Lutz Bender <github@lutz-bender.de> * fix build themes wotkflow (#2903) * temporary settings active: only update temporary settings for changed… (#2879) * temporary settings active: only update temporary settings for changed persistent templates * fix * update config * fix * fix * fix * clean up * Koala - Add Input dialog for editing scheduled and time plans (#2895) * Add scheduled details component * Add / remove scheduled charging plan - details as dialog * Add time plan details - adjust backend command * Backend command * Add DC charging inputs * Display bidi settings in scheduled plan based on vehicle profile bidi enabled * Fix DC power values for scheduled and time planes * Close plan dialog upon plan deletion * Plan energy amount watts to kilowatts * Formatting fixes * Charging current title AC or DC * Formatting fix * Change add plan button outlined to filled * Formatting Delete plan button * Add kW label to DC power input field * Format input fields * Format input toggles --------- Co-authored-by: BrettS <brett.scott@openwb.de> * fix temporary charge temple: inactive-> update all cps using this template (#2907) * Update version 2.1.8-Beta.Patch.4 * Fix: Skoda SoC fehlende Restreichweite (#2789) * fix: handle missing 'remainingRangeInKm' in status data * fix: update vehicle status API endpoint and adjust data extraction for battery status * flake8: format cruising range calculation for better readability * use API v2 endpoints with fallback to API v1 * fix: vehicle status handling to support hybrid cars * add tests * flake8: remove whitespace in blank line * remove id_token from flow (#2915) * Merge pull request #2910 from Brett-S-OWB:charge-plans-text-fix Charging plans text fixes - text as in settings / standard theme * buil UI --------- Co-authored-by: ndrsnhs <156670705+ndrsnhs@users.noreply.github.com> Co-authored-by: SeaSpotter <florian.horch@gmail.com> Co-authored-by: Lutz Bender <github@lutz-bender.de> Co-authored-by: BrettS <brett.scott@openwb.de> Co-authored-by: vuffiraa72 <u.mersewsky+github@gmail.com> Co-authored-by: benderl <benderl@users.noreply.github.com>
LKuemmel
pushed a commit
to LKuemmel/core
that referenced
this pull request
Nov 10, 2025
…2844) * support single phase inverters * handle power and currents NaN values
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
[Ticket #21923403]