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

Insteon: Add User Option to Enable PLM Restart; Prevent Crashes #444

Merged
merged 2 commits into from
Aug 5, 2014

Conversation

krkeegan
Copy link
Collaborator

This adds a new INI variable

Insteon_PLM_reconnect_count

By default this is set to 99, to disable any PLM reconnect attempts. If you have PLM disconnect issues, try setting it to 2 or 3. The setting waits for the number of missing ACK responses from the PLM before trying to reconnect.

This also fixes an issue where trying to close an undefined port would completely crash MH, now it just results in a warning, but MisterHouse continues.

This adds a new INI variable

Insteon_PLM_reconnect_count

By default this is set to 99, to disable any PLM reconnect attempts.  If you have PLM disconnect issues, try setting it to 2 or 3.  The setting waits for the number of missing ACK responses from the PLM before trying to reconnect.

This also fixes an issue where trying to close an undefined port would completely crash MH, now it just results in a warning, but MisterHouse continues.
krkeegan added a commit that referenced this pull request Aug 5, 2014
Insteon: Add User Option to Enable PLM Restart; Prevent Crashes
@krkeegan krkeegan merged commit 976b99f into hollie:master Aug 5, 2014
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.

1 participant