Skip to content
This repository has been archived by the owner on Sep 4, 2021. It is now read-only.

Hola, I wrote an implementation of codetheweb's stuff too #32

Open
Supermortal opened this issue Aug 9, 2018 · 5 comments
Open

Hola, I wrote an implementation of codetheweb's stuff too #32

Supermortal opened this issue Aug 9, 2018 · 5 comments

Comments

@Supermortal
Copy link

Hey, I just wanted to post this to you guys, and see what you thought. I re-wrote the tuyapi stuff as a project to help me learn Python, and I figured I'd post it to you guys. I don't want to step on toes or anything. https://github.com/Supermortal/python-tuya-oittm

@clach04
Copy link
Owner

clach04 commented Aug 9, 2018

@Supermortal neat! I have to confess I've not had/made time to work on this project recently and you've done a few things I was hoping to get into this implementation (e.g. discovery is something I wrote and got working but didn't get into this library).

I only had a few mins to look at your implementation. So these are random comments.

  1. I couldn't get this running under Python 2.7.13 - due to the F-String (PEP-498) usage. I noticed six was a dependency so I suspect you are aiming for support of both.
  2. There are a lot of dependencies, but I did get a quick demo running with Python 3.6 and discovery. Nice job!
  3. I like the use if async, great idea!
  4. Nice to see you have the start of tests for most aspects
  5. Use of a dict for device support is neat. Be interesting to see if you can expand that to other devices
  6. The OITTM_REVERSE_HUMIDIFIER_MAP in device_maps is hard coded, recommend creating that dynamically based on OITTM_HUMIDIFIER_MAP to avoid maintenance overhead (and potential for bugs)

You're welcome to take anything useful from this project (thanks for the acknowledgment BTW, its appreciated), thanks for sharing. Very cool. It would be nice if we could merge the projects some how but I don't have a lot of time these days to work on this so its not high on my todo list.

@Supermortal
Copy link
Author

Thanks for commenting. Yeah, I'd be down to merge, and I can look at the stuff you mentioned for sure.

@Supermortal
Copy link
Author

Would you mind if I pushed this up to PyPi, so I can start development on my Home Assistant component? If/when we do decide to merge code bases, I'd be happy to deprecate the PyPi package, I'd just like to get started on that soon. @clach04

@clach04
Copy link
Owner

clach04 commented Aug 12, 2018

@Supermortal go for it! Thanks for taking the time to work on it.

@cmille34
Copy link

This is exciting news! I still get random timeouts with the tuya.py in Home Assistant. Its great to see someone continuing to develop this for that platform. Thanks for your efforts!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants