Rockbox

Tasklist

FS#7682 - Talking alarm

Attached to Project: Rockbox
Opened by Daniel Dalton (ddalton) - Friday, 31 August 2007, 10:37 GMT
Last edited by Linus Nielsen Feltzing (linusnielsen) - Thursday, 04 October 2007, 08:57 GMT
Task Type Patches
Category User Interface
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Voice the alarm under the system menu.
It wasn't even very hard!
Anyway if your player has a wake up rtc and you go to wake up alarm it will start talking. So you can now set the alarm with the voice and hear if it has worked or not. You should also here errors if they occur.
Since I wrote this patch for a friend I haven't got an ipod to test on. So I have just been testing on the sim.
Anyway let me know how it goes.
Here it is...
This task depends upon

Closed by  Linus Nielsen Feltzing (linusnielsen)
Thursday, 04 October 2007, 08:57 GMT
Reason for closing:  Accepted
Comment by Daniel Dalton (ddalton) - Friday, 31 August 2007, 12:17 GMT
Just removed the voice string for a lang string that wasn't ever spoken. And a snprintf from ID2P to str. Back to normal. Here is the complete patch.
Comment by Daniel Dalton (ddalton) - Friday, 31 August 2007, 22:56 GMT
Sorry I forgot to make it talk when entering the alarm screen. Don't know how I missed that.
Also set a gui_syncsplash to 2 seconds if voice menus isn't on otherwise it is 4 seconds.
It was two seconds before.
Please test...
Comment by Daniel Dalton (ddalton) - Saturday, 22 September 2007, 06:35 GMT
Ok a few fixes:
-There was one case where the voice would talk when talk menus was disabled (Fixed)
-The voice can speak all of what it needs to now unlike before where it was interupted When it is telling you how long until the player will wake up.
-When exiting the alarm screen and setting your alarm the time it gave you was wrong.
I divided togo to get the hours and then by 60 to get the minutes. I should have done the operations to togo in the talk_value statements. But now I just speak the h variable which is hours and m which is minutes.
It works good here with
FS#7814
on a h300.
Please let me know if there are any problems with it.
My update is attached...
Thanks.
Comment by Daniel Dalton (ddalton) - Sunday, 23 September 2007, 05:00 GMT
Attached is hopefully a fix for saying how long the alarm is set.
Before when setting the alarm and pressing ok the voice would be interupted and couldn't finish saying how long until the alarm goes off. I think I have fixed this problem.

Also just cleaned up a bit of code and removed some braces that weren't needed.

I have one question:
When you change the value of the minutes (up or down on the h300) and it gets to 55 then you press up again and the hour goes up by one should the voice say the value of the hour?
Or should it just voice the value of the minutes being incromented and you can check the hours manually. So for example if I was pressing up should it say:
45 minutes
50 minutes
55 minutes
2 hours 0 minutes.
Or like it is now:
45 minutes
50 minutes
55 minutes
0 minutes.
So please let me know what you think.
Is it worth the extra code? Devs?

Also can people please tell me if I have fixed the voice interuption issue?
Here is the patch...

Comment by Daniel Dalton (ddalton) - Sunday, 23 September 2007, 05:01 GMT
Sorry It didn't upload.
Here it is...
Comment by Daniel Dalton (ddalton) - Sunday, 23 September 2007, 08:37 GMT
I have written up a patch to now do what I said above:
-Before it would say the following:
50 minutes
55 minutes
0 minutes.

I didn't find that very helpfull.
-Now:
50 minutes
55 minutes
2 hours 0 minutes

is this what everyone wants?
Here it is anyway...

Loading...