search

skelgaard / homeassistant-apsystems

An APsystems Sensor for Home Assistant
Apache License 2.0
25 stars 9 forks source link

Can add entity apsystems_enrgy|_day to dasboard, but not in the energy tab #14

Closed odiejakkes closed 1 year ago

odiejakkes commented 2 years ago

thanks a lot for this fork, it works indeed,and although thank you for your message, i found your fork myselves after some digging đź‘Ť the entitys show up when making a card, have them in my dashboard besides the ECU-3 energy today. because i own a ECU-R and this damned ECU-3 strangely directly after installing your integration, it showed up as second available energy source in the energy tab but after a couple of minutes the apsystem_energy_day was replaced by the ECU-TODAY_ENERGY entity so it was double after deleting this double, the entities apsystem do not show in the energy tab anymore

any clue why this happened ?

it would all be solved when support apsytems anweres my question whether a ECU-3 ID can be added to a ECU-R account, so both solar installations show up in 1 emea account, now they are in seperate accounts.... but the support of apsystems is very minimal, they don't read questions and answer slowly

i'll inspect both sensor.py if there are conflicting identical defenitions of a string maybe that's why the ECU-R entitys took over your apsystems entity

odiejakkes commented 2 years ago

ah. maybe the sensor.py misses the device class to be able to show up in the energy tab ? this is from the sensor.py from the ECU-R integration đź‘Ť

from homeassistant.const import ( DEVICE_CLASS_TEMPERATURE, DEVICE_CLASS_POWER, DEVICE_CLASS_ENERGY, DEVICE_CLASS_VOLTAGE, DEVICE_CLASS_ENERGY, ENERGY_KILO_WATT_HOUR, POWER_WATT, ELECTRIC_POTENTIAL_VOLT, TEMP_CELSIUS, PERCENTAGE, FREQUENCY_HERTZ )

skelgaard commented 2 years ago

i don't know the ecu-3 and not much into python... i just forked the other one and fixed the few errors i have encountered to be able to use it my selves... if you have any idea whats needs adding to get it to work, i will gladly add it to the source :)

skelgaard commented 1 year ago

new release is out for it, you should be able to get data again

odiejakkes commented 1 year ago

Thanks, i'll give it a try….

Op ma 21 nov. 2022 om 21:09 schreef skelgaard @.***>

Closed #14 https://github.com/skelgaard/homeassistant-apsystems/issues/14 as completed.

— Reply to this email directly, view it on GitHub https://github.com/skelgaard/homeassistant-apsystems/issues/14#event-7860804204, or unsubscribe https://github.com/notifications/unsubscribe-auth/A2QIPBVAYDV4CY5OSKLDO7DWJPJIFANCNFSM564IWDXQ . You are receiving this because you authored the thread.Message ID: @.*** com>

odiejakkes commented 1 year ago

Finaly got the time, and yes it's working now as a sensor, thanks a lot for your now i need to find how i can make an integration of it, so i can add the KWH_day to my energy dashboard, it now lacks a unique ID seen others with the same issue, so hopefully i can find the info on their replies

Op do 18 aug. 2022 om 13:25 schreef skelgaard @.***>:

i don't know the ecu-3 and not much into python... i just forked the other one and fixed the few errors i have encountered to be able to use it my selves... if you have any idea whats needs adding to get it to work, i will gladly add it to the source :)

— Reply to this email directly, view it on GitHub https://github.com/skelgaard/homeassistant-apsystems/issues/14#issuecomment-1219376146, or unsubscribe https://github.com/notifications/unsubscribe-auth/A2QIPBV53JHICOE4DTNQDBDVZYMRHANCNFSM564IWDXQ . You are receiving this because you authored the thread.Message ID: @.***>

skelgaard commented 1 year ago

Nice đź‘Ť i look forward for a PR from you with your changes so that they can be reviewed and merged in :)