Modify

Ticket #7486 (infoneeded Patch - Bug Fix)

Opened 3 years ago

Last modified 3 weeks ago

Channelscanner misses some transponder

Reported by: buehlmann@… Owned by: stuarta
Priority: major Milestone: 0.26
Component: MythTV - Channel Scanner Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

I'm using the 0.22-fixes branche in a dvb-c system.

If I do a tuned full-scan, the scanner misses all channels of some transponders. If I explicity scan one of the missed transponders, all channels are found but some other transponders get missed. I don't think it's a signal problem as it always misses the same transponders.

There is another issue with the channel scanner. Not sure if this belongs into a separate ticket: If I start the channel scan a second time without restarting mythtv-setup, it gets lots of conflicting channels and asks me to asign numbers to them. This does not happen if I restart mythtv-setup after every scan.

Tell me if you need additional information for this?

Attachments

output.log.gz (240.2 KB) - added by Christian Güdel <cg@…> 3 years ago.
mythtv_fix_broken_providers_with_gui.diff (7.9 KB) - added by Christian Güdel <cg@…> 2 years ago.
Updated patch that will not destroy your database schema
mythtv_fix_broken_providers_with_gui1.diff (6.3 KB) - added by Dibblah 2 years ago.
Just the above patch with the now-committed to trunk schema change removed.
mythtv_fix_broken_providers_with_gui.2.diff (6.9 KB) - added by cg@… 11 months ago.
Updated patch for current fixes/0.24
mythtv_fix_broken_providers_with_gui.3.diff (6.9 KB) - added by klaas.de.waal@… 10 months ago.
Updated patch for master/0.25 by changing VERBOSE to LOG in channelscan_sm.cpp line 205

Change History

Changed 3 years ago by Christian Güdel <cg@…>

comment:1 Changed 3 years ago by Christian Güdel <cg@…>

I have the same problem and attached a log from 'mythtv-setup -v channelscan'. MythTV does find all transports but it seems to search for channels only on the first twelve that were found in the first part of the NIT.

comment:2 Changed 2 years ago by Christian Güdel <cg@…>

The following patch fixes this issue for me. The patch allows the setting of the NIT-ID from the GUI. The setting is available from the videosource settings page.

Changed 2 years ago by Christian Güdel <cg@…>

Updated patch that will not destroy your database schema

comment:3 Changed 2 years ago by stuarta

  • Cc stuarta added

comment:4 Changed 2 years ago by stuarta

  • Owner changed from danielk to stuarta
  • Status changed from new to assigned
  • Version changed from unknown to head
  • Milestone changed from unknown to 0.23

We should be able to get something like this in for 0.23

Stuart

comment:5 Changed 2 years ago by stuarta

(In [23364]) Refs #7486. Schema change to allow us to better support broken providers.

Changed 2 years ago by Dibblah

Just the above patch with the now-committed to trunk schema change removed.

comment:6 Changed 2 years ago by Dibblah

Added a version of the above patch without the schema change. This is integrated into the GUI and still seems to function. I'm not however sure if this should be done in different classes.

comment:7 Changed 2 years ago by stuarta

  • Status changed from assigned to started
  • Milestone changed from 0.23 to 0.24

Bumping this one to 0.24 as we have frozen 0.23 for new strings for translations. As this patch adds a new translatable string it needs bumping.

comment:8 Changed 2 years ago by anonymous

For reference, this is a duplicate of bug: #3640

comment:9 Changed 22 months ago by stuartm

  • Owner changed from stuarta to stuartm
  • Status changed from started to accepted

comment:10 follow-up: ↓ 11 Changed 22 months ago by stuartm

  • Cc stuarta removed
  • Owner changed from stuartm to stuarta
  • Status changed from accepted to assigned

The attached patch does not seem to work. Scanning here I get two multiplexes inserted for every freq, one with the correct networkid and one without any networkid, none with the networkid I specified.

comment:11 in reply to: ↑ 10 Changed 21 months ago by p-we

Replying to stuartm:

