Skip to content
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

New command: Identify #234

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

mitkodotcom
Copy link

I have added a new command: Identify
The command can also be executed by pressing the button 8 times

It serves two purposes:

helps you to visually find out and identify a particular device
helps you find the IP address of a device

When you execute the command "Identify" of press the button 8 times, the device's led will start to blink, showing the digits of the last octet of the IP address. You just have to count the blinks for the hundreds, tens and ones. Now you know the ip address, assigned by the dhcp server (provided you know the first 3 octets of your network) and can access the device's web server.

@mitkodotcom
Copy link
Author

Moved pull request to a different branch for easier version checking

@mitkodotcom mitkodotcom changed the title Identify New command: Identify Jan 13, 2017
@arendst
Copy link
Owner

arendst commented Jan 22, 2017

Sorry, delays of 1600 are just too much. Would need asynch protocol during 0.1 second loop.

@mitkodotcom
Copy link
Author

Initially I was thinking about making it asynchronous, but then I decided that this functionality will be used in very, very rare occasions - mostly only during initial setup and when replacing / reconfiguring the device. It is almost certain that during this period the device will not be in production and this delay will not do any harm.

On the other side, making it asynchronous will increase the code size and complexity.

However, we may think of a more general solution - to create a "blink the led N-times" asynchronous procedure, which can be used for other purposes in the future (for example - to confirm a command execution, etc.)

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

Successfully merging this pull request may close these issues.

2 participants