E2OpenPlugins / e2openplugin-MeteoViewer

plugin for satellite's weather pictures
2 stars 8 forks source link

Images are not anymore loaded in panel. #11

Closed miazza99 closed 1 year ago

miazza99 commented 1 year ago

Open ATV 7.1 (and also 7.2). Vu+DUO4KSE and ZGEMMA 9.2HSE

Since some months images are not anymore loaded in the panel. Images are correctly downloaded but not loaded by the viewer.

Is this an issue of OpenATV image ?

ims21 commented 1 year ago

There in setting must be set valid path for downloaded files

miazza99 commented 1 year ago

Thank you for the feedback. Yes, the path is checked and valid: /tmp/ I also tried to change it but with no effect. Also if I insist to try different download the plugin crashes.

Meanwhile I tryed with OpenATV 6.4 and it works perfectly.

Possibly an issue with pyton 3 related to images resolution but I cannot debug it.

It was well working also with 7.1 until November .

ims21 commented 1 year ago

I am using it under PLi develop (py3) and all fully worked. But builders must for image with py3 used "branch" python3, not "master" from this git.

ims21 commented 1 year ago

Ÿou can try download py files from "python3" and replace it in box and restart GUI.

miazza99 commented 1 year ago

This is a good idea ; I've een there is a fix of ui.py dated 3 months ago. How can I download it ? When I click to this link: https://github.com/E2OpenPlugins/e2openplugin-MeteoViewer/blob/python3/plugin/ui.py I only get a text file.

I made a cut and paste in NotePad++ and save it as ui.py.

Same issue ,,,, and crash

11:22:21.4288 Traceback (most recent call last): 11:22:21.4289 File "/usr/lib/enigma2/python/Plugins/Extensions/MeteoViewer/ui.py", line 556, in downloadFrames 11:22:21.4302 self.downloadFiles(TYPE[self.typ]) 11:22:21.4304 File "/usr/lib/enigma2/python/Plugins/Extensions/MeteoViewer/ui.py", line 1009, in downloadFiles 11:22:21.4321 self.downloadWO(typ) 11:22:21.4322 File "/usr/lib/enigma2/python/Plugins/Extensions/MeteoViewer/ui.py", line 1336, in downloadWO 11:22:21.4343 self.deleteOldFiles(typ, gmtime(startDel)) 11:22:21.4345 File "/usr/lib/enigma2/python/Plugins/Extensions/MeteoViewer/ui.py", line 973, in deleteOldFiles 11:22:21.4362 os.unlink("%s%s" % (self.getDir(TYPE.index(typ)), x[0][0])) 11:22:21.4365 FileNotFoundError: [Errno 2] No such file or directory: '/tmp/meteo/im04//tmp/meteo/im04/202301050815.jpg' 11:22:21.4365 [ePyObject] (PyObject_CallObject(<bound method meteoViewer.downloadFrames of <class 'Plugins.Extensions.MeteoViewer.ui.meteoViewer'>>,()) failed)

miazza99 commented 1 year ago

It looks a path issue but I cannot set it differetly. '/tmp/meteo/im04//tmp/meteo/im04/202301050815.jpg'

ims21 commented 1 year ago

in Options - last item ... press OK and select f.eg. /hdd/movie, save and try again

miazza99 commented 1 year ago

Same issue different folder: 11:55:48.0074 File "/usr/lib/enigma2/python/Plugins/Extensions/MeteoViewer/ui.py", line 556, in downloadFrames 11:55:48.0084 self.downloadFiles(TYPE[self.typ]) 11:55:48.0086 File "/usr/lib/enigma2/python/Plugins/Extensions/MeteoViewer/ui.py", line 1009, in downloadFiles 11:55:48.0099 self.downloadWO(typ) 11:55:48.0101 File "/usr/lib/enigma2/python/Plugins/Extensions/MeteoViewer/ui.py", line 1336, in downloadWO 11:55:48.0117 self.deleteOldFiles(typ, gmtime(startDel)) 11:55:48.0119 File "/usr/lib/enigma2/python/Plugins/Extensions/MeteoViewer/ui.py", line 973, in deleteOldFiles 11:55:48.0132 os.unlink("%s%s" % (self.getDir(TYPE.index(typ)), x[0][0])) 11:55:48.0135 FileNotFoundError: [Errno 2] No such file or directory: '/media/hdd/movie/meteo/im04//media/hdd/movie/meteo/im04/202301050845.jpg' 11:55:48.0135 [ePyObject] (PyObject_CallObject(<bound method meteoViewer.downloadFrames of <class 'Plugins.Extensions.MeteoViewer.ui.meteoViewer'>>,()) failed)

miazza99 commented 1 year ago

Is there any way to log what the plug-in is doing in order to understand why the path is set wrongly ? The code looks OK and the files are all stored in the right `TMPDIR = "/tmp/" SUBDIR = "meteo" It is only the play process that is getting the wrong path ...

ims21 commented 1 year ago

what wrong path is in your debug "'/media/hdd/movie/meteo/im04//media/hdd/movie/meteo/im04/202301050845.jpg" ?

miazza99 commented 1 year ago

According to my understanding the right path should be only : "/media/hdd/movie/meteo/im04/202301050845.jpg"

It seems that TMPDIR and SUBDIR are repeated in the loading path with a "//" in the middle.

ims21 commented 1 year ago

but it seems, only on your image...

miazza99 commented 1 year ago

Well, I tried also with a fresh new OpenATV 7.2 and the problem is confirmed noy only by me ... other people have the same problem of images not showed. The same issue confirmed by me on Vu+DUO4KSE and ZGEMMA 9.2 H . Perfectly working on 6.4 on both decoders and I'm confidet it will work with 7.0 and also with 7.1 till October 2022 (by heart this is more or less the date it stopped to work).

This is why I would like to find a way for debugging it and uderstand why the path is considfered twice in the replay with this version of pyton.

miazza99 commented 1 year ago

Other people have the same issue: https://www.opena.tv/openatv-7-2-py3-11-openssl-3-07-beta/61102-openatv-7-2-a-post523400.html#post523400

Somehow the problem might be at row 400 and 401: def getDir(self, num_typ): return TMPDIR + SUBDIR + "/" + TYPE[num_typ] + "/"

jbleyel commented 1 year ago

The FileList.py has been changed.

miazza99 commented 1 year ago

Sorry, I'm not very familiar with py. What does it mean that The FileList.py has been changed ?

jbleyel commented 1 year ago

The usage of FileList in the plugin is wrong.

jbleyel commented 1 year ago

Hello @ims21 , please check this -> https://github.com/E2OpenPlugins/e2openplugin-MeteoViewer/pull/12

It's NOT tested !!!

miazza99 commented 1 year ago

Well done. Thanks