UserRating Error

Coolzero82

Hallo, ich bekomme unter LE 10 immer eine Fehlermeldung zum UserRating Addon, im Log steht dazu

Code

06:34:56.023 T:1471     INFO <general>: CPythonInvoker(4, /storage/.kodi/addons/plugin.video.dazn/startup.py): script successfully run
2021-11-06 06:34:56.023 T:1477     INFO <general>: CPythonInvoker(10, /storage/.kodi/addons/screensaver.atv4/service.py): script successfully run
2021-11-06 06:34:56.112 T:1471     INFO <general>: Python interpreter stopped
2021-11-06 06:34:56.748 T:1546     INFO <general>: Skipped 1 duplicate messages..
2021-11-06 06:34:56.748 T:1546    ERROR <general>: UpdateFromScraper: No EPG scraper defined for table ''
2021-11-06 06:34:56.748 T:1546    ERROR <general>: Update: Failed to update table ''
2021-11-06 06:34:56.748 T:1546    ERROR <general>: UpdateFromScraper: No EPG scraper defined for table ''
2021-11-06 06:34:56.748 T:1546    ERROR <general>: Update: Failed to update table ''
2021-11-06 06:34:56.805 T:1488    ERROR <general>: EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--
                                                    - NOTE: IGNORING THIS CAN LEAD TO MEMORY LEAKS!
                                                   Error Type: <class 'AttributeError'>
                                                   Error Contents: module 'xbmc' has no attribute 'abortRequested'
                                                   Traceback (most recent call last):
                                                     File "/storage/.kodi/addons/script.user.rating/service.py", line 44, in <module>
                                                       while(not xbmc.abortRequested):
                                                   AttributeError: module 'xbmc' has no attribute 'abortRequested'
                                                   -->End of Python script error report<--
                                                   
2021-11-06 06:34:57.005 T:1488     INFO <general>: Python interpreter stopped

Alles anzeigen

Hat einer eine idee dazu wie ich das lösen kann?

Danke

DaVu

Wie kann man den Fehler reproduzieren?

PvD

Zitat von Coolzero82

Hat einer eine idee dazu wie ich das lösen kann?

Du nicht, das muss der Maintainer des Addons machen. abortRequested ist in den Monitor gewandert, dort muss es jetzt xbmc.Monitor().abortRequested heissen. Richtiger (tm) ist es allerdings, zuerst am Anfang des Programms eine Monitor-Instanz zu erzeugen (z.B. mon = xbmc.Monitor()) und diese zu nutzen: while not mon.abortRequested()

DaVu

Laut unserem Wiki ist das die Quelle:

https://github.com/Regss/script.user.rating

Da ist schon verdammt lange nichts mehr passiert

Edit:

Jo....gerade nochmal gecheckt. Das Script ist in Kodi Jarvis ins Repo gewandert und seither nicht mehr aktualisiert worden.

Ich mach mal nen issue bei dem Kollegen auf Vielleicht meldet er sich ja. Ansonsten darf man das Add-on gern als "unmaintained" deklarieren und jeder, der sich berufen fühlt kann ein Update ins Kodi Repo pushen

edit edit:

https://github.com/Regss/script.user.rating/issues/2

Issue erstellt

PvD

Ich hab' mal auf das Issue geantwortet . Die Lösung ist ja relativ einfach, allerdings funktioniert das nur bis Leia. Für Matrix+ stehen dann größere Änderungen an. Sofern der Maintainer (oder jemand anderes) nicht reagiert, wird das Addon wohl sterben...

DaVu

@PvD
Warum nicht forken und ins Kodinerds-Repo?

PvD

Ich betreue schon 40+ Addons. Eigentlich wollte ich da mal kürzer treten. Dazu kommt, dass mich das User Rating Addon nicht unbedingt triggert. Für mich ist das weder von Interesse noch meine Baustelle. Sorry.

DaVu

Alles gut. Gibt nichts zu entschuldigen. Ist ja alles auf freiwilliger Basis. Kann das gut verstehen.

Coolzero82

Danke für eure Rückmeldungen und das ihr es euch angesehen habt, wäre schön wenn sich einer annehmen würde, leider übersteigt das deutlich meine kompetenz.

DaVu

Danke erstmal für den Like. Der diente in dem Fall für mich als Reminder für den Thread.

Ich schaue mir das vielleicht Mal morgen an. Je nachdem obs mein Männerschnupfen zulässt oder ob's mich aus den Schuhen haut

DaVu

@Coolzero82

Kannst du mal bitte berichten, ob das add-on jetzt so funktioniert?

Ich habe das jetzt mal geforkt:

https://github.com/DaVukovic/script.user.rating

Vorgenommene Änderungen waren:

addon.xml:
- Author
- Beschreibungen
default.py
- translatePath ist in das Modul xbmcvfs gewandert. Daher musste das auch umgestellt werden
- die von @PvD vorgeschlagene Änderung (danke dafür )

Bei mir hat es nach der Installation keinen Fehler mehr geworfen.

Ich bin aber ehrlich und verstehe noch nicht so ganz, wie es sich benutzen lässt

PvD

Zitat von DaVu

Ich bin aber ehrlich und verstehe noch nicht so ganz, wie es sich benutzen lässt

So wie ich es beim Überfliegen des Codes verstanden habe, kann man für den laufenden Film/Serie eine Bewertung bei verschiedenen Portalen (TVDb, TMDb, Filmweb) abgeben. Dazu benötigt man aber einen Account und legt die dazugehörenden Daten im Setup des Addons ab. Für TMDB ist ein API-Key hinterlegt.

Letztendlich ist das ein Community-Addon und macht dann für die TMDb/TVDb Community auch Sinn.

Was mir noch am Code aufgefallen ist, dass der Service mit ziemlich hoher Frequenz rotiert (0,1 sec), nur damit er läuft: https://github.com/DaVukovic/scri…/service.py#L45 . Dieser Wert kann gefahrlos auf einen viel höheren Wert gesetzt werden (10 sec oder mehr). Der Monitor benutzt Event-Handler, die beim Auftreten eines Ereignisses, hier:

Player stoppt: https://github.com/DaVukovic/scri…/service.py#L28
Videolibrary wird aktualisiert: https://github.com/DaVukovic/scri…/service.py#L33

quasi sofort reagieren. Ein Erhöhen des o.g. Wertes für den Service kann also den Ressourcenbedarf verringern. Das wird dann im Hinblick auf schwächere Systeme interessant.

Aber ich freue mich natürlich, dass Du als Python Novize das Problem alleine angegangen bist und auch eine funktionierende Lösung gefunden hast.

DaVu

Schauen wir mal, ob es funktioniert. Dann kann man noch ein wenig mehr dran feilen

Zitat von PvD

Aber ich freue mich natürlich, dass Du als Python Novize das Problem alleine angegangen bist und auch eine funktionierende Lösung gefunden hast

Naja...50% kamen von dir. Also Danke

DaVu

@Coolzero82

Bekomme ich bitte noch ein Feedback ob es so läuft wie du es dir vorstellst?

Da wäre ich dir echt dankbar für

Coolzero82

@DaVu
Sorry für die späte Rückmeldung bin leider die ganze Zeit nicht dazu gekommen es zu testen. Aber auf den ersten Blick sieht alles gut aus, es gibt keine Fehlermeldungen mehr und es funktioniert anscheint alles.

Vielen Dank dafür und noch ein frohes neues Jahr.

DaVu

@Coolzero82

Vielen Dank für's Feedback. Dann werde ich das Mal ins Kodi Repo submitten und dann hat jeder was davon

DaVu

@Coolzero82

Ich würde das Add-on gern ins Kodi repo pushen. Dafür würde ich natürlich auch gern selbst die Funktionalität und Matrix testen wollen. So wie ich das Add-on verstehe, kann ich zwar auch mein Rating an Metadata-Seiten wie TVDB oder TMDB übermitteln, ich kann aber auch lokales rating ausführen. Ist das richtig? Oder muss ich tatsächlich einen Account bei TMDB oder TVDB haben um das testen zu können?

@PvD

Wenn du Zeit und Lust hast, kannst du dir bitte mal das Add-on installieren und es dir selbst mal anschauen? In den Einstellungen kann ich optional die Login-Daten für die Metadata-Seiten angeben. Ich habe aber auch eine Einstellung, die besagt "Öffne automatisch den Rating-Dialog wenn ein Film als gesehen markiert wurde" und das passiert bei mir lokal einfach nicht.

Link zum Matrix Branch des Add-ons
https://github.com/DaVukovic/script.user.rating/tree/matrix

PvD

Zitat von DaVu

Wenn du Zeit und Lust hast, kannst du dir bitte mal das Add-on installieren und es dir selbst mal anschauen? In den Einstellungen kann ich optional die Login-Daten für die Metadata-Seiten angeben. Ich habe aber auch eine Einstellung, die besagt "Öffne automatisch den Rating-Dialog wenn ein Film als gesehen markiert wurde" und das passiert bei mir lokal einfach nicht.

Der Service läuft nicht (Syntaxfehler). Ich werde mal Dein Addon klonen und einen PR machen. Ich habe mich im Addon mal mit meinen TMDB-Daten angemeldet.

EDIT 1: Auch wenn der Service jetzt läuft, wird das Skript zum Synchen der Daten falsch aufgerufen. XBMC.RunScript(...) gibt es nicht mehr, das heisst schon seit Leia nur noch RunScript(...). Eine Bewertung wird also nie abgegeben.

DaVu

Zitat von PvD

EDIT 1: Auch wenn der Service jetzt läuft, wird das Skript zum Synchen der Daten falsch aufgerufen. XBMC.RunScript(...) gibt es nicht mehr, das heisst schon seit Leia nur noch RunScript(...). Eine Bewertung wird also nie abgegeben.

Magst du mir noch verraten wo der Syntax-Fehler war? Das mit "RunScript" ist ja schnell geändert

PvD

while(not monitor.abortRequested): --> while not monitor.abortRequested():Das ist eine Methode - zu erkennen hinten an den (). So wie Du es verwendest, ist es eine Eigenschaft. Die ist immer True. Durch die Negation (not) wird der Audruck False und while(False) wird nicht abgearbeitet.Es gibt aber noch mehr Baustellen: Der Service triggert nicht die Notification des Systems, daher wird das Bewertungsscript nicht aufgerufen, ob nun mit oder ohne XBMC.

EDIT: Wozu der [definition=12,4][definition='1','3']Debug[/definition][/definition] Mode drin ist, ist mir ein Rätsel. Der 'debuggt' gerade mal an zwei Stellen. Das kann man anders lösen.

Jetzt mitmachen!