Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#2811 closed enhancement (fixed)

BBC Series link patch

Reported by: dm@… Owned by: Stuart Auchterlonie
Priority: minor Milestone: 0.21
Component: dvb Version: 0.20
Severity: low Keywords:
Cc: Ticket locked: no

Description

This patch interprets the programID and seriesID descriptors in the EIT that the BBC are currently transmitting and adds them to the database.

Attachments (2)

serieslink.patch (4.7 KB) - added by dm@… 17 years ago.
serieslink2.patch.gz (3.0 KB) - added by dm@… 17 years ago.

Download all attachments as: .zip

Change History (9)

Changed 17 years ago by dm@…

Attachment: serieslink.patch added

comment:1 Changed 17 years ago by Stuart Auchterlonie

Owner: changed from danielk to Stuart Auchterlonie

I'll take this as I'll be able to test it...

Changed 17 years ago by dm@…

Attachment: serieslink2.patch.gz added

comment:2 Changed 17 years ago by dm@…

I've updated this patch with a new version. It's gzipped because when I tried uploading the plain text version I got a message about rejecting spam.

This version interprets the default authority descriptor that is transmitted in the SDT and currently puts it in the xmltvid field of the channel table since that doesn't seem to be used when only EIT is used for programme data. It may be better to have a separate field for this but it's complicated because it looks like they will be transmitting it in the NIT in due course. There may be a need for an extra "default authority" field in the dtv_multiplex table as well.

The default authority is used when the CRID transmitted in the EIT begins with a "/", which all the CRIDs currently transmitted do, and is intended to make the full CRID globally unique. CRIDs follow the syntax of a URI and the default authority has the form of a DNS name e.g. fp.bbc.co.uk. It seems that several of the channels apart from the BBC have a default authority descriptor. I've seen bds.tv and communitychannel.org.

Adding the default authority to the CRIDs in the EIT does make the programID and seriesID strings rather longer. I've increased the lengths of the programID and seriesID fields in a few tables in my own database to 30 characters.

David.

comment:3 Changed 17 years ago by Stuart Auchterlonie

Milestone: unknown0.21

This works well when the EPG monkeys actually get the information right.

With the default authority, is this used in a similar way to EIT data with the EITa being the "authoritive" source for that mux, or is the information used to form the fully qualified CRID and that info is broadcast consistently across the muxes?

Stuart

comment:4 Changed 17 years ago by pdbailey@…

I've applied this patch to my backend, and it seems to be working very well for the BBC programmes I record. Finally I can have a recording rule for MotD that will shift recordings around if there is a clash, without duplication.

What other testing do you think is needed in order for this patch to be committed. I'm guessing an addition to upgrade the DB schema would be nice, in order to increase the size of programID and seriesID?

comment:5 Changed 17 years ago by Stuart Auchterlonie

Resolution: fixed
Status: newclosed

(In [12760]) Closes #2811. Implements BBC Series link functionality

With this changeset users of EIT in the UK will now have useful programid & seriesid information which allows the scheduler to better detect duplicates of shows, even when the EPG monkeys manage to describe the same episode 3 different ways.

This adds support for the default_authority that is currently being broadcast in the SDT, and stores that with the channel info.

Database Update!

This increases the size of seriesid & programid to accomodate the larger size of the CRID based seriesid & programid info. Also adds default_authority to the channel table.

UK users wishing to take full advantage of this new functionality will need to rescan their channels to populate the default authority information for each channel.

comment:6 Changed 17 years ago by Stuart Auchterlonie

Forgot to mention, Big thanks to David for the original patch. I added the database updates and made David's code use the new columns etc...

comment:7 Changed 17 years ago by Stuart Auchterlonie

(In [12768]) Refs #2811, Update [12760], one of the tables is temporary.

Note: See TracTickets for help on using tickets.