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

Cannot select correct unit (kW) for sensor.energy_log_current_power_consumption_32306 #127832

Closed
kkub164 opened this issue Oct 7, 2024 · 7 comments · Fixed by #131572
Closed

Comments

@kkub164
Copy link

kkub164 commented Oct 7, 2024

The problem

Entity "NIBE S320 current power consumption" (sensor.energy_log_current_power_consumption_32306) should have W or kW unit of measurement, because it is CURRENT consumption. But only Wh or kWh can be selected .

What version of Home Assistant Core has the issue?

core-2024.10.1

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

NIBE heat pump

Link to integration documentation on our website

https://www.home-assistant.io/integrations/nibe_heatpump/

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

@home-assistant
Copy link

home-assistant bot commented Oct 7, 2024

Hey there @elupus, mind taking a look at this issue as it has been labeled with an integration (nibe_heatpump) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of nibe_heatpump can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign nibe_heatpump Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


nibe_heatpump documentation
nibe_heatpump source
(message by IssueLinks)

@elupus
Copy link
Contributor

elupus commented Oct 7, 2024

Are you sure. "Current".here could mean the momentary power. If it is wrong it need to be fixed in upstream lib.

@cedeherd
Copy link
Contributor

I've launched pull request yozik04/nibe#181 to fix this in the upstream lib.

@maand75
Copy link

maand75 commented Oct 16, 2024

Saw that this fix was commited in upstream lib, would it be possible for a new release to get this updated and fixed?

@cedeherd
Copy link
Contributor

@elupus would it be possible to bump the integration to version 2.13 for this fix to be part of next HA version?

@programmeraren
Copy link

Good evening!
I think this issue is the culprit of my energy graphs being weird with negative values! :)
As I cannot see the actual version of Nibe that is installed in HA (missing version information in the "Unit information" / "Enhet information" section) I guess it is not yet released?
@elupus - A release of this fix would be greatly appreciated! Any information of when it is planned?
Looking forward to this fix, and thanks very much for your work!
Best Regards

@elupus
Copy link
Contributor

elupus commented Nov 23, 2024

Go ahead and create a pull request. its a community project, so its up to all to contribute. I can review, but not time to do the changes myself at the moment.

@cedeherd cedeherd mentioned this issue Nov 25, 2024
19 tasks
@github-actions github-actions bot locked and limited conversation to collaborators Dec 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants