Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NPO werkt niet meer #1730

Closed
2 of 7 tasks
PhyxionNL opened this issue Nov 6, 2023 · 144 comments
Closed
2 of 7 tasks

NPO werkt niet meer #1730

PhyxionNL opened this issue Nov 6, 2023 · 144 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@PhyxionNL
Copy link

PhyxionNL commented Nov 6, 2023

Bug report

Describe the bug

De lijst programma's wordt nog wel opgehaald, maar de streams willen niet meer afspelen. Ze werken wel via de NPO-app zelf (de oude versie).

To Reproduce

Steps to reproduce the behavior:

  1. Open NPO Start.
  2. Recent.
  3. Vandaag.
  4. Open een willekeurige stream.
  5. Afspelen kan niet worden gestart.

Heb het op 2x Android TV geprobeerd en 1x Windows PC, overal dezelfde fout.

Debuglog

The debug retrospect.log can be found here:

Removed pasted log

Screenshots

Additional context or screenshots (if appropriate)

Your Environment

Used Operating system:

  • Android
  • iOS
  • Linux
  • OSX
  • Raspberry-Pi
  • Windows
  • Windows UWP

Used versions:

  • Kodi version: 20.2
  • Retrospect version: laatste
@PhyxionNL PhyxionNL added the bug Something isn't working label Nov 6, 2023
@PhyxionNL
Copy link
Author

Overigens staat mijn hele log vol met "regexer.py - 130 - Re-using cached Compiled Regex object"
Ik denk dat ik wel 2000 regels eruit gehaald heb.

@basrieter
Copy link
Collaborator

Overigens staat mijn hele log vol met "regexer.py - 130 - Re-using cached Compiled Regex object" Ik denk dat ik wel 2000 regels eruit gehaald heb.

Komt door de debug logging.

@basrieter basrieter self-assigned this Nov 6, 2023
Copy link

Branch bug-1730 was created for issue: "NPO werkt niet meer"

@basrieter
Copy link
Collaborator

Zou je deze eens kunnen installeren via de "Install from zip": https://github.com/retrospect-addon/plugin.video.retrospect/archive/refs/heads/bug-1730.zip

Playback zou dan weer moeten werken.

@basrieter
Copy link
Collaborator

Wat nog niet werkt is:

  • Live content
  • NPO favorieten

@chakubie
Copy link

chakubie commented Nov 6, 2023

Bedankt werkt weer, maar kan niet inloggen dus lage resolutie en dus inderdaad ook geen favorieten.

@PhyxionNL
Copy link
Author

Heb het geprobeerd, het afspelen van recente items werkt weer, maar hij kan geen items meer ophalen bij Titels met [A-Z] (bij TV A-Z). De volledige lijst werkt nog wel.

Overigens werkt inloggen hier wel, krijg prima de 1080p streams van NPO Plus.

@chakubie
Copy link

chakubie commented Nov 6, 2023

Nog een keer helemaal opnieuw proberen in te loggen zonder resultaat: ander platform, raspberry pi met osmc..

@basrieter
Copy link
Collaborator

Inloggen is ook veranderd. Ben ik nog mee bezig. Ik ga nog verder kijken, maar voor nu even een start. Zit echt super druk met werk en zo, dus het kan even duren. TV4 heeft ook lang geduurd voordat die gefixed was.

@chakubie
Copy link

chakubie commented Nov 6, 2023

Alle begrip, dank!

@snyke7
Copy link

snyke7 commented Nov 6, 2023

In ieder geval al dank voor de patch hierboven, die werkte voor ons!

@duck7000
Copy link

duck7000 commented Nov 7, 2023

De npo live tv streams zijn ineens vervangen door live radio? De map live tv is er wel maar is gevuld met een paar radio streams.
De patch lost dat niet op

@basrieter
Copy link
Collaborator

Klopt. Work in progress.

@duck7000
Copy link

duck7000 commented Nov 8, 2023

Ah dat maakt een hoop duidelijk, thanks

@basrieter
Copy link
Collaborator

Er werkt weer meer. Graag even deze proberen: https://github.com/retrospect-addon/plugin.video.retrospect/archive/refs/heads/bug-1730.zip

Inloggen zou nu echt goed moeten werken en de listings die er nu in staan ook.

@PhyxionNL
Copy link
Author

Even e.e.a. geprobeerd, Live TV werkt weer. Inloggen ook, al werkte dat hier nog wel want ik kon oude programma's nog zien die onder de betaalmuur staan.
Ik kan alleen niet helemaal een volgorde van de volledige lijst programma's tussen verschillende pagina's. Als je dus naar de volgende pagina gaat dan heb je weer programma's die beginnen met cijfers.

Wat mij ook opviel is dat er programma's zijn bij NPO die een hogere sortering hebben dan .: Meer Pagina's :., bijvoorbeeld #BOOS.
Dat zal wel komen door de standaard sortering, ik weet niet of Kodi ook ^B (start of text) ondersteund, maar anders is ! Meer Pagina's ! misschien nog een optie 😁

@pschoond
Copy link

pschoond commented Nov 9, 2023

Krijg de laatste update hier niet geïnstalleerd op Libreelec 10.0.4 (Kodi Matrix), de vorige patch wel.
foutmelding:
ERROR : Failed to unpack archive 'zip://%2fstorage%2fbackup%2fplugin.video.retrospect-bug-1730.zip/plugin.video.retrospect-bug-1730/' to '/storage/.kodi/addons/temp/f3db0d93-1265-4ba3-8887-4504740a0f7c'

De map is er wel maar bevat niet alle bestanden:
-rw-r--r-- 1 root root 34915 Nov 9 14:58 LICENSE.md
-rw-r--r-- 1 root root 14449 Nov 9 14:58 README.md
-rw-r--r-- 1 root root 1341 Nov 9 14:58 action_install_widevine.py
-rw-r--r-- 1 root root 20048 Nov 9 14:58 addon.xml
drwxr-xr-x 9 root root 1024 Nov 9 14:58 channels

?

@BrodjagaRatnik
Copy link

Krijg de laatste update hier niet geïnstalleerd op Libreelec 10.0.4 (Kodi Matrix), de vorige patch wel. foutmelding: ERROR : Failed to unpack archive 'zip://%2fstorage%2fbackup%2fplugin.video.retrospect-bug-1730.zip/plugin.video.retrospect-bug-1730/' to '/storage/.kodi/addons/temp/f3db0d93-1265-4ba3-8887-4504740a0f7c'

De map is er wel maar bevat niet alle bestanden: -rw-r--r-- 1 root root 34915 Nov 9 14:58 LICENSE.md -rw-r--r-- 1 root root 14449 Nov 9 14:58 README.md -rw-r--r-- 1 root root 1341 Nov 9 14:58 action_install_widevine.py -rw-r--r-- 1 root root 20048 Nov 9 14:58 addon.xml drwxr-xr-x 9 root root 1024 Nov 9 14:58 channels

?

Ik doe een reboot en dan install from zip dan werkt het had ik ook met de vorige 1730

@pschoond
Copy link

pschoond commented Nov 9, 2023

Uh, je verwacht het niet (maar werkt wel). Bedankt. Bas ook bedankt !

@basrieter
Copy link
Collaborator

Even e.e.a. geprobeerd, Live TV werkt weer. Inloggen ook, al werkte dat hier nog wel want ik kon oude programma's nog zien die onder de betaalmuur staan. Ik kan alleen niet helemaal een volgorde van de volledige lijst programma's tussen verschillende pagina's. Als je dus naar de volgende pagina gaat dan heb je weer programma's die beginnen met cijfers.

Wat mij ook opviel is dat er programma's zijn bij NPO die een hogere sortering hebben dan .: Meer Pagina's :., bijvoorbeeld #BOOS. Dat zal wel komen door de standaard sortering, ik weet niet of Kodi ook ^B (start of text) ondersteund, maar anders is ! Meer Pagina's ! misschien nog een optie 😁

Waar kom je dat nog tegen? Want dat moet er echt allemaal uit en worden vervangen. Zal vanavond kijken of ik de lijst met alle programma's kan fixen.

@TonDuino
Copy link

TonDuino commented Nov 9, 2023

Die laatste bug-fix installeert OK, en inloggen werkt weer :-)

Thanx, Ton

@oliv3r
Copy link

oliv3r commented Nov 9, 2023

Even om te bevestigen, de zip zorgt ervoor dat het weer werkt :)

@diekshouse
Copy link

👍 Dank!

@PhyxionNL
Copy link
Author

