search

michaelarnauts / cec-mqtt-bridge

A HDMI-CEC to MQTT bridge for connecting HDMI-CEC-devices to your Home Automation system.
145 stars 48 forks source link

Rewrite #28

Closed michaelarnauts closed 1 year ago

michaelarnauts commented 4 years ago

I'm working on a rewrite of the library that fixes a few open issues.

It won't be backwards compatible with the existing code, so I will probably use a new version for this.

tlk commented 4 years ago

This script is just what I was looking for, thanks!

From the network graph it appears to be common for people to remove IR-support. Perhaps you could consider separating out the IR logic into a separate but almost identical project? Fx "ir-mqtt-bridge". This would align well with the "do one thing and do it well" unix philosophy.

Best, Thomas

michaelarnauts commented 4 years ago

IR is currently broken in this branch, maybe it does make sense to split it off to a separate project, I haven't thought about this. But it shouldn't matter that much, if you disable IR in the config, it won't be initialised.

tlk commented 4 years ago

That's right, although requirements.txt still reference python-lirc.

On another note, I have been looking at this cec python library which is available on pypi.org and piwheels.org (well, not yet but possibly soon-ish)

The API is different (notably libcec_configuration is not implemented) but I think the installation process would be simplified without the need for locally compiled python bindings.

Just wanted to share with you as I am interested in your thoughts on this.

michaelarnauts commented 4 years ago

I've looked at that library also, but it had some strange delays. When I send a command, it had a 2 seconds delay of something. I haven't looked at it further.

michaelarnauts commented 1 year ago

It seems like I have been running these changes for over 2 years, so I might as well merge them :)