Further additions to the Gotham addon

[sub]

2013-11-23 12:01:29.814 [DEBUG][3] Got Web Request (192.168.10.115): /service method=recording.recurring.save&event_id=1217487&recurring_type=2&keep=0&pre_padding=1&post_padding=2&directory_id=[Default]&sid=e422774acfbe4899ab780dab4ba3de86
...
2013-11-23 12:01:29.814 [ERROR][3] event_id is invalid

It thinks the event_id is invalid. I wonder if the EPG information is stale, and there is no longer an event with that ID. Does it help if you go to Settings->Live TV->EPG then click "reset the EPG database", then try again?

[mvallevand]

Does it work if you reset your PVR database in XBMC?

Edit: sub beat me to that one.

Martin

[KRA]

Reset epg database seems to have done the trick.

I did a test and it worked as intended, albeit not perfect.

I choose this as a test : !!

19:30 - 20:00 Beavis and Butthead
20:00 - 20:30 Beavis and Butthead
20:30 - 21:00 Beavis and Butthead

I was in xbmc and choose the 20:00 to 20:30 show, and record all new episodes and to keep 3 of them. This was successful, however..

1. xbmc only marks the first of the 3 as scheduled for recording.
2. NEWA marks the middle one (the one I used) for schedule.
3. NPVR gui reports all three (as it's supposed to)

Not a big issue, although a little confusing.

[mvallevand]

@KRA, the way that clients reload recordings varies but xbmc often takes a full restart. The touch interface used by X-NEWA Chameleon doesn't reload after setting a recording so I have to exit to be able to watch in progress recordings that I set up.

Martin

[Swift_gti]

@sub, thanks for the updates - looking forward to testing series record.

Is there any chance of getting the addon 'group aware' ? - ie. when the XBMC addon is configured allow a specific channel group to be specified as the list of channels to use?

The Mediaportal addon has something similar and it is really useful for multiple client setups where some clients can handle HD and some can't (since you can specify different channel sources for each).

[mvallevand]

I've attached another updated Windows build of the Gotham addon. Main change in this update is provide more options while setting up a recurring recording.

- added support for EDL (though not currently called by XBMC)

I have been able to load EDL files and get really good comskip via http with some trivial changes to XBMC and some changes in pvrclient-nextpvr.cpp

sub could you recompile and upload your Windows plugin with this change I suggested (I'll go back to atoi() for the old gcc's) to see if that the only thing that's made a difference.

Code:

PVR_EDL_ENTRY entry;
          entry.start = atoi(pCommercialNode->FirstChildElement("start")->FirstChild()->Value()) * 1000;
          entry.end = atoi(pCommercialNode->FirstChildElement("end")->FirstChild()->Value()) * 1000 ;
          entry.type = PVR_EDL_TYPE_COMBREAK;
          entries[index] = entry;
          index++;

Martin

[sub]

I have been able to load EDL files and get really good comskip via http with some trivial changes to XBMC and some changes in pvrclient-nextpvr.cpp

sub could you recompile and upload your Windows plugin with this change I suggested (I'll go back to atoi() for the old gcc's) to see if that the only thing that's made a difference.

Code:

PVR_EDL_ENTRY entry;
          entry.start = atoi(pCommercialNode->FirstChildElement("start")->FirstChild()->Value()) * 1000;
          entry.end = atoi(pCommercialNode->FirstChildElement("end")->FirstChild()->Value()) * 1000 ;
          entry.type = PVR_EDL_TYPE_COMBREAK;
          entries[index] = entry;
          index++;

Martin

Here is the latest build, with that change, and the source code.

[mvallevand]

Here is the latest build, with that change, and the source code.

Ok still not working but the test in line 865 should be !=

if (strstr(response, "<rsp stat=\"ok\">") != NULL)

Martin

[sub]

Ok still not working but the test in line 865 should be !=

if (strstr(response, "<rsp stat=\"ok\">") != NULL)

lol - right you are. Attached.

[mvallevand]

Great, now let's see if I can get it inton xbmc

Code:

22:07:10 T:46932   DEBUG: CEdl::AddCut - Pushing new cut to back [00:00:03.000 - 00:00:06.000], 3
22:07:10 T:46932   DEBUG: CEdl::ReadPvr - Added break [00:00:03.000 - 00:00:06.000] found in PVRRecording for: http://172.16.3.2:8866/live?recording=3115.
22:07:10 T:46932   DEBUG: CEdl::AddCut - Pushing new cut to back [00:02:46.000 - 00:06:32.000], 3
22:07:10 T:46932   DEBUG: CEdl::ReadPvr - Added break [00:02:46.000 - 00:06:32.000] found in PVRRecording for: http://172.16.3.2:8866/live?recording=3115.
22:07:10 T:46932   DEBUG: CEdl::AddCut - Pushing new cut to back [00:12:59.000 - 00:15:04.000], 3
22:07:10 T:46932   DEBUG: CEdl::ReadPvr - Added break [00:12:59.000 - 00:15:04.000] found in PVRRecording for: http://172.16.3.2:8866/live?recording=3115.
22:07:10 T:46932   DEBUG: CEdl::AddCut - Pushing new cut to back [00:23:26.000 - 00:25:21.000], 3
22:07:10 T:46932   DEBUG: CEdl::ReadPvr - Added break [00:23:26.000 - 00:25:21.000] found in PVRRecording for: http://172.16.3.2:8866/live?recording=3115.
22:07:10 T:46932   DEBUG: CEdl::AddCut - Pushing new cut to back [00:30:11.000 - 00:31:10.000], 3
22:07:10 T:46932   DEBUG: CEdl::ReadPvr - Added break [00:30:11.000 - 00:31:10.000] found in PVRRecording for: http://172.16.3.2:8866/live?recording=3115.
22:07:10 T:46932   DEBUG: CEdl::AddCut - Pushing new cut to back [00:32:47.000 - 00:32:50.000], 3
22:07:10 T:46932   DEBUG: CEdl::ReadPvr - Added break [00:32:47.000 - 00:32:50.000] found in PVRRecording for: http://172.16.3.2:8866/live?recording=3115.
22:07:10 T:46932   DEBUG: CEdl::MergeShortCommBreaks - Removing short commercial break at start [00:00:03.000 - 00:00:06.000]. <5 seconds
22:07:10 T:46932   DEBUG: CEdl::AddSceneMarker - Inserting new scene marker: 00:02:46.000
22:07:10 T:46932   DEBUG: CEdl::AddSceneMarker - Inserting new scene marker: 00:06:32.000
22:07:10 T:46932   DEBUG: CEdl::AddSceneMarker - Inserting new scene marker: 00:12:59.000
22:07:10 T:46932   DEBUG: CEdl::AddSceneMarker - Inserting new scene marker: 00:15:04.000
22:07:10 T:46932   DEBUG: CEdl::AddSceneMarker - Inserting new scene marker: 00:23:26.000
22:07:10 T:46932   DEBUG: CEdl::AddSceneMarker - Inserting new scene marker: 00:25:21.000
22:07:10 T:46932   DEBUG: CEdl::AddSceneMarker - Inserting new scene marker: 00:30:11.000
22:07:10 T:46932   DEBUG: CEdl::AddSceneMarker - Inserting new scene marker: 00:31:10.000
22:07:10 T:46932   DEBUG: CEdl::AddSceneMarker - Inserting new scene marker: 00:32:47.000
22:07:10 T:46932   DEBUG: CEdl::AddSceneMarker - Inserting new scene marker: 00:32:50.000
22:07:10 T:46932   DEBUG: CEdl::WriteMPlayerEdl - MPlayer EDL file written to: special://temp/xbmc.edl
....
22:07:22 T:46932   DEBUG: CDVDPlayer::CheckAutoSceneSkip - Clock in commercial break [00:02:46.000 - 00:06:32.000]: 00:02:46.015. Automatically skipping to end of commercial break (only done once per break)
22:07:22 T:46932   DEBUG: demuxer seek to: 392001
...
22:07:25 T:46932   DEBUG: CDVDDemuxFFmpeg::SeekTime - seek ended up on time 391991
22:07:25 T:46932   DEBUG: demuxer seek to: 392001, success

Martin