Modify

Ticket #6531 (closed Patch - Feature: Duplicate)

Opened 4 years ago

Last modified 2 years ago

[PATCH] Change in mythfilldatabase to make life easier for xmltv users,

Reported by: anonymous Owned by: stuartm
Priority: minor Milestone: 0.25
Component: MythTV - Mythfilldatabase Version: Master Head
Severity: medium Keywords: mythfilldatabase, xmltv
Cc: Ticket locked: no

Description

Hi,

I always find that configuring TV Card in MyhTV is difficult for non US citizen. What I'm usually do is:

1/ Do a first frequency scan to find the channel. 2/ Go to mythweb, and fill the xmltvid column by hand for each entry in Channel table. 3/ Launch mythfilldatabase to fill missing informations (icons, programs, etc...).

Perhaps miss something somewhere, but what hurts is Step 2 because you don't have a link between channel frequency and xmltv informations. The only link is the channel name that could be the same, or near the same.

What I propose is to modify void ChannelData::handleChannels. The goal is when we are handling a channel coming from the xmltv parser, to verify if we don't already have an entry in the Channel table without xmltvid but with the same name. If so, we can do an update of this entry, instead of doing an insert.

Laurent.

Attachments

patch.checkExistingChannelForXmlTvGrabber.diff (8.2 KB) - added by laurent@… 4 years ago.
Patch to enhance xmltv channel import
patch.checkExistingChannelForXmlTvGrabber.v1-1.diff (8.4 KB) - added by laurent@… 4 years ago.
Patch review to comply to coding standard,

Change History

Changed 4 years ago by laurent@…

Patch to enhance xmltv channel import

Changed 4 years ago by laurent@…

Patch review to comply to coding standard,

comment:1 Changed 4 years ago by stuartm

  • Owner changed from stuartm to ijr
  • Status changed from new to assigned

Don't have time to test this one.

comment:2 Changed 4 years ago by stuartm

  • Milestone changed from 0.22 to unknown

comment:3 Changed 3 years ago by robertm

  • Owner changed from ijr to stuartm

comment:4 Changed 3 years ago by stuartm

  • Owner changed from stuartm to danielk

comment:5 Changed 2 years ago by beirdo

  • Type changed from enhancement to Patch - Feature
  • Summary changed from Change in mythfilldatabase to make life easier for xmltv users, to [PATCH] Change in mythfilldatabase to make life easier for xmltv users,

comment:6 Changed 2 years ago by mdean

Seems to be a dup of #9417, which has a different patch

comment:7 Changed 2 years ago by stuartm

  • Owner changed from danielk to stuartm
  • Status changed from assigned to accepted
  • Version changed from head to Trunk Head
  • Milestone changed from unknown to 0.25

comment:8 Changed 2 years ago by stuartm

  • Status changed from accepted to infoneeded

The problem I have with fuzzy matching is the potential for false positives which would be much more serious than false negatives here. The user would probably not notice the error until they started watching recordings and found the wrong channel was recorded. At least when requiring manual intervention this is unlikely to occur.

The threshold of 0.9 is high but I'd still like to see some stats on the number of false positives in testing given the size of the sample (DVB-S in Europe would be a decent and varied sample set). This has the potential to be more useful than the patch attached to #9417 but only if it achieves an extremely low or zero error rate.

Our preferred fix for this has always been for work to be done on 'lineups' at the xmltv end but this appears to have stalled. If xmltv was responsible for matching up channel names, frequencies or service ids to the xmltvid then the results would be much more reliable. Therefore this patch would only be an interim solution at best.

comment:9 Changed 2 years ago by stuartm

  • Status changed from infoneeded to closed
  • Resolution set to Duplicate

Since I've just noticed that both tickets were opened by the same submitter with different patches I'm closing this ticket and any further submissions/replies should go to #9417. The fact that the author submitted a simpler patch without fuzzy matching 18 months after the first suggests that this version wasn't 100% satisfactory.

View

Add a 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.