333 replies to this topic

[Loui (PD1LOI)]

 

Los van of dit nu met twisted of wat het verschil kan zijn? Maar de satelliet beelden werken hier gewoon.

Trouwens de rest ook,mocht ik de crashes hebben welke jullie ondervinden zou er reeds lang een oplossing zijn van mijn kant.

Ik kan me niet voorstellen dat ik dit niet voor elkaar zou krijgen.

Akkoord groene schermen kunnen maar kunnen ook opgelost worden dacht ik.

 

Hier werkt hij ook gewoon zonder groen scherm, ook op een Mutant HD60 en de ET10K.

 

Klopt werkt ook gewoon hier op mijn HD51, maar kan zijn dat ik een dag later of een uur later weer opvraag de box wel crasht vandaar. Het ophalen van de beelden gaat soms fout.

[Caught]

Laten we de crashlog's vanaf nu hier posten.  

Dan wordt dit zo snel mogelijk bekeken.

[Jork]

Zo af en toe maar...

Een paar dagen geleden deze voor het laatst:

 

OpenPLi Enigma2 crash log

crashdate=do jun  6 13:05:20 2019
compiledate=May 26 2019
skin=PLi-FullNightHD/skin.xml
sourcedate=2019-05-24
branch=rc
rev=869dc9a
component=Enigma2

stbmodel=gigablue
stbmodel=gbquad4k
kernelcmdline=root=/dev/mmcblk0p7 rootwait rw rootflags=data=journal libata.force=1:3.0G,2:3.0G,3:3.0G coherent_poll=2M vmalloc=525m bmem=529m@491m bmem=608m@2464m
nimsockets=NIM Socket 0:
imageissue=openpli 7.1-rc %h

[gPixmap] ignore unsupported 32bpp -> 16bpp alphablend!

[gPixmap] ignore unsupported 32bpp -> 16bpp alphablend!
Traceback (most recent call last):
  File "/usr/lib/enigma2/python/mytest.py", line 210, in processDelay
    callback(*retval)
  File "/usr/lib/enigma2/python/Screens/Hotkey.py", line 624, in execHotkey
  File "/usr/lib/enigma2/python/Screens/InfoBarGenerics.py", line 1154, in runPlugin
  File "/usr/lib/enigma2/python/Plugins/Extensions/HetWeer/plugin.py", line 1224, in main
  File "/usr/lib/python2.7/urllib2.py", line 154, in urlopen
  File "/usr/lib/python2.7/urllib2.py", line 429, in open
  File "/usr/lib/python2.7/urllib2.py", line 447, in _open
  File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
  File "/usr/lib/python2.7/urllib2.py", line 1228, in http_open
  File "/usr/lib/python2.7/urllib2.py", line 1198, in do_open
urllib2.URLError: <urlopen error [Errno 110] Connection timed out>

[littlesat]

Says enough... connection time-out.... this should be catched with a try except construction... arrange one Function as urlgetter and catch errors there... and show something like picture or info cannot be retreived from the internet af this moment message instread.

Dit zegt genoeg. Time out verbinding. Dit zou ondervangen moeten worden met een ‘probeer en indien niet constructie’. Regel één functie als URLgetter en als er fouten zijn toon dan een alternatief plaatje met iets van ‘deze data is nu onbereikbaar’.

[Caught]

Ok duidelijk, er wordt vandaag voor gekeken. 

[littlesat]

Je kunt het nabootsen door oa tijdelijk een foute url op te vragen...

Edited by littlesat, 10 June 2019 - 09:04.

[Caught]

Je kunt het nabootsen door oa tijdelijk een foute url op te vragen...

In ieder geval een nieuwe versie met zoveel mogelijk vangnetten.

 

Telnet:

wget users.telenet.be/caught/HetWeer/hetweer.sh -O - | /bin/sh

[Jork]

 

Je kunt het nabootsen door oa tijdelijk een foute url op te vragen...

In ieder geval een nieuwe versie met zoveel mogelijk vangnetten.

 

Telnet:

wget users.telenet.be/caught/HetWeer/hetweer.sh -O - | /bin/sh

 

Na laatste update deze net nog gezien: (bij gebruik weerinfo)

 

Traceback (most recent call last):
  File "/usr/lib/enigma2/python/Components/ActionMap.py", line 49, in action
  File "/usr/lib/enigma2/python/Plugins/Extensions/HetWeer/plugin.py", line 1195, in go
  File "/usr/lib/enigma2/python/Plugins/Extensions/HetWeer/plugin.py", line 225, in getLocWeer
  File "/usr/lib/python2.7/urllib2.py", line 154, in urlopen
  File "/usr/lib/python2.7/urllib2.py", line 435, in open
  File "/usr/lib/python2.7/urllib2.py", line 548, in http_response
  File "/usr/lib/python2.7/urllib2.py", line 473, in error
  File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
  File "/usr/lib/python2.7/urllib2.py", line 556, in http_error_default
urllib2.HTTPError: HTTP Error 503: Service Unavailable

[Caught]

 

 

Je kunt het nabootsen door oa tijdelijk een foute url op te vragen...

In ieder geval een nieuwe versie met zoveel mogelijk vangnetten.

 

Telnet:

wget users.telenet.be/caught/HetWeer/hetweer.sh -O - | /bin/sh

 

Na laatste update deze net nog gezien: (bij gebruik weerinfo)

 

