search

danielkaldheim / pyairstage

MIT License
5 stars 5 forks source link

Usage docs #2

Open riazkarim opened 11 months ago

riazkarim commented 11 months ago

Hi,

I love that you created this. I've used it to set up some automations in my home.

I was wondering what the usage of it was, as I ran into some issues trying to read the on_off state of my AC unit.

It seems the call to the refresh_parameters() does not await the call to get_devices() in the class AirstageAC.

I went through the code and I couldn't figure out whether it was a design issue, or I was using it incorrectly. I ended up calling get_devices() and just reading the parameters I needed in my own class, and using those, but that doesn't sit right with me, especially as I grow my usage of the API.

I guess the first question is, how do I properly instantiate an instance of the AirstageAC class such that I can use the method get_device_on_off_state()?

TIA

danielkaldheim commented 11 months ago

Hi,

Glad to hear!

It was made to be used with Home Assistant integration https://github.com/danielkaldheim/ha_airstage and that's why there is no documentation (and probably not all methods not working as expected) .

I see that there is a missing await in refresh_parameters(), I'll fix that.

Please check that repo for how I implemented the HA Integration. And I'll create some documentation when I have some spare time.

If you have any suggestions, please open a pull request, I appreciate any input!