Plugin satip wer pflegt das ?

wirbel,

um zu Prüfen, ob ein Device, das bereits einen Kanal empfängt, gleichzeitig noch einen anderen Kanal empfangen kann wird

  virtual bool IsTunedToTransponder(const cChannel *Channel) const;                                                                                                          
  virtual bool MaySwitchTransponder(const cChannel *Channel) const;                                                                                                          

von dem Device implementiert. Ich würde jetzt nicht unbedingt sagen, dass damit PLUGINS.html falsch ist. Es sind da halt nicht alle Methoden gelistet, die ein Device implementieren sollte. Die komplette Liste steht in device.h, es sind die virtuellen Methoden von class cDevice.

Wie schon vor einer Woche geschrieben bin ich im Urlaub: kein VDR, kein Code.

Wenn ein (plugin-) device eine Funktion implementieren soll, dann muss das genau so in Plugins.html erklärt sein. Nicht im Source. VDR ist da ein echtes Vorbild, erstklassige Doku, schon solange ich das Projekt kenne. Wenn Doku unklar ist, erst die Doku fixen, und dann externen Source Code der Doku anpassen.

Das Femon-Plugin zeigt doch nur die Empfangsstärke des Transponders an, nicht aber die darin eingebetteten Kanäle. Oder hab ich das nur noch nicht entdeckt?

..echt?

Wenn ich es nicht weiß, du aber schon, wäre es nur fair, mir zu sagen, wie ich mit Femon die Kanäle eines Transponders auflisten kann.

Mit DevStatus geht das halt ganz bequem. Ich bin erstaunt, dass ihr damit nicht zurecht kommt…

Ich bin gerade nicht am VDR, aber ich glaube links/rechts listet alle PIDs, die empfangen werden, alle streams mit Erklärungen, Bitraten und Typ von Audio und Video.

Irgendwie hätte ich jetzt erwartet dass die Empfangsqualität bei allen Kanälen die zum gleichen Transponder gehören gleich ist.

Oder habe ich da etwas übersehen?

Quote

ChanUp/ChanDn - Switch channel up/down
Up/Down - Switch channel up/down
0-9 - Select channel
Ok - Switch between display modes: basic, transponder, stream, AC-3
Green - Select next audio track
Yellow - Select audio channel: stereo, mono left, mono right
Back - Exit plugin
Left/Right - Switch to next/previous device that provides the current channel

Display More

Links rechts schaltet den gerade aktiven Kanal durch die Devices. Mit OK iteriert man durch die Anzeigemodi: Transponder-Daten, Videostream-Daten und Audiostream-Daten.

Von mehr weiß ich auch nicht, weshalb ich über deine Anfrage etwas erstaunt war.

Aber wenn du gerade im Urlaub bist, musst du dir darüber jetzt wirklich keine Gedanken machen. Das kann sicherlich noch ein paar Tage warten… gute Erholung!

Darf ich darauf hoffen, dass du dir den Sachverhalt nochmals anschaust? Es geht darum, dass dem VDR bei SatIP-Devices nicht alle Kanäle eines Transponders bekannt sind (zu sehen bspw. in DevStatus) und somit auch nicht alle Transponder-Kanäle für parallele Aufnahmen zur Verfügung stehen.

Das wäre echt super!

Ist nicht vergessen, nur noch keine Zeit.

Alles klar, danke.

Friendly reminder…

Fair enough.

Immer noch viel zu busy, aber das hier schmort schon viel zu lange..
Ich versuche das mal, das in eine machbare To-Do Liste umzuarbeiten.

1. du sagst, dass ein bereits getunter Transponder nicht wieder verwendet wird - korrekt? Bitte korrigieren, falls nicht 100%
-> das sollte sich testen lassen.
Mir fällt sowas selten auf, weil ich acht SAT>IP tuner auf dem server habe.
Bedeutet das > acht Aufnahmen für mich zum Gegentest, oder tritt der Effekt mit weniger auf?

2. laut der letzten von mir gelesenen Plugins.html (nicht unbedingt der aktuellen) muss ein device nur implementieren:

virtual int NumProvidedSystems(void) const;
virtual bool ProvidesSource(int Source) const;
virtual bool ProvidesTransponder(const cChannel *Channel) const;
virtual bool ProvidesChannel(const cChannel *Channel, int Priority = -1, bool *NeedsDetachReceivers = NULL) const;
virtual bool SetChannelDevice(const cChannel *Channel, bool LiveView);