The attached patch does not seem to work. Scanning here I get two multiplexes inserted for every freq, one with the correct networkid and one without any networkid, none with the networkid I specified.

For whats it's worth. I applied the 2nd patch to my 0.23 branches and it works perfect voor DVB-C from Ziggo The Netherlands

comment:12 Changed 20 months ago by anonymous

I've applied the patch to mythbuntu 10.10. Besides one single line in dvbstreamdata.h the patch does still apply without other problems. This line could easy be added manually. The patched mythbuntu backend does find all channels without any problems in the Cablecom network in Switzerland. There have been some channels that where reported as duplicate, but all other channels are working. Let me know if you'd like to have further infos.

comment:13 Changed 20 months ago by robertm

  • Milestone changed from 0.24 to 0.25

comment:14 follow-up: ↓ 15 Changed 19 months ago by klaas.de.waal@…

The patch can be applied on trunk svn 26882 with manual editing of dvbstreamdata.h. It works OK on cable network Ziggo in the Netherlands.

comment:15 in reply to: ↑ 14 Changed 19 months ago by anonymous

Replying to klaas.de.waal@…:

The patch can be applied on trunk svn 26882 with manual editing of dvbstreamdata.h. It works OK on cable network Ziggo in the Netherlands.

Too bad this got bumped to 0.25

Klaas, which of the 2 patches did you apply to 0.24 trunk? With or without the DB schema change? Could you post your amended patch here, or elaborate on your manual edit. I, and probably many others, will be using the patch on the 0.24 fixes until 0.25 arrives.

Thanks Guys, it works great.

comment:16 Changed 19 months ago by robertm

  • Ticket locked set

Please read the ticket howto.

comment:17 Changed 11 months ago by beirdo

  • Ticket locked unset

Unlocking to allow for a new patch upload.

Changed 11 months ago by cg@…

Updated patch for current fixes/0.24

Changed 10 months ago by klaas.de.waal@…

Updated patch for master/0.25 by changing VERBOSE to LOG in channelscan_sm.cpp line 205

comment:18 Changed 7 months ago by musicaze@…

This issue is over 4 years old (it replaced an older issue for some reason). It prevents me from using MythTV, since it cannot tune all my programs in Denmark. I really hope to see this in 0.25

comment:19 Changed 7 months ago by wagnerrp

  • Ticket locked set

... and locked again

comment:20 Changed 3 months ago by stuartm

  • Owner changed from stuarta to danielk
  • Priority changed from minor to major
  • Version changed from head to Master Head
  • Ticket locked unset
  • Type changed from defect to Patch - Bug Fix

comment:21 Changed 3 months ago by danielk

  • Status changed from assigned to infoneeded

Thanks for the patch.

Am I correct in understanding that the stream has the correct channels on an "Other Network" NIT table?

Is the modification required only at the time of scanning or is it required for regular use?

I don't really like having this hack in DVBStreamData. But we could handle this in the channel scanner itself without much difficulty.

comment:22 Changed 3 months ago by danielk

  • Owner changed from danielk to stuarta
  • Milestone changed from 0.25 to 0.26

I believe we may also need to add NIT id filtering if we allow setting a NIT id. I'm giving this one back to stuarta, since it really needs someone in a DVB country to look at it.

comment:23 Changed 3 months ago by klaas.de.waal@…

This patch allows you to do a "Full Scan (Tuned)" and find all channels in all multiplexes in mythtv-setup when the network ID is set correct in the "Video sources" page. The patch is only effective at the moment at the time of scanning in mythtv-setup. MythTV never rescans the services automatically. On my DVB-C capable TV I also need to enter the network ID, there is absolutely no way around it.

comment:24 Changed 3 weeks ago by bas-t <tycholursen@…>

I used mythtv_fix_broken_providers_with_gui.3.diff​ today with fixes/0.25 Works like a charm, thanks a lot!

View

Add a comment

Modify Ticket

Action
as infoneeded
Next status will be 'assigned'
Author


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

 
Note: See TracTickets for help on using tickets.