New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added send simple ir command #4

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@betonme

betonme commented Jan 3, 2015

Needs additional arguments: device_id and command
Use it with:
PYTHONPATH="." python harmony --email user@example.com --password pass
--harmony_ip 192.168.0.1 send_command --device_id=123456 --command=PowerToggle

Added send simple ir command
Needs additional arguments: device_id and command
Use it with:
PYTHONPATH="." python harmony --email user@example.com --password pass \
    --harmony_ip 192.168.0.1 send_command --device_id=123456 --command=PowerToggle

rkitover added a commit to rkitover/pyharmony that referenced this pull request Jan 3, 2015

Merge PR for petele/pyharmony #4
Fix up for my fork, tested to work fine.
@rkitover

This comment has been minimized.

Show comment
Hide comment
@rkitover

rkitover Jan 3, 2015

I merged this into my fork:

https://github.com/rkitover/pyharmony

it has some extra features like sync and turn off support.

I really wanted the device command functionality and was planning to do it, thank you very much for doing it!

rkitover commented Jan 3, 2015

I merged this into my fork:

https://github.com/rkitover/pyharmony

it has some extra features like sync and turn off support.

I really wanted the device command functionality and was planning to do it, thank you very much for doing it!

rkitover added a commit to rkitover/pyharmony that referenced this pull request Jan 3, 2015

Improvements for PR petele/harmony #4
Add a time.sleep(0.5) to send_command in client.py to give the command a
chance to complete. This is an ugly hack, we need to actually follow the
protocol, but works fine for now.

Support using device labels or ids case-insensitively with --device.

Give an error if the device does not exist.

Update README.md.
@rkitover

This comment has been minimized.

Show comment
Hide comment
@rkitover

rkitover Jan 3, 2015

I made some improvements in my fork. Added a mutually exclusive --device param that takes a label, case-insensitively. Added a time.sleep(0.5) in client.py to give the command a chance to complete, since it has to be sent block=False.

rkitover commented Jan 3, 2015

I made some improvements in my fork. Added a mutually exclusive --device param that takes a label, case-insensitively. Added a time.sleep(0.5) in client.py to give the command a chance to complete, since it has to be sent block=False.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment