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.

Sign up for GitHub

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

Jump to bottom

Add penalty score to avoid false-positives #8

Closed
tzerk opened this issue Mar 2, 2017 · 1 comment
Closed

Add penalty score to avoid false-positives #8

tzerk opened this issue Mar 2, 2017 · 1 comment
Assignees
@tzerk
Labels
enhancement

Comments

@tzerk
Copy link
Owner

tzerk commented Mar 2, 2017

Add an integer penalty score that gets raised for each iteration the process is running, but disconnected. This is to avoid certain circumstances where the process is only temporarily "offline", e.g. when switching channels. The user must provide a penalty threshold value that must be reached to finally execute the exit conditions.

suggested by u/Xytech_
@tzerk tzerk self-assigned this Mar 2, 2017
tzerk added a commit that referenced this issue Mar 3, 2017
@tzerk
New option to set maximum number of tries (#8)
0ed0e66 
'faillimit' specifies the threshold value of failed checks that must be reached to procede with the exit routine
tzerk pushed a commit that referenced this issue Mar 3, 2017
New option to set maximum number of tries (#8)
faea0f8 
'faillimit' specifies the threshold value of failed checks that must be reached to procede with the exit routine
@tzerk
Copy link
Owner Author

tzerk commented Mar 3, 2017

Should work as intended. Re-open this issue in case of unexpected side-effects.

@tzerk tzerk closed this as completed Mar 3, 2017
tzerk pushed a commit that referenced this issue Mar 3, 2017
reset penalty only requires checking the connection (#8)
918afc8 
If connection is true process must also be true, so no need to check both. Furthermore, now the penalty also resets when the process is restarted.
tzerk added a commit that referenced this issue Mar 3, 2017
@tzerk
Include 'faillimit' option (#8)
4d12c9f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tzerk tzerk

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tzerk