Project

General

Profile

Bug #2700

Picon name generation is not correct

Added by Akos Sz over 9 years ago. Updated over 9 years ago.

Status:
Fixed
Priority:
Normal
Category:
Configuration
Target version:
-
Start date:
2015-03-01
Due date:
% Done:

100%

Estimated time:
Found in version:
Legacy, present on 3.9.2496 Openelec addon (4.3.11)
Affected Versions:

Description

Hi,

I assume that Tvheadend picon file names should be fully compatible with Enigma 2 systems. However there are differences in file name generation which results that picons migrated from Enigma box will not be shown or only a part of them will be shown due to file name mismatch.

I found this explanation of picon filename:

__What does an example file name pointing to the .png file look like? 1_0_1_957_327B_3277_6400000_0_0_0.png = the Plicon of 1600e_golden_west_network.png

The file name, what does it mean?
1 = reftype, fixed
0 = flags, fixed
1 = servicetype (1=tv)
957 = SID
327B = transponder id
3277 = network id
6400000 = namespace (orbital position 1600 --> 640 hex)
0 = parent SID
0 = parent transponder id
0 = unused_
_

The main problem is with satellites on the west side. e.g. Thor 0,8W picon generated filename in Tvheadend look like this: 1_0_1_7601_2BE_600_FFF70000_0_0_0.png. The Enigma version is: 1_0_1_7601_2BE_600_E0800000_0_0_0.png

The difference is that TVH assigns FFF7 -> (-9 decimal?) while the picon has E08 -> 3592 (359.2 degree East)
So the problem is with the calculation of orbital position. This is a bug.

On the other hand the namespace part of the picon name is not always the satellite position with traling 0s. It sometimes contains discrimination values which makes possible to distuingish between two DVB services having similar SID/TID/NID.
Unfortunately I could not find a reliable algorithm how to generate the rigth DVB namespace value.


Files

picons_sample.zip (90.6 KB) picons_sample.zip Akos Sz, 2015-03-01 17:26
picons.png (32.1 KB) picons.png jenda benda, 2015-03-10 10:03
i1.JPG (27.9 KB) i1.JPG saen acro, 2015-03-19 02:12

History

#1

Updated by Jaroslav Kysela over 9 years ago

  • Status changed from New to Fixed

Fixed in v3.9-2506-g1bd1564 .

#2

Updated by saen acro over 9 years ago

Jaroslav Kysela wrote:

Fixed in v3.9-2506-g1bd1564 .

on some services there is also strange naming
on hellas sat 2 39 degree

TVH search for 1_0_1_6E_4_1_1860000_0_0_0.png
in enigma for 1_0_1_6E_4_1_1863115_0_0_0.png

#3

Updated by saen acro over 9 years ago

Jaroslav Kysela wrote:

Fixed in v3.9-2506-g1bd1564 .

on some services there is also strange naming
on hellas sat 2 39 degree

TVH search for 1_0_1_6E_4_1_1860000_0_0_0.png
in enigma2 for 1_0_1_6E_4_1_1863115_0_0_0.png
same channel Alpha TV
it happens in some other services on same satellite.

#4

Updated by Akos Sz over 9 years ago

saen acro wrote:

Jaroslav Kysela wrote:

Fixed in v3.9-2506-g1bd1564 .

on some services there is also strange naming
on hellas sat 2 39 degree

TVH search for 1_0_1_6E_4_1_1860000_0_0_0.png
in enigma2 for 1_0_1_6E_4_1_1863115_0_0_0.png
same channel Alpha TV
it happens in some other services on same satellite.

This is what I meant when mentioned namespace and trailing zeros. Check my original post and attachment.

#5

Updated by Jaroslav Kysela over 9 years ago

The hash calculation is here:

https://github.com/libo/Enigma2/blob/master/lib/dvb/frontend.cpp#L398-L419

But to make things more complicated, the low 16-bits are zeroed when ONID and TSID are validated:

https://github.com/libo/Enigma2/blob/master/lib/dvb/scan.cpp#L56-L157

#7

Updated by Jaroslav Kysela over 9 years ago

  • Status changed from Fixed to Accepted
  • Assignee set to Jaroslav Kysela
#8

Updated by jenda benda over 9 years ago

Hi there,

are you sure it's fixed? I have installed latest git and generated name is still the same.

I tried to reset icons/save settings but the picon path will become the same. Is there a way how should I reset it or the picon name is still generated in wrong way?
Please see the attached picture.

Thanks a lot
Kind regards
j.

#9

Updated by Jaroslav Kysela over 9 years ago

jenda benda wrote:

Hi there,

are you sure it's fixed? I have installed latest git and generated name is still the same.

I tried to reset icons/save settings but the picon path will become the same. Is there a way how should I reset it or the picon name is still generated in wrong way?
Please see the attached picture.

Thanks a lot
Kind regards

You should select the channel in the channel configuration, then click to 'Reset Icon' and 'Save'.. The icons should be re-generated. Also, check, if you have correct orbital position assigned to the network (network grid) before.

#10

Updated by saen acro over 9 years ago

3.9.2525 no change still zeroes
is there a possible to add some mark if picon or logo was been found

#11

Updated by Jaroslav Kysela over 9 years ago

saen acro wrote:

3.9.2525 no change still zeroes

No idea. Could you show me the network settings ?

is there a possible to add some mark if picon or logo was been found

It might be useful indeed.

#12

Updated by saen acro over 9 years ago

what settings exactly
am scanning hellas sat 39E bulsatcom pack some programs have "zeroes" problem

picon://1_0_1_6E_4_1_1860000_0_0_0.png is what is search
real is 1_0_1_6E_4_1_1863115_0_0_0

#13

Updated by Jaroslav Kysela over 9 years ago

saen acro wrote:

what settings exactly
am scanning hellas sat 39E bulsatcom pack some programs have "zeroes" problem

picon://1_0_1_6E_4_1_1860000_0_0_0.png is what is search
real is 1_0_1_6E_4_1_1863115_0_0_0

OK. So the satellite position is correct (0x186 = 390 -> 39E), the code (hacks) mentioned in comment#5 and comment#6 are not added yet to the tvheadend. So all lower values are zero at the moment. That's the reason why I re-opened the bug to take care when the SAT>IP Server code is merged.

#14

Updated by Jaroslav Kysela over 9 years ago

  • Status changed from Accepted to Fixed
  • % Done changed from 0 to 100

Applied in changeset commit:tvheadend|e247500a080292b4c2a2f562fac9c30534753e50.

#15

Updated by Jaroslav Kysela over 9 years ago

Added code from comment#6 to v3.9-2614-ge247500 .

Also available in: Atom PDF