rsnodgrass / hass-poolmath

Pool Math for Home Assistant
Other
30 stars 9 forks source link

Integration only PH & FC entities available #1

Closed Pavleau closed 4 years ago

Pavleau commented 4 years ago

Hi @rsnodgrass,

Firstly thanks for creating this integration. I've successfully installed and had no issue pulling data for PH & FC, however I am unable to get any of the other entities to be available.

My logs show this: 2020-02-09 12:47:44 WARNING (SyncWorker_4) [custom_components.poolmath.sensor] Unknown Pool Math sensor 'cc' discovered at https://api.poolmathapp.com/share/mqxyPPr 2020-02-09 12:47:45 WARNING (SyncWorker_4) [custom_components.poolmath.sensor] Unknown Pool Math sensor 'temp' discovered at https://api.poolmathapp.com/share/mqxyPPr 2020-02-09 12:47:45 WARNING (SyncWorker_4) [custom_components.poolmath.sensor] Unknown Pool Math sensor 'csi' discovered at https://api.poolmathapp.com/share/mqxyPPr 2020-02-09 12:47:46 WARNING (SyncWorker_13) [custom_components.poolmath.sensor] Unknown Pool Math sensor 'cc' discovered at https://api.poolmathapp.com/share/mqxyPPr

I've had a look around the /config/custom_components/poolmath/sensor.py file and with my very limited code knowledge, I can't see why HA can't see these. Is there additional steps required to add these manually?

rsnodgrass commented 4 years ago

Hi Pavleau, these sensors aren't supported in the current version. However, I just added them to master. Please select the 'master' version in HACS, upgrade to that, and let me know how it works!

On Sat, Feb 8, 2020 at 6:16 PM Pavleau notifications@github.com wrote:

Hi @rsnodgrass https://github.com/rsnodgrass,

Firstly thanks for creating this integration. I've successfully installed and had no issue pulling data for PH & FC, however I am unable to get any of the other entities to be available.

My logs show this: 2020-02-09 12:47:44 WARNING (SyncWorker_4) [custom_components.poolmath.sensor] Unknown Pool Math sensor 'cc' discovered at https://api.poolmathapp.com/share/mqxyPPr 2020-02-09 12:47:45 WARNING (SyncWorker_4) [custom_components.poolmath.sensor] Unknown Pool Math sensor 'temp' discovered at https://api.poolmathapp.com/share/mqxyPPr 2020-02-09 12:47:45 WARNING (SyncWorker_4) [custom_components.poolmath.sensor] Unknown Pool Math sensor 'csi' discovered at https://api.poolmathapp.com/share/mqxyPPr 2020-02-09 12:47:46 WARNING (SyncWorker_13) [custom_components.poolmath.sensor] Unknown Pool Math sensor 'cc' discovered at https://api.poolmathapp.com/share/mqxyPPr

I've had a look around the /config/custom_components/poolmath/sensor.py file and with my very limited code knowledge, I can't see why HA can't see these. Is there additional steps required to add these manually and / or more likely there is a step I'm missing to add these.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/rsnodgrass/hass-poolmath/issues/1?email_source=notifications&email_token=AAQY4XF4ATBECCOAHSHS6CTRB5ROPA5CNFSM4KR6QV42YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IMBDMYQ, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAQY4XDFOCWK54IUMBILLTDRB5ROPANCNFSM4KR6QV4Q .

Pavleau commented 4 years ago

Thanks, all data is now available as per my pool math logs. Note: I did need to make a test log entry to 'activate' the additional sensors in home assistant.

My logs now look good, although there is just this one entry after restart that you may be interested in knowing about.

2020-02-10 14:10:13 WARNING (MainThread) [homeassistant.components.sensor] Setup of sensor platform poolmath is taking over 10 seconds.`

Appreciate the quick response, thank you.