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

MQTT Publish "Inverter power limit Set" as number #59

Closed
pitrade opened this issue Oct 19, 2024 · 5 comments
Closed

MQTT Publish "Inverter power limit Set" as number #59

pitrade opened this issue Oct 19, 2024 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@pitrade
Copy link

pitrade commented Oct 19, 2024

Problem:
MQTT /inverter/PowerLimit_Set becomes sensor.()_inverter_power_limit_set instead of number.()_inverter_power_limit_set in HomeAssistant and is not editable with UI or automations.

Suggestion:
Publish as number like OpenDTU MqttHandleHass

@ohAnd
Copy link
Owner

ohAnd commented Oct 20, 2024

@pitrade

you're fully right - this point was never tested (and at least for me not used ;-) )

currently the set value is only a sensor - and in HA it will be only a display but no usable switch/ dimmer/ controller

I will check the needed changes to get a full integration for HA and other frameworks that using MQTT autodiscovery/ config

thanks for the hint!

@ohAnd ohAnd self-assigned this Oct 20, 2024
@ohAnd ohAnd added the bug Something isn't working label Oct 20, 2024
ohAnd added a commit that referenced this issue Oct 24, 2024
…ge openhab item name ..._PowerLimit_Set to ..._PowerLimitSet
@ohAnd
Copy link
Owner

ohAnd commented Oct 26, 2024

@pitrade can you please check the snapshot https://github.com/ohAnd/dtuGateway/releases/tag/snapshot
for the entry of the "power limit set" via HA I set also the box style instead of a slider
thx

@pitrade
Copy link
Author

pitrade commented Oct 26, 2024

@ohAnd works fine. Thank you!

@pitrade
Copy link
Author

pitrade commented Oct 26, 2024

There is a minimum value of 2%, so you can't set the limit to 0% when you won't discharge the battery. At least for me, a 0% minimum value would be better because my algorithm only sets the inverter limit. Then the inverter is turned on/off depending of the limit value.

Also an on/off button would be great. I currently use a custom Hoymiles integration for this but it is unstable. Should I create a new issue for this feature request?

ohAnd added a commit that referenced this issue Oct 27, 2024
fixing #59 and smaller changes around this topic incl. breakable change openhab item name ..._PowerLimit_Set to ..._PowerLimitSet
@ohAnd
Copy link
Owner

ohAnd commented Oct 27, 2024

Hi @pitrade ,

yep, would be nice... (creating a new feature, due to other topic than setting the PowerLimit via MQTT and HA).
Then there is good point to extend the discussion around deactivating or things like that.

thx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants