[Announce] vdr-plugin-tvscraper 1.2

Ich bin gerade dabei, den Service "GetPoster" in einem meiner Skins zu benutzen.

Zuerst dazu die Frage, ob Du auch vorhast den abzuschaffen.

Und wenn nicht:

Mir ist aufgefallen das dieser Service alle möglichen Formate zurückliefert, also auch Banner und Landscape-Bilder. Im Prinzip sollte dieser Service nur Poster, und wenn kein Poster vorhanden, ein Bild im Porträt Format zurückliefern. Wenn beides nicht vorhanden ist, wäre vielleicht ein Bild im Landscape Format auch noch vertretbar???

Aber auf jeden Fall kein Banner.

Beispiel 1: ZDF Die Rosenheim Cops, da wird manchmal ein Banner zurückgeliefert und manchmal ein Poster

Beispiel 2: ARD Brisant, hier wird ein Landscape Format zurückgeliefert. Hier gibt es kein anderes.

Grüße

kamel5

Ist es eigentlich beabsichtigt, dass bei Aufnahmen eine tvscrapper.json erstellt wird?

Oder ist das einfach ein bekannter, historischer Typo-Bug, den man aus Rückwärtskompatibilitätsgründen besser so lässt?

> Das ist ein Bug . Aber aus Kompatibilitätsgründen schwer zu korrigieren.

Erlaubt die VDR-Plugin-API kein Umbenennen von bestehenden tvscrapper.json in Aufnahmeordnern?

Nur ne Idee - Aufwand/Nutzen sicherlich in keinem Verhältnis.

Mal noch was anderes:

Bei Neuaufnahmen landen fanart.jpg, poster.jpg und tvscrapper.json im Aufnahmeordner.

Wie ist denn das beim Scrapen von Bestandsaufnahmen? Wird das dort ebenso gehandelt oder wird dafür /var/cache/vdr/plugins/tvscraper/recordings genutzt?

Hintergrund der Frage ist, dass ich ungern die mtime-Werte der alten Aufnahmen-Ordner verändern möchte.

Wenn dort neue Dateien drin landen, wird die mtime aktuell - deswegen bin ich bislang vom Scrapen der Altaufnahmen noch zurückgewichen.

Quote from MarkusE

Du kannst das Bild ignorieren, wenn das Format für Dich nicht verwertbar ist.

Schon klar, aber:

Wenn ich explizit "GetPoster" aufrufe, dann will ich ja kein Banner...

Ich würde hier ein Banner nicht zurückliefern.

Grüße

kamel5

Was mir eben noch aufgefallen ist:

im Zuge eines Schnitts werden fanart.jpg, poster.jpg und tvscrapper.json nicht in das neue %-Verzeichnis mit übernommen.

Ist das ein Bug oder mit der aktuellen VDR-API schlicht nicht so machbar?

Das lässt sich leicht der VDR recordingaction erledigen. Einfach bei edited einbauen.

Edit: Könnte so aussehen:

  edited)  
    # Copy files from TVScraper to edited recording
    for file in fanart.jpg poster.jpg tvscrapper.json ; do          
      [[ -e "${3}/${file}" ]] && cp "${3}/${file}" "${2}"
    done
  ;;

Habe das bei mir im Recordinghook auch eingebaut:

    # Copy files from TVScraper and epg2vdr to edited recording
    files=(fanart.jpg poster.jpg tvscrapper.json)  # TVScraper
    files+=(info.epg2vdr)                          # epg2vdr
    for file in "${files[@]}" ; do
      [[ -e "${3}/${file}" ]] && cp --archive --update "${3}/${file}" "${2}"
    done

Welche Dateien oder Plugins müsste man noch berücksichtigen?

Quote from MegaV0lt

Welche Dateien oder Plugins müsste man noch berücksichtigen?

Sehr gut, vielen Dank. Aber sollte man sowas nicht direkt mit den jeweiligen Plugins mitliefern?

Oder eine Art "sync_non-core_files_after_edit" Hook zentral VDR-seitig für solche Fälle bereitstellen?

Quote from rüsseltier

Aber sollte man sowas nicht direkt mit den jeweiligen Plugins mitliefern?

Wenn das Plugin mitbekommt, dass geschnitten wurde ist das sicher kein Problem... Ist nur die Frage nach dem Aufwand...

Bei TV Guide NG erhalte ich jetzt immer dieses Meldung:

tvscraper: ERROR calling vdr service interface, call->event && call->recording are provided. Please set one of these parameters to NULL

TV Guide NG ist auch sehr träge geworden.

Muss TV Guide NG noch angepasst werden?

Quote from Murry

TV Guide NG ist auch sehr träge geworden.

Das konnte ich so nicht feststellen...

Quote from Murry

Muss TV Guide NG noch angepasst werden?

Ich habe dazu mal ein Bugfix Release gemacht.

Grüße

kamel5

kamel5

Sag mal wäre es möglich die Anzahl der dargestellten Sender drastisch zu erhöhen (von maximal 12 auf 18-20)?

Taipan,

das ist kein Problem. Ich kann das gerne in eine nächste Version mit rein nehmen.

Grüße

kamel5

kamel5

Danke, ich habe es eben mal in der setup.c auf 20 für mich geändert und übersetzt - passt!