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

audio drop outs every 5 minutes #1

Closed
JohanDegraeve opened this issue Feb 19, 2018 · 9 comments
Closed

audio drop outs every 5 minutes #1

JohanDegraeve opened this issue Feb 19, 2018 · 9 comments

Comments

@JohanDegraeve
Copy link
Collaborator

If it's every 5 minutes, then that's probably due to the phone muted check. This check is done very 5 minutes, although it was originally not aligned with a reading, but maybe in your version it is.

It plays a sound also, irrespective if mute check alert is on or off.

It's probably better to do the mutecheck only if the alert is on for the time.

And maybe the method to play that sound also needs to be changed. It's different than playing an alert sound
ANE, mutechecker.m, playmutesound

@miguelkennedy
Copy link
Collaborator

miguelkennedy commented Feb 19, 2018

I haven't aligned it with the readings... But normally the AlarmService does kick in when a new reading comes and then the mutechecker fires... do you think we can play the mutechecker sound file inside the ane using the same audiosessions that we use in the spoken readings?

@JohanDegraeve
Copy link
Collaborator Author

I think yes, would need to be tried

@JohanDegraeve
Copy link
Collaborator Author

ok done

@miguelkennedy
Copy link
Collaborator

Done? You changed the ane and are testing it now?

@JohanDegraeve
Copy link
Collaborator Author

JohanDegraeve commented Feb 20, 2018 via email

@miguelkennedy
Copy link
Collaborator

OK!

JohanDegraeve added a commit that referenced this issue Feb 22, 2018
…he interrupts when listening to music in most cases. A user listening to music will normally not have his phone muted, or not have the phone muted alert on, or have it snoozed. Closes issue #1
JohanDegraeve added a commit that referenced this issue Feb 22, 2018
…ch check if phonemuted alert enabled or not. This will reduce CPU usage
@miguelkennedy
Copy link
Collaborator

Can we close this?

@JohanDegraeve
Copy link
Collaborator Author

JohanDegraeve commented Feb 23, 2018 via email

@miguelkennedy
Copy link
Collaborator

Done!

miguelkennedy pushed a commit that referenced this issue Mar 14, 2018
miguelkennedy pushed a commit that referenced this issue Jun 8, 2018
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

No branches or pull requests

2 participants