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

Corrected F730 reg 43141 scale factor. x10. #35

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

AndersHoglund
Copy link

@AndersHoglund AndersHoglund commented Nov 8, 2023

Nibe F730 modbus register 43141 value , compressor input power, seemed too low. Got 63W at 80% frekuency. Asked Nibe Support and got this reply:

"
Hej och tack för ditt mejl till NIBE Uplink.
Vi har undersökt viadre pch svaret ni fär är i 0,01kw=10w så värdet ni läser skall gångas med 10.
"

This might very well hit other models and possibly other register values as well. Not handled by this PR.
/A

@AndersHoglund
Copy link
Author

Turned out the "factor" is not a multiplier but a divisor, fixed. Also reg address 43375, cmpr mean power, is affected by the same scaling, fixed. All models with 43141 and 43375 probably also affected, F1155, F1255,F1355 and F750 fixed.
Now tested on my F730 via NibePi to my HA instance.
Screenshot 2023-11-09 at 16-26-22 Nibe – Home Assistant

@AndersHoglund
Copy link
Author

AndersHoglund commented Nov 9, 2023

Updated files also exists in docker/nibepi/models, updated too. Why this duplication?

@AndersHoglund
Copy link
Author

Confirmation from Nibe Support :

Hej och tack för ditt mejl till NIBE Uplink.

Det gäller för de enheter med en inverter styrning .

@elupus
Copy link

elupus commented Jan 2, 2024

I can confirm this. We have this adjustment in nibe python lib too: https://github.com/yozik04/nibe/blob/3eb7b88cb7e61553bbf3d53831a17faba7cb4b61/nibe/data/extensions.json#L162

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