Neue Version 1.1.16 im git:
- Update "Prevents *magick from occupying the segfaults"
- Fix some typo (thx to MarkusE at vdr-portal.de)
- Add theme "Keep it Light Blue" (thx to Dr. Seltsam at vdr-portal.de)
- Better scaling of Poster/Banner in display channel
- Show poster in display replay (configureable in the setup)
- Optimize progressbar in display replay
Grüße
kamel5
Hallo,
es gibt eine neue Version 1.2.22.
- Performance optimization in the detail view
- Update skin metrixhd
- If no true color OSD is availiable the BackupSkin is activated
- Fix segfault in cViewDetailAdvancedPlugin::SetTokenContainer()
Grüße
kamel5
Update ist im tvscraper git
Danke, das passt jetzt soweit und es sollte in der Darstellung kein Unterschied zu vorher sichtbar sein.
Grüße
kamel5
ScraperGetPosterBannerV2->banner im git geändert: es wird jetzt ein Bild im "landscape" Format zurückgegeben, falls es kein Banner gibt. Analog zu cSeries.
Das passt schon mal beim Anzeigen ganz gut. Allerdings wird jetzt fanart_0.jpg als Banner und Poster zürückgegeben:
skindesigner: extensions/scrapmanager.c SetHeaderScrapInfo 101 0
skindesigner: extensions/scrapmanager.c SetHeaderScrapInfo 103 isSerie
skindesigner: extensions/scrapmanager.c SetHeaderScrapInfo 119 Poster=/etc/vdr/plugins/tvscraper/series/360443/fanart_0.jpg
skindesigner: extensions/scrapmanager.c SetHeaderScrapInfo 128 Banner=/etc/vdr/plugins/tvscraper/series/360443/fanart_0.jpg
skindesigner: extensions/scrapmanager.c SetFullScrapInfo 54 isSerie
skindesigner: extensions/scrapmanager.c SetSeries 337 Banner=/etc/vdr/plugins/tvscraper/series/360443/fanart_0.jpg
skindesigner: extensions/scrapmanager.c SetSeries 348 Fanart=/etc/vdr/plugins/tvscraper/series/360443/fanart_0.jpg
skindesigner: extensions/scrapmanager.c SetSeries 348 Fanart=/etc/vdr/plugins/tvscraper/series/360443/fanart_1.jpgUm die Kompatibilität zu GetSeries noch weiter zu erhöhen, dürfte ein Bild im Landscapeformat nur als Banner zurückgegeben werden, und nicht als Poster. Also dann so:
ScraperGetPosterBannerV2:
cTvMedia poster -> Bild im "portrait" Format, falls verfügbar. Falls nicht verfügbar: Nichts (Leerstring)
cTvMedia banner -> Bild im banner Format. Falls es das nicht gibt: Bild im "landscape" Format (das ist dann fanart). Falls das auch nicht verfügbar ist: Nichts (Leerstring)Ich glaube, das wäre dann momentan auch mein letzter Änderungswunsch.
Grüße
kamel5
OK, irgendwie ist der Wurm drin.
Jetzt ging es auf einmal wieder beim Service "SetSeries", naja.
Log:
Okt 10 18:41:06 vdr[708368]: [708368] skindesigner: extensions/scrapmanager.c SetHeaderScrapInfo 101 0
Okt 10 18:41:06 vdr[708368]: [708368] skindesigner: extensions/scrapmanager.c SetHeaderScrapInfo 103 isSerie
Okt 10 18:41:06 vdr[708368]: [708368] skindesigner: extensions/scrapmanager.c SetHeaderScrapInfo 119 Poster=/etc/vdr/plugins/tvscraper/series/360443/fanart_0.jpg
Okt 10 18:41:06 vdr[708368]: [708368] skindesigner: extensions/scrapmanager.c SetFullScrapInfo 54 isSerie
Okt 10 18:41:06 vdr[708368]: [708368] skindesigner: extensions/scrapmanager.c SetSeries 337 Banner=/etc/vdr/plugins/tvscraper/series/360443/fanart_0.jpg
Okt 10 18:41:06 vdr[708368]: [708368] skindesigner: extensions/scrapmanager.c SetSeries 348 Fanart=/etc/vdr/plugins/tvscraper/series/360443/fanart_0.jpg
Okt 10 18:41:06 vdr[708368]: [708368] skindesigner: extensions/scrapmanager.c SetSeries 348 Fanart=/etc/vdr/plugins/tvscraper/series/360443/fanart_1.jpgKann ich gerne machen. Die Skins müssten das dann halt "vernünftig" darstellen.
Oder hast Du einen besseren Vorschlag für das Verhalten?
Was hätte ich gerne?
Wie man in dem Log sieht, wird "fanart_0.jpg" bei GetSeries als Banner zurückgeliefert, bei ScraperGetPosterBannerV2 aber als Poster.
Ich habe leider nur das eine Beispiel "Brisant" und weiß auch nicht, ob es noch andere gibt.
Wünschenswert wäre wahrscheinlich, das, wenn es kein richtiges Banner gibt, dann fanart_0.jpg auch als Banner zurückgegeben wird, so wie es bei GetSeries auch gemacht wird.
Also, im Endeffekt sollte GetSeries und ScraperGetPosterBannerV2 zum gleichen Ergebnis führen.
Grüße
kamel5
Also, wenn Du 2 mal hintereinander "GetSeries" aufrufst, bekommst Du beide Male das gleiche zurück.
Nein, vielleicht habe ich das wenig deutlich dargestellt.
Einmal bekomme ich das: mit branch master wird Banner befüllt
skindesigner: extensions/scrapmanager.c SetSeries 325 Banner=/etc/vdr/plugins/tvscraper/series/360443/fanart_0.jpg
skindesigner: extensions/scrapmanager.c SetSeries 336 Fanart=/etc/vdr/plugins/tvscraper/series/360443/fanart_0.jpg
skindesigner: extensions/scrapmanager.c SetSeries 336 Fanart=/etc/vdr/plugins/tvscraper/series/360443/fanart_1.jpgdas andere mal bekomme ich das: mit branch develop wird Poster befüllt
skindesigner: extensions/scrapmanager.c SetSeries 337 Poster=/etc/vdr/plugins/tvscraper/series/360443/fanart_0.jpg
skindesigner: extensions/scrapmanager.c SetSeries 348 Fanart=/etc/vdr/plugins/tvscraper/series/360443/fanart_0.jpg
skindesigner: extensions/scrapmanager.c SetSeries 348 Fanart=/etc/vdr/plugins/tvscraper/series/360443/fanart_1.jpgEs hat sich aber an SetSeries() zwischen den beiden Branches nichts geändert und beides wird vom Service "GetSeries" abgeleitet. Und die Frage ist, warum da unterschiedliche Befüllungen entstehen können.
Grüße
kamel5
Damit gibt "GetPosterBannerV2" nun den korrekten type zurück.
Das funktioniert soweit.
Allerdings gibt es trotzdem noch Unterschiede:
Beides mit tvscraper letzter Stand.
Wenn ich mir mit Branch master Debug-Ausgaben anzeigen lasse, bekomme ich bei ARD Brisant, da gibt es keinen richtigen Banner, trotzdem fanart_0.jpg als Banner zurück. Hier werden SetHeaderScrapInfo() und SetSeries() von GetSeries befüllt.
skindesigner: extensions/scrapmanager.c SetHeaderScrapInfo 91 isSerie
skindesigner: extensions/scrapmanager.c SetHeaderScrapInfo 108 Banner=/etc/vdr/plugins/tvscraper/series/360443/fanart_0.jpg
skindesigner: extensions/scrapmanager.c SetSeries 325 Banner=/etc/vdr/plugins/tvscraper/series/360443/fanart_0.jpg
skindesigner: extensions/scrapmanager.c SetSeries 336 Fanart=/etc/vdr/plugins/tvscraper/series/360443/fanart_0.jpg
skindesigner: extensions/scrapmanager.c SetSeries 336 Fanart=/etc/vdr/plugins/tvscraper/series/360443/fanart_1.jpgWenn ich das mit Branch develop mache, also mit dem geänderten SetHeaderScrapInfo, dann bekomme ich fanart_0.jpg als Poster zurück. Hier wird SetSeries() von GetSeries und SetHeaderScrapInfo() von GetPosterBannerV2 befüllt.
skindesigner: extensions/scrapmanager.c SetHeaderScrapInfo 103 isSerie
skindesigner: extensions/scrapmanager.c SetHeaderScrapInfo 119 Poster=/etc/vdr/plugins/tvscraper/series/360443/fanart_0.jpg
skindesigner: extensions/scrapmanager.c SetSeries 337 Poster=/etc/vdr/plugins/tvscraper/series/3604skindesigner: extensions/scrapmanager.c SetSeries 348 Fanart=/etc/vdr/plugins/tvscraper/series/360443/fanart_0.jpg
skindesigner: extensions/scrapmanager.c SetSeries 348 Fanart=/etc/vdr/plugins/tvscraper/series/360443/fanart_1.jpgSchon seltsam, da ich an der Funktion SetSeries() nichts geändert habe. Ich hätte jetzt erwartet, das zumindest in SetSeries() beide Male das gleiche Ergebnis erzeugt wird.
Aus Sicht vom skindesigner wäre es wünschenswert, wenn zumindest im Falle von GetPosterBannerV2 Banner befüllt wird, dann könnte man die skins so lassen, und müsste keine Änderungen machen.
Vielleicht hasst Du da noch eine Idee.
PS: Ich habe den Branch develop noch mal upgedatet.
Grüße
kamel5
Ich habe das Anzeigen des Posters/Banners in skinnopacity im Branch devel mal etwas angepasst.
Kannst Du mal prüfen, ob damit der "Workaround" zum type in tvscraper obsolet werden könnte.
Grüße
kamel5
Ich weiß, das ist unsauber. Grund für den Rückgabewert ist nOpacity:
nOpacity zeigt beim Drücken von OK links oben ein Bild.
OK, verstehe.
Da muss ich mir das Handling da wohl mal ansehen.
Grüße
kamel5
Alles anzeigen"GetSeries" hat verschiedene Felder für die verschiedenen Bilder.
"GetPosterBannerV2" versucht immer ein Bild zurückzugeben, notfalls im falschen Format (besser als gar kein Bild):
"ScraperGetPosterBannerV2.poster":Bild im "portrait" Format, falls verfügbar. Falls nicht verfügbar: Bild im "landscape" Format (das ist dann fanart).
"ScraperGetPosterBannerV2.banner":Bild im "banner" Format. Falls das verfügbar ist, wird type = tSeries; zurückgegeben. Falls das nicht verfügbar ist, wird type = tMovie; zurückgegeben.
OK, irgendwie muss man ja was wählen.
Die Frage ist halt, ob beide Anfragen zu einer Sendung den gleichen type zurückgeben sollten.
Bei ARD-Brisant ist es z.B. so, das bei Nutzung vom Service "GetEventType" der type tSeries, und bei Nutzung vom Service"ScraperGetPosterBannerV2" der type tMovie zurückgegen wird.
Das hat bei der Umstellung zur Verwirrung geführt. 
Grüße
kamel5
Könntest Du noch eine Performance-Optimierung machen?
Ich habe mir das jetzt mal angesehen. Durch den Programmablauf ist es hier nicht möglich, die beiden Aufrufe von "GetMovie" ohne größere Umbauten zusammenzufassen.
Allerdings ist es so, das der zweite Aufruf nur für die Bilder in der Kopfzeile der Detailansicht benötigt wird. Mit ein wenig Umbau konnte ich das über den Service "GetPosterBannerV2" abbilden, so das dadurch schon deutlich weniger Datenmenge anfallen dürfte.
Dabei habe ich allerdings festgestellt, das hier die beiden Service "GetSeries" und "GetPosterBannerV2" unterschiedliche Ergebnisse liefern können. Dazu gibt es ein reproduzierbares Beispiel, mehr habe ich bei meinen Tests erst einmal nicht feststellen können, das bedeutet aber nicht, das es sie nicht gibt.
Beispiel ARD - Brisant:
- wird als Serie erkannt
- bei "GetSeries" wird ein Bild als Banner zurückgegeben (fanart_0.jpg)
- bei "GetPosterBannerV2" wird das gleiche Bild als Poster zurückgegeben (fanart_0.jpg)
- bei "GetPosterBannerV2" gibt es kein Banner
Das bedeutet, das sich mit Nutzung von "GetPosterBannerV2" möglicherweise andere Darstellungen ergeben.
Beim Skin "metrixhd" und auch anderen kann man das feststellen.
Den Skin "metrixhd" habe ich in einem zweiten commit angepasst. Die anderen betroffenen Skins kann ich leider nicht ändern.
Schön wäre es natürlich, wenn beide Services gleiche Ergebnisse liefern würden.
MarkusE, kannst Du Dir das nochmal ansehen.
Um abwärts kompatibel zu bleiben, müsste "GetPosterBannerV2" in so einem Falle, solche Poster auch als Banner zurückgeben???
Commits sind im Branch develop.
Bitte diese Änderung ausführlich testen, Bugs sind nicht ausgeschlossen.
Grüße
kamel5
Den dafür notwendigen Aufwand kann ich aber nicht beurteilen.
Das ist kein großer Aufwand.
Ich nehme das Theme gern in die nächste Version mit auf.
Grüße
kamel5
Da fällt mir garade noch ein, im git unter Wiki gibt es auch noch ein paar hilfreiche Informationen dazu.
Grüße
kamel5
jedoch habe ich bei manchem Sendungen hinter dem Text des Programmtitels und der Zeiten ein Senderlogo
Ein Senderlogo sollte links vom Sendernamen stehen und nur dort. Ob es ein Senderlogo gibt, hängt davon ab, ob es im Setup aktiviert ist und ob im entsprechenden Verzeichnis von tvguide eins hinterlegt ist. Die muss man sich selbst besorgen, sie werden nicht mitgeliefert.
manche Sendungen sind mit einem roten Farbverlauf hinterlegt, manche haben einen schwarzen Hintergrund. Hat das einen Grund? Manche Kanäle haben gemischt einen schwarzen/rot-schattierten Hintergrund und manche Kanäle nur schwarz oder nur rot-schattiert
Das hängt vom ausgewählten Theme ab und kann auch in den entsprechenden Dateien selbst angepasst werden.
Ich sehe gerade, ich sollte wohl das Default-Theme mal überarbeiten. Du kannst "Dark Red" mal probieren, da sollte alles passend aussehen.
in manchen TV-EPGs oder auch in SetTop-Boxen kann man in der EPG Übersicht mit Links und rechts eine Sendung vor- und zurückspringen (anstelle zeitlichem Springen wie im tvguide-Plugin).
Bei mir springt es zur vorhergehenden/nächsten Sendung, wenn ich die Links/Rechts Taste betätige.
Die ganzen Dinge, wie sich das Plugin verhält, können im Plugin-Setup konfiguriert werden. Einfach da mal schauen und verschiedene Einstellungen testen.
Grüße
kamel5
Ich kann mir das gerne mal ansehen. Da ich aber erst einmal sehen muss, wie da die Zusammenhänge sind, schaffe ich das jetzt vor meinem Urlaub nicht mehr. Erst danach, ab Oktober, habe ich wieder mehr Zeit.
Grüße
kamel5
Noch ein kleiner Schönheitsfehler:
Danke, das habe ich mit noch ein paar ähnlichen Typos geändert und ist jetzt im git.
Grüße
kamel5
Hallo,
es gibt ein neues Bugfix-Release 1.2.21.
- [pbiering] Fix off-by-one strcpy buffer overflow
- Don't show emty actor pictures in detailview
Grüße
kamel5
Beides ist legitim und kann benutzt werden. CreatePixmap() hat hier gegenüber "osd->CreatePixmap" den Vorteil, das da noch Prüfungen eingebaut sind.
Und ja, Du kannst das im Prinzip ersetzen. Also ausprobieren.
Grüße
kamel5
Ich vermute mal, du möchtest auch noch scraper2vdr unterstützen (?).
Ja, das soll schon noch funktionieren.
Manchmal hilft eine 2. Sicht um die Konsequenzen zu verstehen.
Ich habe erst einmal skindesigner angepasst, ich muss da aber leider im Moment 2 mal die ganzen Bilder abfragen,
deshalb die Idee. Vieleicht sollte ich da später noch mal eine Überarbeitung machen.
Grüße
kamel5
MarkusE , ich habe da mal eine Frage:
Bei meiner aktuellen Arbeit am skindesigner ist mir aufgefallen, das es da auch noch in der Kategorie Schauspieler, Namen ohne Bilder gibt. Ich könnte das zwar jetzt auch wieder im skindesigner, wie bei anderen Skins auch, realisieren, andererseits wäre es nicht schöner, sowas zentral zu machen.
Also, ließe sich das mit vertretbarem Aufwand als Option in tvscraper nachrüsten, z.B. in dem in cMovie und dort wo noch Actors zurückgegeben werden, eine zusätzliche Option, z.B. "actors_all", eingeführt wird, die den alten Zustand default mäßig bereit stellt, und zusätzlich Actors nur dann, wenn sie auch ein Bild haben. Dann ließe sich das in den Skins nochmals vereinfachen.
Grüße
kamel5
