Loosing scheduling

Bug #276752 reported by Tomáš Pikálek
4
Affects Status Importance Assigned to Milestone
Me TV
Fix Released
High
Unassigned

Bug Description

I compiled 0.6.4 in Ubuntu 8.04 and I have a problem with record scheduling. Sometimes it is not saved to the list (and of course not recorded) at all. Sometimes when it is saved, the scheduling is lost when I change a channel.

Revision history for this message
Michael Lamothe (lamothe-deactivatedaccount-deactivatedaccount) wrote :

I can't say that I've ever seen this. Are you able to provide any reproducible pattern to this?

Revision history for this message
Tomáš Pikálek (tpikalek) wrote :

I tried this again in the morning and everything went good, I was able to schedule recording of everything.
But now I can't schedule again. Look at this video:
http://tpikalek.netuje.cz/tmp/me-tv.ogg

Revision history for this message
Michael Lamothe (lamothe-deactivatedaccount-deactivatedaccount) wrote :

Thanks for the video, it was really useful. I think that I can see the issue. Can you please pay careful attention to the "Start time" (next to the date picker) in the scheduled recording dialog. Is it reporting 12 hours out?

For your information, Me TV will automatically remove scheduled recordings that occur in the past.

Do you think that I'm on the right path?

Revision history for this message
Tomáš Pikálek (tpikalek) wrote :

Yes, you are right. I didn't see this, but now I see that if I want to record something in the afternoon, Me TV wants to record 12 hours in the past.

So, Me TV should read from system settings if 24-hours time is used and use it.

Revision history for this message
Michael Lamothe (lamothe-deactivatedaccount-deactivatedaccount) wrote :

It doesn't really work like that. I simply ask the date/time control to give me the UTC time as an integer.

I don't understand why this is happening. Can you please try 0.6.5 when it comes out?

Revision history for this message
Michael Lamothe (lamothe-deactivatedaccount-deactivatedaccount) wrote :

I've made numerous changes to the timing code in 0.6.5 that might fix this.

Changed in me-tv:
importance: Undecided → Medium
milestone: none → station-6
status: New → In Progress
Revision history for this message
Michael Lamothe (lamothe-deactivatedaccount-deactivatedaccount) wrote :

I'm currently at revision 219 which should resolve the issue.

Changed in me-tv:
status: In Progress → Fix Committed
Revision history for this message
Tomáš Pikálek (tpikalek) wrote :

I'm afraid it's not fixed in revision 219. I also tried it on Ubuntu 8.10 Beta and the problem is still occurring.

Revision history for this message
Michael Lamothe (lamothe-deactivatedaccount-deactivatedaccount) wrote :

Can you please describe the conditions that caused this bug. i.e. What time was the program? What time of day was it when you tried to record? How long did the requested program go for?

I've tried to replicate but I'm not having much luck.

Changed in me-tv:
assignee: nobody → lamothe
status: Fix Committed → In Progress
Changed in me-tv:
status: In Progress → Incomplete
Revision history for this message
Jeff Bailes (thepizzaking) wrote :

I'm getting the same problem here. This happens with all programs I try to record in the afternoon. If I set a programme to record in the afternoon, it sets the start time as 12 hours earlier. This is running 0.7.0.

Revision history for this message
Tomáš Pikálek (tpikalek) wrote :

I recorded another video:
http://tpikalek.netuje.cz/tmp/me-tv2.ogg
I also have looked in "me-tv.db" database. When I want to record something in the afternoon, it also writes wrong time (12 hours fewer) in the database.
When I click on the program in the afternoon and then I manually change the time item, it records ok.

And I have a question - when I use 12-hours time and I want to record manually, how Me-TV know if I mean 9:30 AM or 9:30 PM? Maybe there is the problem.

Revision history for this message
Michael Lamothe (lamothe-deactivatedaccount-deactivatedaccount) wrote :

I now see the confusion with the AM/PM thing. Let me clear some of it up. The Date/Time control is a standard GNOME control, I didn't write it and I don't know why it's not showing you the AM/PM. It does on my system.

Given that Jeff also has this issue, this is obviously something that is not specific to you or your time zone. I'm definitely looking into it. I now have enough information, thank you.

In the meantime I guess you at least have a work around which is to set it manually. Sorry for the inconvenience. This will have to be fixed for 0.7.1 release.

Changed in me-tv:
importance: Medium → High
status: Incomplete → Confirmed
Revision history for this message
Tomáš Pikálek (tpikalek) wrote :

For now I added this line to glade file to "date_edit_start_time" which forces 24-hour format and everything works fine:
<property name="dateedit_flags">GNOME_DATE_EDIT_SHOW_TIME|GNOME_DATE_EDIT_24_HR</property>

But it is only a partial solution, because now it uses 24-hour format even if your system uses 12-hour format.

Revision history for this message
Michael Lamothe (lamothe-deactivatedaccount-deactivatedaccount) wrote :

Hi Tomáš,

I've confirmed with Jeff that his GnomeDateEdit control does not have AM/PM either. This must be a GNOME control issue, not a Me TV issue. I don't know why mine works. Are you guys on 64 bit maybe?

Thanks,

Michael

Revision history for this message
Jeff Bailes (thepizzaking) wrote :

No, I'm running 32-bit (pentium M). Could it possibly have something to do with the locale? (en_GB here) Though I'm not sure how that would have such a substantial effect on a program's scheduling.

Revision history for this message
Michael Lamothe (lamothe-deactivatedaccount-deactivatedaccount) wrote :

I think that maybe there's an issue with the GnomeDateEdit AM/PM thing that might be related to your locale e.g. maybe it's not translated. When the control parses the time field it doesn't know if it's AM or PM. I think that this is a bug we should send up to GNOME.

Revision history for this message
Michael Lamothe (lamothe-deactivatedaccount-deactivatedaccount) wrote :

In 0.7.1, I've implemented a 24 hour workaround that can be turned off in the preferences dialog.

Changed in me-tv:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.