24 replies to this topic

[elsato]

Picon.py commit gives me this crash?

https://github.com/O...d5b31e02ec24a62

Traceback (most recent call last):
  File "/usr/lib/enigma2/python/Navigation.py", line 88, in dispatchEvent
    x(i)
    ~^^^
  File "/usr/lib/enigma2/python/Components/PerServiceDisplay.py", line 17, in event
    func[1](ev)
    ~~~~~~~^^^^
  File "/usr/lib/enigma2/python/Components/Sources/CurrentService.py", line 33, in serviceEvent
    self.changed((self.CHANGED_SPECIFIC, event))
    ~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/enigma2/python/Components/Element.py", line 92, in changed
    self.downstream_elements.changed(*args, **kwargs)
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^
  File "/usr/lib/enigma2/python/Tools/CList.py", line 7, in __call__
    x(*args, **kwargs)
    ~^^^^^^^^^^^^^^^^^
  File "/usr/lib/enigma2/python/Components/Converter/ServiceName.py", line 103, in changed
    Converter.changed(self, what)
    ~~~~~~~~~~~~~~~~~^^^^^^^^^^^^
  File "/usr/lib/enigma2/python/Components/Element.py", line 92, in changed
    self.downstream_elements.changed(*args, **kwargs)
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^
  File "/usr/lib/enigma2/python/Tools/CList.py", line 7, in __call__
    x(*args, **kwargs)
    ~^^^^^^^^^^^^^^^^^
  File "/usr/lib/enigma2/python/Components/Renderer/Picon.py", line 169, in changed
    pngname = getPiconName(self.source.text)
  File "/usr/lib/enigma2/python/Components/Renderer/Picon.py", line 101, in getPiconName
    series = re.sub(r"s[0-9]*e[0-9]*$", "", name)
                                            ^^^^
NameError: name 'name' is not defined. Did you mean: 'sname'?
[ePyObject] (CallObject(<bound method Navigation.dispatchEvent of <Navigation.Navigation object at 0xb04c88d0>>,(0,)) failed)

[WanWizard]

I've let Huevos know.

[elsato]

Crash is gone after changing Picon.py line 101

name to sname

series = re.sub(r"s[0-9]*e[0-9]*$", "", sname)

[elsato]

I've let Huevos know.

Thanks

[Huevos]

Fixed here:

https://github.com/O...igma2/pull/4106

 

But this series thing looks like someone's personal IPTV code has somehow entered the public image because no real channels are named like that. Why would a channel name contain "Season" and "Episode"? And of course it only works for English, and only in that one format. So, "picon not found" == "another unnecessary lookup".

Edited by Huevos, 23 November 2024 - 20:38.

[WanWizard]

Merged by Littlesat.

[WanWizard]

But this series thing looks like someone's personal IPTV code has somehow entered the public image because no real channels are named like that. Why would a channel name contain "Season" and "Episode"? And of course it only works for English, and only in that one format. So, "picon not found" == "another unnecessary lookup".

 

I agree it shouldn't be there. Apart from IPTV support (which isn't a bad idea these days), having VOD / Series as bouquet entries is a bad idea anyway.
 

[Huevos]

Well with @DimitarCC plugin there is no need for VoD in bouquets. All that stuff is in a nice menu with a search function, TMDB, etc.

 

 

Attached Files

•  1.jpg   42.62KB   2 downloads
•  2.jpg   73.53KB   2 downloads
•  3.jpg   88.69KB   2 downloads

[littlesat]

It also doesn’t belongs in normal channel lists… at least not ‘right away’

Edited by littlesat, 23 November 2024 - 21:41.

[Huevos]

It also doesn’t belongs in normal channel lists… at least not ‘right away’

It was added here: https://github.com/O...fe200dfbbca82ab

[WanWizard]

There it wasn't an issue, as the variable "name" existed. The refactor changed the variable name.

[yyvon66]

hello,

now i have green screen,

 

how i do to repair ?

 

thanks

Attached Files

•  20241125_140224.jpg   155.88KB   2 downloads

[Huevos]

Load Picon.py from the enigma repo.

[littlesat]

Or try via console/putty

 

init 4

opkg update

opkg upgrade

init 3

[ricki]

Or try via console/putty

 

init 4

opkg update

opkg upgrade

init 3

it does not fix the error

[littlesat]

Then I’m afraid you need to wait until a new build completes see the right top corner https://openpli.org/

[littlesat]

I’m afraid you need to wait a moment or manually put the latest picons.py on your box manually… today the buildserver get hardware revision.

[Tech]

I’m afraid you need to wait a moment or manually put the latest picons.py on your box manually… today the buildserver get hardware revision.

All three buildservers get a revision.

[WanWizard]

And a quick search ( or even a scan of yesterday and todays topics ) would have avoided all these duplicate messages.

 

This is development, it can break.

[ricki]

The question was whether you can do something without flashing, that was never really answered in my opinion.

Huevos wrote something about Load Picon.py from the enigma repo. Not so easy to understand for everyone