Even e.e.a. geprobeerd, Live TV werkt weer. Inloggen ook, al werkte dat hier nog wel want ik kon oude programma's nog zien die onder de betaalmuur staan. Ik kan alleen niet helemaal een volgorde van de volledige lijst programma's tussen verschillende pagina's. Als je dus naar de volgende pagina gaat dan heb je weer programma's die beginnen met cijfers.
Wat mij ook opviel is dat er programma's zijn bij NPO die een hogere sortering hebben dan .: Meer Pagina's :., bijvoorbeeld #BOOS. Dat zal wel komen door de standaard sortering, ik weet niet of Kodi ook ^B (start of text) ondersteund, maar anders is ! Meer Pagina's ! misschien nog een optie 😁

Waar kom je dat nog tegen? Want dat moet er echt allemaal uit en worden vervangen. Zal vanavond kijken of ik de lijst met alle programma's kan fixen.

Wanneer je naar NPO gaat > TV programma's (Hele lijst). Je hebt dan bij de eerste pagina .: Meer pagina's :., en dan "2 voor 12" boven. Als je dan naar ".: Meer pagina's :." gaat dan staat #BOOS en #forever bovenaan, en dan daarna de ".: Meer pagina's :.". Onder de .: Meer pagina's :. heb je nu "13 In de oorlog", terwijl de vorige pagina ook al items had die begonnen met Z (Zonder Ouders Op Vakantie). Het zal wel de NPO zijn die alles zo teruggeeft, maar dan kan misschien beter alles in één keer ingeladen worden voor een logischere sortering.

@basrieter
Copy link
Collaborator

Even e.e.a. geprobeerd, Live TV werkt weer. Inloggen ook, al werkte dat hier nog wel want ik kon oude programma's nog zien die onder de betaalmuur staan. Ik kan alleen niet helemaal een volgorde van de volledige lijst programma's tussen verschillende pagina's. Als je dus naar de volgende pagina gaat dan heb je weer programma's die beginnen met cijfers.
Wat mij ook opviel is dat er programma's zijn bij NPO die een hogere sortering hebben dan .: Meer Pagina's :., bijvoorbeeld #BOOS. Dat zal wel komen door de standaard sortering, ik weet niet of Kodi ook ^B (start of text) ondersteund, maar anders is ! Meer Pagina's ! misschien nog een optie 😁

Waar kom je dat nog tegen? Want dat moet er echt allemaal uit en worden vervangen. Zal vanavond kijken of ik de lijst met alle programma's kan fixen.

Wanneer je naar NPO gaat > TV programma's (Hele lijst). Je hebt dan bij de eerste pagina .: Meer pagina's :., en dan "2 voor 12" boven. Als je dan naar ".: Meer pagina's :." gaat dan staat #BOOS en #forever bovenaan, en dan daarna de ".: Meer pagina's :.". Onder de .: Meer pagina's :. heb je nu "13 In de oorlog", terwijl de vorige pagina ook al items had die begonnen met Z (Zonder Ouders Op Vakantie). Het zal wel de NPO zijn die alles zo teruggeeft, maar dan kan misschien beter alles in één keer ingeladen worden voor een logischere sortering.

