search

gazoodle / geckolib

Library to interface with Gecko Alliance spa pack systems via in.touch2 module
GNU General Public License v3.0
62 stars 24 forks source link

Tried the update #1

Closed walterb65 closed 3 years ago

walterb65 commented 4 years ago

The update fails um my YE-5 with the following error File "/Users/walter/Desktop/Spa_Control/Geckolib/geckolib-main 2/client.py", line 118, in facade = GeckoFacade(spa) File "/Users/walter/Desktop/Spa_Control/Geckolib/geckolib-main 2/geckoautomation.py", line 237, in init self.scan_outputs() File "/Users/walter/Desktop/Spa_Control/Geckolib/geckolib-main 2/geckoautomation.py", line 266, in scan_outputs self._pumps = [ GeckoPump(self, device, gecko_constants.devices[device]) File "/Users/walter/Desktop/Spa_Control/Geckolib/geckolib-main 2/geckoautomation.py", line 268, in if gecko_constants.devices[device][3] == gecko_constants.device_class_pump]

builtins.KeyError: 'Waterfall'

BR Walter

gazoodle commented 4 years ago

Thanks for the prompt report! 0.3.7 Should fix that issue. I'll try to deal with the Waterfall class in the next release. If this works and it finds your spa successfully, can you

  1. Enter the command snapshot waterfall off
  2. Turn the waterfall on from your smartphone
  3. Enter the command snapshot waterfall on
  4. Turn the waterfall off from you smartphone
  5. Enter the command snapshot done

Then PM me the client.log file as it ought to contain sufficient info to deal with the waterfall. Edit: Just realized there is no PM facility on GitHub, I'll have to think about how to deal with that ...

walterb65 commented 4 years ago

Thank you for the quick fix. 1st test work now with 0.3.7. Will create the log tomorrow.

Thanks a lot for for great work.

gazoodle commented 4 years ago

You can email the log to gazoodle at hash dot fyi

gazoodle commented 3 years ago

0.3.12 ought to fix this issue.