64 replies to this topic

[Loui (PD1LOI)]

Leverd weer een totale crash op.

Deze is jou schuld

Python klaagt over verkeerde indentatie. Dat plusje aan het begin is van de diff, die hoort niet in de code natuurlijk...

Je had een gedeelte geplaatst, maar misschien was het niet gekopieerd. Maar goed, heb de backup van de file nog? Dus was geen probleem.

[WanWizard]

Nee, ik had een diff geplaatst, die aangaf wat je had moeten toevoegen aan jou lokale code om het te testen. Ik ging er even vanuit dat je wel zou weten wat een diff was (assumpties en zo...).

 

De bedoeling is dus dat je die twee regels toevoegd (zoals je gedaan hebt), maar wel zonder dat plusje aan het begin van de regel, die geeft alleen maar aan dat die regel in de diff is toegevoegd, dat is geen deel van de code.

[Tech]

De diff zoals Wanwizard die gekopieerd heeft, is een patch.

Als je een file patched, geef je met een + aan dat er regels moeten worden toegevoegd, met een - geef je aan dat er regels weg moeten worden gehaald, het patch commando voegt dan de regels toe of haalt ze weg maar haalt ook de + voor de regel weg, die hoort er niet in de code te komen.

[Loui (PD1LOI)]

Nee, ik had een diff geplaatst, die aangaf wat je had moeten toevoegen aan jou lokale code om het te testen. Ik ging er even vanuit dat je wel zou weten wat een diff was (assumpties en zo...).

De bedoeling is dus dat je die twee regels toevoegd (zoals je gedaan hebt), maar wel zonder dat plusje aan het begin van de regel, die geeft alleen maar aan dat die regel in de diff is toegevoegd, dat is geen deel van de code.

Soms is het voor mij lastig om terug kijken. Ik vergeet de laatste tijd wel veel, (ouderdom verschijnselen) en een tia dat ik heb gehad bevordert ook niet alles. Maar ik kom er wel uit gewoon doorgaan.

[luisteraar]

Zeker kom je er wel uit twijfel in niet aan.

WW heeft deze wijziging doorgegeven

 

in file merlinmusicplayer/src/plugin.py

 

def __init__(self, url, method='GET', postdata=None, headers=None,

agent="SHOUTcast", timeout=0, cookies=None,
followRedirect=1, lastModified=None, etag=None):
 if type(url) is str:
 url = bytes(url, 'utf-8')
HTTPClientFactory.__init__(self, url, method=method, postdata=postdata,
headers=headers, agent=agent, timeout=timeout, cookies=cookies, followRedirect=followRedirect)

 

 

Je zal in de file zien dat twee regels toegevoegd zijn nl

 if type(url) is str:

     url = bytes(url, 'utf-8')

[Loui (PD1LOI)]

Nee gaat oo net goed.

Deze regel s dus toegevoegd

if type(url) is str:

   url = bytes(url, 'utf-8')

Attached Files

•  enigma2_crash_1694431356.log   1.38KB   1 downloads

[luisteraar]

Probleem zit hier

File "/usr/lib/enigma2/python/Plugins/Extensions/MerlinMusicPlayer/plugin.py", line 295

        if type(url) is str:

        ^

IndentationError: expected an indented block

 

dus rond de  if

[Loui (PD1LOI)]

Kun je even een zetje geven,door die plugin.py aan te passen?
Ik begrijp het even niet.

[WanWizard]

Soms is het voor mij lastig om terug kijken. Ik vergeet de laatste tijd wel veel, (ouderdom verschijnselen) en een tia dat ik heb gehad bevordert ook niet alles. Maar ik kom er wel uit gewoon doorgaan.

 

Ik weet er alles van, ik zit in hetzelfde schuitje Doorgaan is de enige manier om de resterende grijze cellen in beweging te houden....

 

Hier is de hele file, dat is wellicht makkelijker.

Attached Files

•  plugin.py   147.23KB   2 downloads

[Loui (PD1LOI)]

Soms is het voor mij lastig om terug kijken. Ik vergeet de laatste tijd wel veel, (ouderdom verschijnselen) en een tia dat ik heb gehad bevordert ook niet alles. Maar ik kom er wel uit gewoon doorgaan.

Ik weet er alles van, ik zit in hetzelfde schuitje Doorgaan is de enige manier om de resterende grijze cellen in beweging te houden....

Hier is de hele file, dat is wellicht makkelijker.

Ja de ene dag is er geen probleem, de volgende dag weet je soms niet waar je mee bezig bent, lastig soms.

