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

DSMR_Reader: not creating sensors for mqtt topic 'dsmr/consumption/quarter-hour-peak-electricity' #83999

Closed
davidvf opened this issue Dec 14, 2022 · 4 comments · Fixed by #84271

Comments

@davidvf
Copy link

davidvf commented Dec 14, 2022

The problem

I have DSMR application running on my network. An mqtt broker is forwarding the telegrams to homeassistant.
Using this integration https://www.home-assistant.io/integrations/dsmr_reader/

It should automatically create sensors for each topic, but it seems it does not create those sensors for the new:
dsmr/consumption/quarter-hour-peak-electricity

What version of Home Assistant Core has the issue?

Home Assistant 2022.12.5

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Container

Integration causing the issue

DSMR Reader

Link to integration documentation on our website

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

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

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

Code owner commands

Code owners of dsmr_reader can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Change the title of the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign dsmr_reader Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)


dsmr_reader documentation
dsmr_reader source
(message by IssueLinks)

@Glodenox
Copy link
Contributor

Glodenox commented Dec 14, 2022

Correct, I was already planning on creating a pull request for that in the coming days :) At the same time, I'll revert the state class removal, as that has been fixed in DSMR Reader.

@davidvf
Copy link
Author

davidvf commented Jan 6, 2023

@Glodenox thanks for the implementation, but i think the unit_of_measurement is incorrect.
In homeassistant it is shown in kWh, while i see the same value in DSMR, but there the value is expressed in kW.

@Glodenox
Copy link
Contributor

Glodenox commented Jan 6, 2023

Oh, good remark! I did indeed put it as kWh without a lot of thought, mostly because it has to do with a measurement over time. But the measurement is an average of how much kW is being used and isn't expressing the volume of power used over time, so the unit should indeed just remain kW. I'll create a bugfix for that.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
2 participants