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

service set_room_mode not working for sensor.####_betriebsart_heizkreis_[abcd] #142

Closed
Jueff opened this issue Sep 22, 2024 · 2 comments · Fixed by #166
Closed

service set_room_mode not working for sensor.####_betriebsart_heizkreis_[abcd] #142

Jueff opened this issue Sep 22, 2024 · 2 comments · Fixed by #166

Comments

@Jueff
Copy link
Contributor

Jueff commented Sep 22, 2024

Version 0.9.0

Bug, missing feature or user error?

When trying to set heating mode for sensors sensor.[.*]betriebsart_heizkreis[abcd] the integration returns
"Entity sensor.warmepumpe_betriebsart_heizkreis_b does not support this service."

service: idm_heatpump.set_room_mode
data:
  target: sensor.warmepumpe_betriebsart_heizkreis_b
  value: eco
  acknowledge_risk: true
@Jueff
Copy link
Contributor Author

Jueff commented Sep 22, 2024

just seen the ciructe sensor use "CircuitMode" enum, not "RoomMode".
So seems to be a missing feature.

@kodebach
Copy link
Owner

Yes, the values for these sensors (Modbus registers) are slightly different, so they need different enums. Setting CircuitMode is currently not implemented and haven't found a way to better filter the entities shown by Home Assistant for each service.

For now I added set_circuit_mode to #50, but I'm not sure, when I'll get round to implementing it. I'd gladly accept a PR.

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 a pull request may close this issue.

2 participants