[WanWizard]

Ja de ene dag is er geen probleem, de volgende dag weet je soms niet waar je mee bezig bent, lastig soms.

 

Ik probeer een klus zo veel mogelijk af te ronden, of in stukken te knippen als deze groot is. En verder alles opschrijven.

 

Het lastigste is nog wel dat ik tegenwoordig voor alles een manual nodig heb, omdat ik syntax niet meer goed kan onthouden. En dat is frustrerend vertragend.

[Loui (PD1LOI)]

 

Soms is het voor mij lastig om terug kijken. Ik vergeet de laatste tijd wel veel, (ouderdom verschijnselen) en een tia dat ik heb gehad bevordert ook niet alles. Maar ik kom er wel uit gewoon doorgaan.

 

Ik weet er alles van, ik zit in hetzelfde schuitje Doorgaan is de enige manier om de resterende grijze cellen in beweging te houden....

 

Hier is de hele file, dat is wellicht makkelijker.

 

Het probleem is nog niet verholpen, zodra ik op de (INFO) druk tijdens het afspelen krijg ik een groen scherm.

Attached Files

•  enigma2_crash_1694433918.log   2.56KB   1 downloads

Edited by Loui (PD1LOI), 11 September 2023 - 13:13.

[WanWizard]

Het zou ook de method nog kunnen zijn, gegeven het feit dat de crash op treed in _validMethod.

Attached Files

•  plugin.py   147.29KB   2 downloads

[Loui (PD1LOI)]

Het zou ook de method nog kunnen zijn, gegeven het feit dat de crash op treed in _validMethod.

Nee het zit volgens mj meer n de commando naar info 

 enigma2_crash_1694435339.log   1.83KB   3 downloads

Edited by Loui (PD1LOI), 11 September 2023 - 13:39.

[luisteraar]

Mijn python kennis is bijna nul.

Ik lees iets van het moet een byte order zijn en geen string.

In C ga je dan denken om een pointer door te geven

[WanWizard]

Het probleem is dat er in Python2 geen onderscheid is tussen "bytes" (een array met 8-bit waardes) en "str" (een string waarde met tekst in een bepaald characterset), terwijl Python 3 dat verschil wel heeft.

 

En dus moet je nu continue bytes omzetten naar string of andersom, een grote ellende in bestaande code.

Dus:

# is een array-of-bytes ook een string?
>>> b"test" == "test"
False

# omzetten van string naar een array-of-bytes
>>> "er stîget ûf mit grôzer kraft".encode('utf-8')
b'er st\xc3\xaeget \xc3\xbbf mit gr\xc3\xb4zer kraft'

# een string operatie op een array-of-bytes
>>> b'abc'.replace('b', 'z')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: a bytes-like object is required, not 'str'

# een string operatie op een array-of-bytes geconverteerd naar string
>>> b'abc'.decode('utf-8').replace('b', 'z')
'azc

Nee het zit volgens mj meer n de commando naar info

 

Dit is een hele andere, maar wel van hetzelfde soort:

  File "/usr/lib/enigma2/python/Plugins/Extensions/MerlinMusicPlayer/plugin.py", line 1555, in startRun
    text = getEncodedString(self.getLyricsFromID3Tag(audio)).replace("\r\n", "\n")
TypeError: a bytes-like object is required, not 'str'

het probleem is hier dat getEncodedString() een byte object ( = encoded string ) retourneert, terwijl repalce() een string object verwacht.

[WanWizard]

Volgende fix

Attached Files

•  plugin.py   147.34KB   6 downloads

[Loui (PD1LOI)]

Volgende fix

Goedemorgen je bent er bijna, dit zie je op 8.3 onder de knop info   4097_0_0_0_0_0_0_0_0_0_20230912080145.jpg   134.78KB   1 downloads

 

En dit zie je onder de knop info bij 9.0 zelfde nummer  4097_0_0_0_0_0_0_0_0_0_20230912075353.jpg   52.31KB   2 downloads

Dus er zit nog een klein foutje in. Geen crash meer.

Edited by Loui (PD1LOI), 12 September 2023 - 07:20.

[WanWizard]

Kun je hetzelfde nog een keer testen, maar dan met Enigma in debug modes?

 

Hopeljik wordt de complete fout dan in de debug log opgenomen, ik ben wel beniewd wat er komt na "argument 2 of type..." en waar die fout op treed.

[WanWizard]

Een mogelijke oplossing:

 

Attached Files

•  plugin.py   147.35KB   3 downloads