1132 replies to this topic

[littlesat]

Ik heb eigenlijk een hekel aan plugins die zitten te rommelen in /etc/enigma2.... Deze map is voor enigma2... Het wordt dan ook aldaro een rommeltje daardoor...
Mij mijkt het dan ook stukken beter dat zo'n plugin zelf een map maakt op /etc met de naam van de plugin. Of misschien nig liever een /etc/plugins/naamplugin of /etc/engima2/plugins/naamplugin... (Ivm autobackup) Ik denk dat dit veel overzichtelijker wordt...

Edited by littlesat, 19 September 2016 - 07:04.

[Loui (PD1LOI)]

Het werkt prima bij mij eindelijk nu groter radar beelden perfect zo. Gelijk een vraag aan Rob, jij bent hier net zoveel mee bezig al Caught merk ik, Is het niet beter om een eigen repro te maken en daar deze plugin naar toe te kopieren dan hoef je niet steeds mij een pull request te vragen. Niet dat ik het erg vind maar dan heb je alles in eigen beheer.

Edited by Loui (PD1LOI), 19 September 2016 - 08:31.

[Loui (PD1LOI)]

Een opmerking wil ik nog plaatsen, namelijk over Europa radar satelliet, wolken, zon en motregenen radar zijn hetzelfde als Nederland. Is dit te veranderen zodat je ook echt Europa ziet i.p.v Alleen Nederland.

[HPPli]

Dit wou ik gisteren nog melden maar er was slechte verbinding afin...

Toen Pedro gisteren vroeg vanwaar ik de temperatuur info vandaan haal had ik nog geen vermoeden.
Maar met het 2de voorval, plaatsen zoeken op IP met ("  *")) wat een groen scherm veroorzaakt dus wel.
Het komt erop neer dat Buienradar de plaatsnaam database en het converteren naar nrs. beter heeft beveiligd of heeft of aangepast, zeer recent dus.
De getoonde info is dus verkeerd, wat je ziet is de info voor Brussel.

Ik ben nu aan het kijken voor een oplossing.
Hetzij een eigen database zonder PHP (telenet webruimte), met nadeel het inlezen gebeurt dan op de box en ik weet nog niet hoe snel dit zal gaan.
Of ik moet een webserver vinden die PHP ondersteunt om een 5mb file op te plaatsen.

Of andere oplossingen bedenken, we zien wel, eerst gaan werken 

Klopt, gisteren was het Pli-board slecht bereikbaar.

 

@Caught, misschien eens kijken voor een goede bron van plaatsnamen // locaties naar de Foreca Weer Voorspelling applicatie ?

[Caught]

Een opmerking wil ik nog plaatsen, namelijk over Europa radar satelliet, wolken, zon en motregenen radar zijn hetzelfde als Nederland. Is dit te veranderen zodat je ook echt Europa ziet i.p.v Alleen Nederland.

 

Ik denk dat alle andere landen uit de EU unie gestapt zijn, of er over nadenken

Nee, dit komt omdat bij het aanmaken v/d plugin en de vernieuwde Buienradar site toen nog niet veel voor EU beschikbaar was. 
Ik zie dat er ondertussen al wat meer beschikbaar is en zal dit aanpassen. 

[Pedro_Newbie]

Gedaan: https://github.com/E...n-HetWeer/pulls

Maar hoe roep je deze aan want mijn voorbeeld zat in het script in de backupsuite aan het begin van het backupproces. Je hebt nu een losse .sh die nergens aangeroepen wordt.
In python zou iets soortgelijks gedaan kunnen worden maar in python weet ik zo de exacte syntax niet daarvoor.

Edited by Pedro_Newbie, 19 September 2016 - 15:41.

[Rob van der Does]

Tsjaaa, daar was ik dus al bang voor. Geen idee dus

[WanWizard]

Zie http://www.embeddedl...on_scripts.html over hoe dit moet.

[WanWizard]

Prima leesbare docs: http://buffalo.nas-c...ctory_structure

[Pedro_Newbie]

Ja dat verhaal is wel duidelijk zeker als je zelf een IPK maakt. Maar als "jullie" het pakketje bouwen met een "bak recept" zitten die er dus niet bij. Dit probleem had ik dus ook toen de BackupSuite rechtstreeks gebouwd werd uit de Github.
De administratie moet gedaan worden in jullie "bak recept" zoals de depends en recommends, maar dus ook de preinst en postrm etc.

Mijn, niet zo elegante, oplossing is dus om zelf maar een /var/lib/opkg/info/enigma2-plugin-extensions-backupsuite.postrm te plaatsen via de eerste keer starten van het script.
 

[Caught]

 

Dit wou ik gisteren nog melden maar er was slechte verbinding afin...

Toen Pedro gisteren vroeg vanwaar ik de temperatuur info vandaan haal had ik nog geen vermoeden.
Maar met het 2de voorval, plaatsen zoeken op IP met ("  *")) wat een groen scherm veroorzaakt dus wel.
Het komt erop neer dat Buienradar de plaatsnaam database en het converteren naar nrs. beter heeft beveiligd of heeft of aangepast, zeer recent dus.
De getoonde info is dus verkeerd, wat je ziet is de info voor Brussel.

Ik ben nu aan het kijken voor een oplossing.
Hetzij een eigen database zonder PHP (telenet webruimte), met nadeel het inlezen gebeurt dan op de box en ik weet nog niet hoe snel dit zal gaan.
Of ik moet een webserver vinden die PHP ondersteunt om een 5mb file op te plaatsen.

Of andere oplossingen bedenken, we zien wel, eerst gaan werken 

Klopt, gisteren was het Pli-board slecht bereikbaar.

 

@Caught, misschien eens kijken voor een goede bron van plaatsnamen // locaties naar de Foreca Weer Voorspelling applicatie ?

 

Toch nog eerst even de andere mogelijkheden bekijken anders wordt het terug een weekje programmeren.

En verlof heb ik niet meer

[Tech]

Ja dat verhaal is wel duidelijk zeker als je zelf een IPK maakt. Maar als "jullie" het pakketje bouwen met een "bak recept" zitten die er dus niet bij. Dit probleem had ik dus ook toen de BackupSuite rechtstreeks gebouwd werd uit de Github.
De administratie moet gedaan worden in jullie "bak recept" zoals de depends en recommends, maar dus ook de preinst en postrm etc.

Mijn, niet zo elegante, oplossing is dus om zelf maar een /var/lib/opkg/info/enigma2-plugin-extensions-backupsuite.postrm te plaatsen via de eerste keer starten van het script.
 

Dit kan je ook in de ipk regelen door postinstall scripts te gebruiken, die zet je in de control folder voordat je die inpakt.

[Pedro_Newbie]

Maar dat is nu het probleem, er wordt geen IPK meer gemaakt door de pluginmaker maar er wordt door jullie een IPK gemaakt vanuit github met een bb recipe van de bestanden die daar klaar staan, het "systeem" maakt daar dan een IPK voor de feed van. Hier zsit echter alleen een CONTROL file in.

Het maken van een postrm moet dus, lijkt mij, ook via het bb recipe gebeuren. Het hoe, dat weten jullie ongetwijfeld

 

Of om alle ellende te voorkomen ook gewoon ook de bijgeleverde pyo's van de pluginmaker mee bakken.

[Tech]

 

Het maken van een postrm moet dus, lijkt mij, ook via het bb recipe gebeuren. Het hoe, dat weten jullie ongetwijfeld

 

Klopt, die zal in de bb moeten worden gedefinieerd.

 

Maar als ik het goed begrijp moet, voordat de plugin wordt geïnstalleerd eerst even gekeken worden of de plugin folders al bestaan, en indien dat het geval is, moet die worden gewist?

[Pedro_Newbie]

Dat weet ik niet wat de makers willen, de wens was primair dat bij het verwijderen van de plugin dan ook alles weg was, dus ook de pyo waardoor de plugin na verwijderen toch gewoon in de lijst met applicaties staat en op te starten is.

[Rob van der Does]

Maar als ik het goed begrijp moet, voordat de plugin wordt geïnstalleerd eerst even gekeken worden of de plugin folders al bestaan, en indien dat het geval is, moet die worden gewist?

Nee hoor; in dat geval moet er op de normale manier worden geüpdate. En dat gaat vast wel goed, zeker nu er geen config-bestand meer wordt bijgeleverd.
Het enige waar het nog om gaat, is het elegant (en dus compleet, dus inclusief de niet geïnstalleerde maar door E2 gecompileerde pyo's)) verwijderen van de gehele plugin map als de plugin (door OPKG) wordt gede-installeerd.
Ik ken ook alleen de trucjes die je kunt gebruiken als je een compleet IKP-bestand maakt, maar niet nu er gebouwd wordt vanuit de bron.

[Caught]

Nadat ik dit meerdere malen geprobeerd had krijg ik nu die fout niet meer voor elkaar. An sich mooi, maar wel vreemd.

Ik heb toch maar de laatste regels van het log hier toegevoegd.

<235422.942> Traceback (most recent call last):
<235422.943>   File "/usr/lib/enigma2/python/Plugins/Extensions/WeerInfo/plugin.py", line 717, in updatePic
<235422.948>     if pos >= (get_image_info("/tmp/WeerInfo/00.png")[0]/550):
<235422.949>   File "/usr/lib/enigma2/python/Plugins/Extensions/WeerInfo/plugin.py", line 120, in get_image_info
<235422.950>     raise Exception("not a png image")
<235422.951> Exception: not a png image

EDIT: Het betreffende bestand, /tmp/WeerInfo/00.png, was ook binnen Windows niet te openen en had slecht een grootte van 2 kB.

 

Kreeg net even een greenscreen bij het opstarten van Nederland-> Buienradar maar ik kan het nu niet meer reproduceren, misschien kwam het plaatje niet snel genoeg.
 

/HetWeer/00.png not found!. Please contact the skin's author!
<  1254.181> Traceback (most recent call last):
<  1254.181>   File "/usr/lib/enigma2/python/Plugins/Extensions/HetWeer/plugin.py", line 703, in updatePic
<  1254.183>     if pos >= (get_image_info("/tmp/HetWeer/00.png")[0]/550):
<  1254.183>   File "/usr/lib/enigma2/python/Plugins/Extensions/HetWeer/plugin.py", line 115, in get_image_info
<  1254.184>     raise Exception("not a png image")
<  1254.184> Exception: not a png image

 

 

Een opmerking wil ik nog plaatsen, namelijk over Europa radar satelliet, wolken, zon en motregenen radar zijn hetzelfde als Nederland. Is dit te veranderen zodat je ook echt Europa ziet i.p.v Alleen Nederland.

 

Vermoedelijk is het probleem gemeld door Pedro en Frenske nu in de versie van vandaag ook opgelost.

Het kwam eigenlijk voor tijdens het updaten van de site zelf (om de 5 min 2sec lang)

 

Verder

- OnweerRadar & Satelliet voor EU toegevoegd

- Radar venster nog wat bijgewerkt (randen mooier gemaakt).

 

Nog uit te zoeken

- Plaatsnamen database

 

Wil iemand voor me deze toevoegen op Github? Ook de Images, er is nl. ééntje bijgekomen in Images/lo

 

Grts,

Attached Files

•  HetWeer20-09-16.rar   1.47MB   7 downloads

[Rob van der Does]

Sorry Caught, maar je hebt niet de laatste versie gebruikt om je wijzigingen in te maken. Als we deze update gebruiken, zijn de voorgaande updates weer teloorgegaan.
Helaas, maar dat is nu eenmaal het gevolg van het feit dat er zich nu meerdere personen met de plugin bezig houden, waar trouwens GitHub juist voor is bedoeld.
Als je dus alsjeblieft (voortaan altijd) de laatste versie van GitHub wilt plukken en daar je wijzigingen in wilt aanbrengen?

[Rob van der Does]

Sorry Caught, maar je hebt niet de laatste versie gebruikt om je wijzigingen in te maken. Als we deze update gebruiken, zijn de voorgaande updates weer teloorgegaan.
Helaas, maar dat is nu eenmaal het gevolg van het feit dat er zich nu meerdere personen met de plugin bezig houden, waar trouwens GitHub juist voor is bedoeld.
Als je dus alsjeblieft (voortaan altijd) de laatste versie van GitHub wilt plukken en daar je wijzigingen in wilt aanbrengen?

Ik heb voor nu alles bijgewerkt in dit pullrequest: https://github.com/E...-HetWeer/pull/6

[Loui (PD1LOI)]

Rob is uitgevoerd

Caught ik heb een voorstel, misschien wat overbodig maar misschien ook wel mooi. Namelijk dit

 

        <screen position="center,center" size=\""""+str(550*self.scaler-16)+""","""+str(512*self.scaler)+"""" title="HetWeer">

            <widget name="picd" position="0,0" size=\""""+str(picformat[0]*self.scaler)+""","""+str(picformat[1]*self.scaler)+"""" zPosition="5" alphatest="on"/>

        </screen>"""

 

Hier is het scherm in het midden met de radar beelden, maar mijn opmerking is, is het ook mogelijk om de video achtergrond te verwijderen dus background zwart? 

dat geeft volgens mij een duidelijker en rustiger beeld.

Edited by Loui (PD1LOI), 20 September 2016 - 09:12.