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

Stuck "in action" flag #81

Closed
denis-stepanov opened this issue May 1, 2024 · 2 comments · Fixed by #82
Closed

Stuck "in action" flag #81

denis-stepanov opened this issue May 1, 2024 · 2 comments · Fixed by #82
Assignees
Labels
bug Something isn't working
Milestone

Comments

@denis-stepanov
Copy link
Owner

Looks like "in action" option is conflicting with multi-jingle feature:

$ advent -t broadlink -n4 -i2.5 -c8 -l debug -A
AdVent v1.6.0
TV control is 'broadlink' with action 'mute' for 600 s max and 1 exit jingle
...
Warning: TV starts with action in progress: 'mute'
Recognition interval is 2.5 s with confidence of 8%
Started 4 listening thread(s)
Type 'h' for help
...
::::::::::::::::::::::::::::::::::::::::::oO
Hit: FR_TF1_230205_ESPRITSCRIMINELS1_3
Muti-jingle ignored; remaining in action
Oo:::::::::::::::::::oo::::::::::::::::::::oo::::::::::::O
Hit: FR_TF1_221019_CHICAGOMED3_3
Muti-jingle ignored; remaining in action
Ooo::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::o::::::oO
Hit: FR_TF1_220221_EVENING1_3
Muti-jingle ignored; remaining in action

I.e. it never exits the action. Similar effect if a is pressed in console.

@denis-stepanov denis-stepanov added the bug Something isn't working label May 1, 2024
@denis-stepanov denis-stepanov added this to the v1.6.1 milestone May 3, 2024
@denis-stepanov denis-stepanov self-assigned this May 3, 2024
@denis-stepanov denis-stepanov linked a pull request May 3, 2024 that will close this issue
@denis-stepanov
Copy link
Owner Author

Looks like it is not completely fixed;

  1. start with -A
  2. hit
  3. it writes that it has unmuted but actually mutes

@denis-stepanov
Copy link
Owner Author

After some testing, this was misunderstanding / wrong manipulation, so no further action.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant