CJNE / pymyenergi

An async python library for interfacing with MyEnergi devices
MIT License
19 stars 15 forks source link

[Request] MyEnergi device priority #2

Closed conormce closed 2 years ago

conormce commented 2 years ago

Hi, I am creating custom sensors to correctly calculate the green energy used by the house, zappi and eddi. They essentially work backwards using House power (total - myenergi) and solar, if solar is > house power then put it into priority 1 device, if what this is greater than what is in priority 1 device then assign it to priority 2 device .... To do this accurately going forward it would be great to have the priority of the devices. Right now I have it hardcoded.

G6EJD commented 2 years ago

The ME system already does this, the device set as priority-1 takes solar excess then when satisfied device with priority-2 takes over?

conormce commented 2 years ago

I think you've taken this up a little wrong. The goal here is to accurately estimate the green energy used by each device. The green energy values from ME are not accurate, particularly for the zappi.

To estimate them correctly using this integrations sensors I need to know each devices priority. Right now I have the device priority hard coded based on what I have set in the app, what I want is to replace this hard coded value with the polled correct device priority.

G6EJD commented 2 years ago

I was merely observing that in the ME systems priority is taken care of and no way to turn it off, so why try and override it. I find the ME accuracy to be within 1% which is adequate for most applications and is certainly as good as most MID devices. Some subjects can be overthought.

conormce commented 2 years ago

Thanks for your feedback but I think you are picking this request up wrong and this issue is becoming a discussion without need. I am not asking for the ability to change the priority but if there is a way to read it from ME. I want to know out of my two devices (zappi and eddi) which is priority 1 and which is priority 2. I haven't asked for and have no use case for the ability to modify this priority.

conormce commented 2 years ago

Fixed in latest version