Het feit dat Kodi de sortering sloopt kan ik eigenlijk niet echt helpen (ik voeg al een \b aan het begin toe van '\b.:...., maar #` is kennelijk eerder volgens Kodi.

Wat betreft de sortering: die is op recent bij NPO start. Dus de eerste page in Kodi bevat de meest recente items, de tweede page de minder recente items. Kodi sorteert enkel de items op in de pagina en weet niet van de de volgende pagina af. Dus jouw observatie klopt.

@chakubie
Copy link

Hoop wel dat bij die release de programma's na 24.00 zichtbaar zijn. Het hield gisteren na 24.00 op, zoals ik eerder schreef: dat kan toch niet de bedoeling zijn..

@chakubie
Copy link

Desnoods komt de nieuwe datum er om 24.00 bij, dat was vannacht ook niet het geval..

@Johpin1
Copy link

Johpin1 commented Nov 18, 2023

Hoop wel dat bij die release de programma's na 24.00 zichtbaar zijn. Het hield gisteren na 24.00 op, zoals ik eerder schreef: dat kan toch niet de bedoeling zijn..

De NPO Start app heeft 00.00 tot 24.00 uur. Het is logischer. Ik weet niet of het gemakkelijk zelf is aan te passen in een van de Python scriptjes.

@chakubie
Copy link

Prima mag voor mij zo blijven hoor, maar nieuwe programma's ná 24.00 moeten wel op te roepen zijn.., kan in NPO app ook!

@basrieter
Copy link
Collaborator

Prima mag voor mij zo blijven hoor, maar nieuwe programma's ná 24.00 moeten wel op te roepen zijn.., kan in NPO app ook!

Niks MOET, alles zou kunnen. Voor nu maak ik eerst een nieuwe release. Het is echt 100% exact hetzelfde als de oude versie nu. Aanpassingen wellicht later.

@chakubie
Copy link

Natuurlijk moet niets, ik ben dankbaar voor al je moeite hoor! Maar de programma's na 24.00 waren toch op te roepen in de oude versie?

@basrieter
Copy link
Collaborator

Natuurlijk moet niets, ik ben dankbaar voor al je moeite hoor! Maar de programma's na 24.00 waren toch op te roepen in de oude versie?

Nee, dat was echt exact het zelfde als nu.

@Johpin1
Copy link

Johpin1 commented Nov 18, 2023

NPO heeft volgens mij radio verwijderd uit NPO Start: ze hebben een nieuwe app, genaamd NPO luister

In NPO Start staan de twee mappen "Live radio" en "Live TV" en als je Retrospect opstart krijg je een menu met "Radio" onder "Nationaal". Het bevat 538.

Zou het niet beter zijn om radio uit "NPO Start" te verwijderen, ook uit "Live TV" en de werkende te verplaatsen naar "Radio".

Dus veel Funx huppeldepup verwijderen en NPO Radio 1, 2, 3 FM, Klassiek naast 538 plaatsen.

@diekshouse
Copy link

Screenshot_20231118-144327

Bedien Kodi met Yatse. Voorheen had ik hier pictogrammen en tekst van de actuele programma's. Nu alleen nog een universeel pictogram. Het functioneert verder goed hoor, maar is net iets minder luxe ;-)

@basrieter
Copy link
Collaborator

Had ik inderdaad gezien. Eerst maar eens dit releasen.

@basrieter
Copy link
Collaborator

Closed with f3f8e2b

@Johpin1
Copy link

Johpin1 commented Nov 18, 2023

@diekshouse : nieuwe app van NPO Start heeft volgens mij een donkerblauwe achtergrond

@basrieter
Copy link
Collaborator

Update onderweg: xbmc/repo-plugins#4422

@bartmans
Copy link

Hoe loopt zon release via de officiële Kodi repo? Je zei dat ook andere devs erbij betrokken zijn?

@basrieter
Copy link
Collaborator

Hoe loopt zon release via de officiële Kodi repo? Je zei dat ook andere devs erbij betrokken zijn?

Klopt! Normaal loopt het zo:

  1. PR via de Kodi repo
  2. Automatische validate van basic zaken
  3. Review door een Kodi developer
  4. Mergen in GitHub
  5. Sync van GitHub naar Kodi add-on repository
  6. Sync naar alle mirrors

Stap 3 heb ik via Slack overlegd met een van de andere Kodi Python devs. En daarom heb ik hem zelf kunnen morgen.

@basrieter
Copy link
Collaborator

Volgens mij zou het zo beter moeten zijn. De uren tussen 0:00-5:00 na vandaag-middernacht komen nu nog bij "Vandaag" te staan:
https://github.com/retrospect-addon/plugin.video.retrospect/archive/refs/heads/feature-after-midnight.zip

@chakubie
Copy link

Geweldig, heel fijn, dank!!

@BrodjagaRatnik
Copy link

Hoelang duurt het tot Sync van GitHub naar Kodi add-on repository?

@basrieter
Copy link
Collaborator

Hoelang duurt het tot Sync van GitHub naar Kodi add-on repository?

Volgens mij gaat de sync elke 15 minuten. Daarna nog een sync naar de mirrors. Dat laatste kan wel een paar uur duren.

@AlbieNL
Copy link

AlbieNL commented Nov 19, 2023

Lijst met 'NOS Journaal' toont geen uitzendtijd meer na update

@basrieter
Copy link
Collaborator

Graag alle "rest-zaken" even hier melden: #1734

@retrospect-addon retrospect-addon locked as resolved and limited conversation to collaborators Nov 19, 2023
@basrieter basrieter pinned this issue Nov 19, 2023
@basrieter basrieter added this to the v5.7.0 milestone Nov 19, 2023
@basrieter
Copy link
Collaborator

@basrieter
Copy link
Collaborator

Wellicht voor alle mensen met feedback op de "recente"-items: Deze zijn kapot (API changes) en zijn hier gefixed #1750

Graag daar (In #1750) nog wat feedback.

@basrieter basrieter unpinned this issue Jan 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests