Skip to content

fix: multisensor current and energy sensor scales#180

Merged
bbx-a merged 1 commit intoblebox:masterfrom
bkobus-bbx:fix/multisensor-current-energy-scale
Apr 21, 2026
Merged

fix: multisensor current and energy sensor scales#180
bbx-a merged 1 commit intoblebox:masterfrom
bkobus-bbx:fix/multisensor-current-energy-scale

Conversation

@bkobus-bbx
Copy link
Copy Markdown
Contributor

Fixes root cause of home-assistant/core#166181.

multiSensor returns current in mA (not µA) and energy in Wh (not kWh). Corrected scale factors accordingly.

@MakaanPL
Copy link
Copy Markdown

@swistakm @pvsti could you please approve this PR, prepare new uniapi release, and then also update uniapi dependency in HA blebox integration? Thank you.

@bartoszkaminski
Copy link
Copy Markdown

@MakaanPL: I contacted BleBox support regarding this issue.

They confirmed that the problem is known on their side and has already been forwarded internally for analysis. However, due to low priority, they are currently not able to provide any estimated timeline for an official fix.
They also mentioned that the Pstryk energy meter is not officially supported in the Home Assistant BleBox integration.

Sharing this here for context — it seems that without changes on the library side, the issue is unlikely to be resolved anytime soon.

@MakaanPL
Copy link
Copy Markdown

But THIS is the fix in blebox_uniapi library, apparently done by their own employee. What I'm asking for is mostly "paperwork" necessary for the change to propagate to all users of Pstryk meter in HA.

I also prepared a workaround in HA integration after reporting the issue, but obviously a fix in upstream library is a better choice.

@bbx-a bbx-a merged commit 06f2b4d into blebox:master Apr 21, 2026
2 checks passed
@bkobus-bbx bkobus-bbx deleted the fix/multisensor-current-energy-scale branch April 21, 2026 11:18
@bartoszkaminski
Copy link
Copy Markdown

apparently done by their own employee

That I didn't know. I assumed that it's a fix by someone from the community and the PR is going to stay open for ages. Glad to see it merged 🙂

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.

4 participants