3. soweit ich gerade nachlese, fehlt nach eurer Meinung z.Z.
a) MarkusE #72: cDevice::IsTunedToTransponder(Channel);
-> prüfen, was aktuell implementiert ist.
Zusätzlich, wäre gut zu verstehen, warum die generelle Lösung nicht greift. Dafür ist die doch da.

b) MarkusE #81:
virtual bool IsTunedToTransponder(const cChannel *Channel) const; << Wiederholung von a)
virtual bool MaySwitchTransponder(const cChannel *Channel) const;

Weiter aus #81 + #82:
" Es sind da halt nicht alle Methoden gelistet, die ein Device implementieren sollte."
-> Wenn das so sein sollte, dann sollte Plugins.html aktualisiert werden.
Gute Doku ist besser als in headern zu wühlen. Und so, dass kls ein möglichst easy Patch angeboten wird, mit Begründung. Machbar.

Korrekt, @ MarkusE + SHofmann ?

Quote from wirbel

Bedeutet das > acht Aufnahmen für mich zum Gegentest, oder tritt der Effekt mit weniger auf?

Glücklicherweise geht es relativ einfach zu testen: Eine Aufnahme auf einem Transponder starten, danach noch eine weitere auf dem gleichen Transponder (also bspw. ZDF und ZDFneo). Jede weitere Aufnahme auf dem gleichen Transponder sollte auch das gleiche Device nutzen. Das kann mit dem Gerätestatus-Plugin recht einfach geprüft werden (siehe meine Screenshots weiter oben, etwa #79).

Zu den anderen Fragestellungen kann ich leider nicht viel beitragen. Würde ich mich da auskennen, hätte ich ja gleich einen Patch angeboten. Aber schau dir zum Vergleich doch mal den Fork von Firefly (bei mir aktuell der Commit 275afb6) an, denn damit funktioniert das alles so, wie es soll.

Danke jedenfalls, dass du das jetzt angehen willst.

Wenn ein Plugin keine Transponder kennt (z.B. iptv) oder das gleichzeitige Aufnehmen von mehrere Sendungen eines Transponders nicht unterstützen möchte, muss IsTunedToTransponder und MaySwitchTransponder natürlich nicht implementiert werden. Dann funktioniert das halt auch nicht ...

kls , was meinst Du? Sollte die Dokumentation in Plugins.html verbessert werden? Sollte genauer beschrieben werden, welche Interface Methoden ein solches Plugin implementieren muss, um welche Features bereitzustellen?

Die Doku sollte alle Infos haben, damit ein device in VDR zu 100% funktioniert. Vielleicht wäre das ein guter Nenner.

Quote from MarkusE

kls , was meinst Du? Sollte die Dokumentation in Plugins.html verbessert werden? Sollte genauer beschrieben werden, welche Interface Methoden ein solches Plugin implementieren muss, um welche Features bereitzustellen?

Ich bin offen für Patches.

wirbel , magst Du einen Patch bauen?

Den Sachverhalt habe ich ja in #95 beschrieben.

Hallo kls, hallo MarkusE,

diff für PLUGINS.html im Anhang, schaut bitte mal drüber.

Hallo SHofmann,

dein Problem lässt sich gut nachvollziehen.
Das Problem liegt bei bool cSatipDevice::IsTunedToTransponder() und dem Update von cSatipDevice::currentTransponder per RTCP.

Bei IsTunedToTransponder traten drei Probleme auf
* wenn die Frequenz um auch nur ein MHz falsch ist, gibt die Funktion false zurück, obwohl es der selbe TP ist.
Abweichungen von bis 2MHz kommen aber bei Sat ab und zu sogar aus den SI Daten.
* die Kanal Parameter wurden per strcmp() verglichen, ein 'M64C0' ist damit ein anderer TP als 'C0M64' oder 'M64'.
* die Parameter werden zu genau verglichen, kleine normale Abweichungen stören schon.

Zumindest IsTunedToTransponder sollte jetzt besser funktionieren.

Probiere mal bitte die aktuelle Version im git, ob die dein Problem löst. Im Anhang mal live TV mit Aufnahme vom gleichen TP aus der Sicht des satip Servers.