Traceback (most recent call last):
  File "/usr/lib/enigma2/python/Components/ActionMap.py", line 49, in action
  File "/usr/lib/enigma2/python/Plugins/Extensions/HetWeer/plugin.py", line 1195, in go
  File "/usr/lib/enigma2/python/Plugins/Extensions/HetWeer/plugin.py", line 225, in getLocWeer
  File "/usr/lib/python2.7/urllib2.py", line 154, in urlopen
  File "/usr/lib/python2.7/urllib2.py", line 435, in open
  File "/usr/lib/python2.7/urllib2.py", line 548, in http_response
  File "/usr/lib/python2.7/urllib2.py", line 473, in error
  File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
  File "/usr/lib/python2.7/urllib2.py", line 556, in http_error_default
urllib2.HTTPError: HTTP Error 503: Service Unavailable

 

Wat was de input hiervan, welke stad?

[Caught]

Nog een aanpassing gedaan

 

wget users.telenet.be/caught/HetWeer/hetweer.sh -O - | /bin/sh

[WanWizard]

Geef ff een seintje als de repository is bijgewerkt, dan kunnen we de build aanpassen...

[littlesat]

Staan de sources ook ergens op github?

[Caught]

De laatste versie heb ik hier geplaatst, ik heb ze eigenlijk nooit elders geplaatst.

 

https://github.com/E...nplugin-HetWeer

[WanWizard]

Wij bouwen daar ook gewoon van. Maar we pinnen release en rc dependencies (op commit hash), om te voorkomen dat een build run "work in progess" gaat bouwen en een release image instabiel maakt.

 

Dus als er een update is die wel doorgevoerd moet worden, moet de commit hash manueel aangepast worden. Die staat hier (https://github.com/O...porefs.conf#L92) voor de rc, de release branches hebben deze file ook.

Dat aanpassen kan gebeuren door middel van het indienen van een pull request (bij voorkeur, jij weet zelf het beste wanneer een update in productie kan), of via een melding hier, dan passen wij hem aan.

 

Ik heb dat nu voor deze versie gedaan: https://github.com/O...5f3385d5312ca35, dus die gaat in de rc build van vannacht mee.

[Caught]

Ja ik ben er minder thuis in github bedankt om het nodige te doen.

Belangrijkste is dat iedereen minder groene schermen heeft uiteindelijk.

Er zijn i.i.g. met de updates van vandaag wel een paar risico's gecoverd. 

[littlesat]

Hier nog wat tips.... tenminste ik vind dat het hier als oneliner leesbaarder wordt en je bent meteen van die gekke kaas variabele af die eigenlijk nergens op slaat... Ook +text en zo kan beter met een %s in een string en dat % text achter de string.

 

req = urllib2.Request("http://claudck193.193.axc.nl/hetweer.php?cn="+text)
response = urllib2.urlopen(req)
kaas = response.read()
regx = '''(.*?),(.*?),'''
match = re.findall(regx, kaas, re.DOTALL)   
 
-->
match = re.findall('''(.*?),(.*?),''', urllib2.urlopen(urllib2.Request("http://claudck193.193.axc.nl/hetweer.php?cn=%s" % text)).read(). re.DOTALL)
 
response = urllib.urlopen("http://api.buienradar.nl/data/forecast/1.1/all/"+match[0][0])
kaas = response.read()
global weatherData
weatherData = json.loads(kaas)
 
-->
weatherData = json.loads(urllib.urlopen("http://api.buienradar.nl/data/forecast/1.1/all/%s" % match[0][0]).read())
 
req = urllib2.Request("http://www.buienradar."+country+"/weerbericht")
response = urllib2.urlopen(req)
kaas = response.read()
kaas = kaas.replace("\t", "").replace("\r", "").replace("\n", "").replace("<strong>", "")
kaas = kaas.replace("<br />", "").replace("</strong>", "").replace("</a>", "")
kaas = re.sub("""<a href=".*?">""" , "", kaas)
regx = '''<div id="readarea" class="description">(.*?)</div>'''
match = re.findall(regx, kaas, re.DOTALL)
return match[0]
 
-->
return re.findall('''<div id="readarea" class="description">(.*?)</div>''',
    re.sub("""<a href=".*?">""" , "", urllib2.urlopen(urllib2.Request("http://www.buienradar.%s/weerbericht" % country)).read()\
    .replace("\t", "").replace("\r", "").replace("\n", "").replace("<strong>", ""),("<br />", "").replace("</strong>", "").\
    replace("</a>", "")),
    re.DOTALL)

[WanWizard]

Het is maar net wat je preferenties zijn.

 

Ik ben persoonlijk helemaal niet zo kapot van one-liners, zeker niet als ze idioot lang zijn. Het komt de leesbaarheid niet ten goede, het is moelijker te onderhouden, en daarom meer bug gevoelig.

[littlesat]

De kunst is om het goed te balanceren... Als voorbeeld die met die boel replaces is inderdaad niet handig... Maar misschien kan dat hele stuk handiger.

Maar een variabele kaas noemen is eigenlijk geen geweldige keuze. Noem het dan tmp of zo iets.

Edited by littlesat, 11 June 2019 - 15:01.

[Jork]

Beste Caught,

Hartelijk dank voor uw aanpassing.

Ik heb zover ik getest heb als gebruiker geen problemen meer met uw plug-in bemerkt.

Bedankt!

[Caught]

Ik heb geen probleem om een variabele aan te passen maar je moet er ook niets achter zoeken.

De variabele kaas komt van mijn zoon en werd op dat moment zo aangeleerd op school.

Maar het kon evengoed appel of lepel zijn   

 

@ Jork bedankt voor de reply.