Modify
Warning Please read the Ticket HowTo before creating or commenting on a ticket. Failure to do so may cause your ticket to be rejected or result in a slower response.

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#2811 closed enhancement (fixed)

BBC Series link patch

Reported by: dm@… Owned by: stuarta
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@… 7 years ago.
serieslink2.patch.gz (3.0 KB) - added by dm@… 7 years ago.

Download all attachments as: .zip

Change History (9)

Changed 7 years ago by dm@…

comment:1 Changed 7 years ago by stuarta

  • Owner changed from danielk to stuarta

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

Changed 7 years ago by dm@…

comment:2 Changed 7 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 7 years ago by stuarta

  • Milestone changed from unknown to 0.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 7 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 7 years ago by stuarta

  • Resolution set to fixed
  • Status changed from new to closed

(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 7 years ago by stuarta

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 7 years ago by stuarta

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

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'new'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.