Opened 7 years ago
Closed 7 years ago
Last modified 7 years ago
#13131 closed Bug Report - General (fixed)
mythtv-setup channel editor
Reported by: | Owned by: | Peter Bennett | |
---|---|---|---|
Priority: | minor | Milestone: | 29.1 |
Component: | MythTV - Mythtv-setup | Version: | Master Head |
Severity: | medium | Keywords: | scan, channel editor |
Cc: | Ticket locked: | no |
Description
The channel editor in mythtv-setup is overwriting channel.freqid with the channel number. For DVB this is wrong. Every time you edit the channel number it overwrites the Freq/Channel?.
Typically, the default post-scan channel numbers & channel order are not acceptable so some post scan editing is required.
So now.. User has to note the existing freq channel (DVB multiplex channel) before changing channel number & then edit the Freq/Channel? (channel.freqid) back to a meaningful working value.
It is possible that scanning tuners & selecting manual updates is inserting garbage into freqid column.
Attachments (1)
Change History (14)
comment:1 Changed 7 years ago by
comment:2 Changed 7 years ago by
Resolution: | → Invalid |
---|---|
Status: | new → closed |
comment:3 Changed 7 years ago by
Actually, the problem is in master mythtv-setup dvb scanner. The dvb scanner is not setting the networkid in dB table dtv_multiplex. Or it is setting it to zero.
After directly inserting the correct (non-zero) networkid , the problem channels (all on new freq/mux) work fine.
Once the backend is free to be stopped, I'll re-scan with debug logging etc. Thanks.
comment:4 Changed 7 years ago by
In "Video sources" in setup there is a place to enter the network id. You should probably enter it there before scanning. Does that solve the problem?
comment:5 Changed 7 years ago by
Is that right.. I'm pretty sure under normal usage, we're meant to leave that setting at value -1.
In 10 yrs of mythtv & using dvb-t, I don't recall ever having to enter the networkID. The correct value for networkID is plastered all over the mythtv-setup output logs. The mythtv-setup output logging detail for tuner scanning, is excellent.
Seems networkID issue is same as 6 yr old ticket 10217.
Full scan (tuned) does not work either - error parsing command options - or similar. Sure that worked before the GUI change.
comment:6 Changed 7 years ago by
In the recent past I have tuned boxes running 0.29-fixes or 30-Pre in locations served by 3 different masts in the UK, and I haven't had to enter any netids manually. But most of my recordings are SD from DVB-T. A few months ago 'roger' stated on the dev list that, IIRC, mux parameters cannot be signalled for DVB-T2 muxes. I used to be able to use the BBC DVB-T2 mux as the starter and get almost everything else automatically. Now my home mast transmits 3 DVB-T2 muxes and the new ones are only found by a full scan or by entering freq, BW and 'DVB-T2' via the editor.
comment:7 Changed 7 years ago by
Resolution: | Invalid |
---|---|
Status: | closed → new |
Different 'roger' but re-opening for further review. This is a consequence of 65f99284fc68da0446dbdf711b6ee9bee6fd4ddf
Historically freqid has been either a channel number or a kHz frequency. Why is a mystery, but there's plenty of old code that interprets it as a frequency (Frequency Tables) https://github.com/MythTV/mythtv/blob/master/mythtv/libs/libmythtv/recorders/dtvchannel.cpp#L239
grepping "freqid" finds plenty.
As per #comment:1 DVB-T tuning still works (for me) by using the multiplex data. However it seems dangerous for the UI to start clobbering settings that have a dubious usage. Note it is clobbered merely by entry to the Edit Channel screen, not by actually changing the channel number. #13046#comment:32
This is my database after viewing Edit Channel for channels 1 & 2:
mysql> SELECT chanid,channum,freqid,sourceid,callsign FROM channel
ORDER BY chanid
LIMIT 10;
chanid | channum | freqid | callsign |
---|---|---|---|
1001 | 1 | 1 | BBC ONE N West |
1002 | 2 | 2 | BBC TWO |
1003 | 3 | 59 | ITV |
1004 | 4 | 59 | Channel 4 |
1005 | 5 | 59 | Channel 5 |
1006 | 6 | 59 | ITV2 |
1009 | 9 | 50 | BBC FOUR |
1010 | 10 | 58 | ITV3 |
1011 | 11 | 49 | Pick |
1012 | 12 | 49 | Dave |
The previous freqids appear to be transport "friendly numbers" being set at https://github.com/MythTV/mythtv/blob/master/mythtv/libs/libmythtv/frequencytables.cpp#L95 Possibly they're linked to the connected muxes #comment:6
OTOH maybe frequency usage is redundant now. An opportunity to comment it all out & see what breaks ?
comment:8 Changed 7 years ago by
Ticket #10994 may be related. Freqid is used when tuning firewire, ceton, hdhomerun prime, and I added a setting for it, because any channel added manually for these types could not be tuned as the freqid was not being set. Freqid allows you to have one channel number that is tuned by the firewire or ceton box and another number visible to the MythTV user. Not a great feature in my opinion, but unless freqid was set the channel could not be tuned.
Perhaps the code is now changing the value, which would not be good. Only when adding a channel should it default to the channel number.
comment:9 Changed 7 years ago by
Owner: | changed from JYA to Peter Bennett |
---|---|
Status: | new → assigned |
Changed 7 years ago by
Attachment: | 20170926_ticket_13131_chaneditor.patch added |
---|
Remove code that sets freqid to channel number
comment:10 Changed 7 years ago by
Milestone: | needs_triage → 29.1 |
---|---|
Status: | assigned → accepted |
The code that defaults freqid to channel number is incorrect and is setting freqid on editing of an existing channel.
Attached patch removes this code. Users will have to manually set freqid as needed when adding a channel with the channel editor.
comment:11 Changed 7 years ago by
Resolution: | → fixed |
---|---|
Status: | accepted → closed |
comment:13 Changed 7 years ago by
Owner: | changed from Peter Bennett to Peter Bennett |
---|
Please close/ignore this ticket Tuning is failing for some other reason.
Now remember the channel.freqid is not used for DVB. But I have been in habit of putting the allocated channel number into that field to make note of which channels are on each mux.
The channel.mplexid links to another table for actual tuning data. That data seems okay.