ryannazaretian / hacs-nexia-climate-integration Goto Github PK

This custom integration uses deprecated DEVICE_CLASS_* constants in its codebase.

The DEVICE_CLASS_* constants have been deprecated and replaced in Home Assistant Core 2021.12 (over a year ago). I would highly suggest updating/migrating this integration to the new enums.

For example, for the device classes supported by the sensor platform, there is now a SensorDeviceClass enum. So if a sensor previously used the DEVICE_CLASS_ENERGY constant, it should now use SensorDeviceClass.ENERGY. Other platforms (like binary_sensor, and number) provide similar enumerations for their supported device classes.

The migration thus only consists of replacing constants with an enumeration member and is, therefore, very low impact and should be fairly straightforward.

If I can help resolve any questions regarding this change or migration, feel free to ask or respond to this issue. I'm happy to help!

Kindest regards,

../Frenck

in 104.3 I get the following error in the logs when adding the nexia component.
2020-01-22 11:10:30 ERROR (MainThread) [homeassistant.setup] Error during setup of component nexia Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/setup.py", line 174, in _async_setup_component component.setup, hass, processed_config # type: ignore File "/usr/local/lib/python3.7/concurrent/futures/thread.py", line 57, in run result = self.fn(*self.args, **self.kwargs) File "/config/custom_components/nexia/__init__.py", line 83, in setup update_rate=NexiaThermostat.DISABLE_AUTO_UPDATE) File "/config/custom_components/nexia/nexia_thermostat.py", line 126, in __init__ self.login() File "/config/custom_components/nexia/nexia_thermostat.py", line 413, in login request = self._post_url("/session", payload) File "/config/custom_components/nexia/nexia_thermostat.py", line 213, in _post_url self.update() File "/config/custom_components/nexia/nexia_thermostat.py", line 449, in update force_update=True) File "/config/custom_components/nexia/nexia_thermostat.py", line 287, in _get_thermostat_json raise Exception("Nothing in the JSON") Exception: Nothing in the JSON

Is this a known issue?

It looks like I/O is happening inside the properties.

home-assistant/core#4210

It appears that and uncached _get_thermostat_json is being called inside one of the state properties.

Haven't looked into this yet, but the has_emergency_heat() function seems to be returning False on my system now.

Hope it's okay to leave this comment here Ryan. I have your Nexia integration running in Home Assistant. Working well except I'm noticing the following. The requested and actual compressor speed do not update in HA nor on the nexia.com/JSON page (https://www.mynexia.com/houses/?????/xxl_thermostats) but the other parameters seemed to be updating fine. This may seem strange but I've tried this several times: as soon as I run the Nexia app on my Android phone, the nexia.com/JSON compressor speeds start updating along with HA compressor speed. Any ideas?
Steve W. ([email protected])

Nexia changed websites. I am assuming this is why my nexia integration no longer works.

The new website should be https://asairhome.com/houses/<YOUR_HOUSE_ID>/climate.

Also, I uninstalled the nexia integration and it won't reinstall.

From the Log...

Unable to prepare setup for platform nexia.binary_sensor: Unable to set up component.
February 25, 2021, 12:36:05 PM – setup.py (ERROR) - message first occurred at February 25, 2021, 12:36:05 PM and shows up 3 times
Setup of nexia is taking longer than 300 seconds. Startup will proceed without waiting any longer
February 25, 2021, 12:36:05 PM – setup.py (ERROR)
Waiting on integrations to complete setup: nexia
February 25, 2021, 12:35:07 PM – bootstrap.py (WARNING) - message first occurred at February 25, 2021, 12:32:07 PM and shows up 4 times

ClimateDevice is deprecated, modify NexiaZone to extend ClimateEntity
February 25, 2021, 12:31:07 PM – Climate (WARNING)
BinarySensorDevice is deprecated, modify NexiaBinarySensor to extend BinarySensorEntity
February 25, 2021, 12:31:05 PM – Binary sensor (WARNING)

Hi Ryan,

Thanks again for the great work on this.

I noticed your PR to home assistant here: home-assistant/core#24947

It looks like the web-scraping concern can be solved by using the mobile JSON API (https://github.com/bdraco/homebridge-nexia-thermostat). Would you consider breaking out the nexia connector from the rest of the code and putting it on pypi under the same license as home assistant?
(https://github.com/home-assistant/home-assistant/blob/dev/LICENSE.md)

If you don't have the time right now, I'm happy to have in any way I can including helping converting it to use the JSON API.