rass und radiotext

Das Original Benutzerinterface Neutrino-SD incl. zapit, sectionsd, yWeb etc...
liab
Einsteiger
Einsteiger
Beiträge: 111
Registriert: Samstag 9. Februar 2008, 15:07

Re: rass und radiotext

Beitrag von liab »

rhabarber1848 hat geschrieben: Bitte testen: radiotext_record.diff

To-Do
Während der Aufnahme ist kein RT-Icon in der Infobar, es sollte stattdessen dunkelgrau sein.
Der Kurztest war erfolgreich - Aufnahme geht.

Hier ein Vorschlag für das fehlende Icon bei Aufnahme. In der infoviewer.cpp gibt es eine Stellle für den roten Aufnahmepunkt. Da ließe sich das Radiotext Icon recht einfach einbinden, wahrscheinlich sogar blinkend.
dwilx

Re: rass und radiotext

Beitrag von dwilx »

rhabarber1848 hat geschrieben: To-Do
Während der Aufnahme ist kein RT-Icon in der Infobar, es sollte stattdessen dunkelgrau sein.
Aus dem Bauch heraus fällt mir das dazu ein:

Code: Alles auswählen

void CInfoViewer::showIcon_RadioText(bool rt_available) const
// painting the icon for radiotext mode
{
	if (showButtonBar)
	{
		int mode = g_Zapit->getMode();
		std::string rt_icon = "radiotextoff.raw";
		if ((!virtual_zap_mode) && (mode == 2))
		{
			if (g_settings.radiotext_enable){
					rt_icon = rt_available ? "radiotextget.raw" : "radiotextwait.raw";
				}
		}
		else if ((g_settings.radiotext_enable) && (mode == 7))
			rt_icon = "radiotextoff.raw";
		
		
		frameBuffer->paintIcon(rt_icon, BoxEndX - (ICON_LARGE_WIDTH + 2 + ICON_LARGE_WIDTH + 2 + ICON_SMALL_WIDTH + 2 + ICON_SMALL_WIDTH + 6),BoxEndY + (InfoHeightY_Info - ICON_HEIGHT) / 2);
	}
}
liab
Einsteiger
Einsteiger
Beiträge: 111
Registriert: Samstag 9. Februar 2008, 15:07

Re: rass und radiotext

Beitrag von liab »

dixidix hat geschrieben:
rhabarber1848 hat geschrieben: To-Do

Aus dem Bauch heraus fällt mir das dazu ein:
Ich habe es anders gemacht, geht einwandfrei. Macht das Icon nur bei Aufnahme und Radio-Modus.

Code: Alles auswählen

#ifdef ENABLE_RADIOTEXT
	if ((g_settings.radiotext_enable) && (CNeutrinoApp::getInstance()->getMode() == 2)) {
		showRadiotext();
	}
	else if ((!g_settings.radiotext_enable) && (CNeutrinoApp::getInstance()->getMode() == 2))
	{
		showIcon_RadioText(false);
	}
	if (((CNeutrinoApp::getInstance()->getMode()) == NeutrinoMessages::mode_radio) && (recordModeActive))
	{
	frameBuffer->paintIcon("radiotextoff.raw", BoxEndX - (ICON_LARGE_WIDTH + 2 + ICON_LARGE_WIDTH + 2 + ICON_SMALL_WIDTH + 2 + ICON_SMALL_WIDTH + 6),BoxEndY + (InfoHeightY_Info - ICON_HEIGHT) / 2);
	}
#endif
Ist so etwa bei Zeile 620 in der infoviewer.cpp.

Zur Lösung von dixidix, ich meine, daß void CInfoViewer::showIcon_RadioText(bool rt_available) const bei Aufnahme nicht angesprungen wird. Hatte ich zuerst auch dort versucht.
Zuletzt geändert von liab am Donnerstag 10. September 2009, 23:55, insgesamt 1-mal geändert.
liab
Einsteiger
Einsteiger
Beiträge: 111
Registriert: Samstag 9. Februar 2008, 15:07

Re: rass und radiotext

Beitrag von liab »

Habe das jetzt länger getestet. Bin begeistert, alles geht wunschgemäß. Wenn noch einer den Radiotext während der Aufnahme hin bekommt ist gut, wenn nicht ist auch gut.
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: rass und radiotext

Beitrag von rhabarber1848 »

rhabarber1848 hat geschrieben:
liab hat geschrieben:Mit automatischer Abschaltung des Radiotextes bei Aufnahme kann man aber auch leben.
Bitte testen: EDIT: Patch ist im CVS
Der Patch ist im CVS:
http://article.gmane.org/gmane.comp.vid ... x.scm/1095

Wenn jemand einen Weg findet, Radiotext und Aufnahmen
miteinander zu versöhnen, können die paar Zeilen des
Patches leicht wieder entfernt werden.

Die Icon-Patches teste ich demnächst.
trulli

Re: rass und radiotext

Beitrag von trulli »

@Rhabarber1848

Hi,

ich muste wegen dem Zapit-Menü und dem SPTS-Fix sowieso neu bauen.

Und da habe ich sofort mal das Radio-Text ausprobieren wollen.

Habe beim configure auch das "--enable-radiotext" hinzugefügt, aber ich kann unter Einstellungen-->Diverse Einstellungen-->Infobar" nichts finden mit Radio-Text.

Was könnte das sein ? Und wie könnte ich herausbekommen, ob er das Radio-Text überhaupt mitgebaut hat ?

MfG
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: rass und radiotext

Beitrag von rhabarber1848 »

trulli hat geschrieben:Und wie könnte ich herausbekommen, ob er das Radio-Text überhaupt mitgebaut hat ?
Erstens in der Zusammenfassung von cdk/configure,
zweitens, existiert apps/tuxbox/neutrino/src/driver/radiotext.o?
trulli

Re: rass und radiotext

Beitrag von trulli »

rhabarber1848 hat geschrieben:
trulli hat geschrieben:Und wie könnte ich herausbekommen, ob er das Radio-Text überhaupt mitgebaut hat ?
Erstens in der Zusammenfassung von cdk/configure,
zweitens, existiert apps/tuxbox/neutrino/src/driver/radiotext.o?
Hi,

also in der configure ist es drin, aber die radiotext.o ist nicht vorhanden.

?????

MfG
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: rass und radiotext

Beitrag von rhabarber1848 »

Hast Du ein leeres Verzeichnis zum Kompilieren genutzt?
Hilft es, apps/tuxbox/neutrino/config.status zu löschen
und nochmals Neutrino zu bauen? Ist dann beim Neutrino
configure die Option --enable-radiotext enthalten?
trulli

Re: rass und radiotext

Beitrag von trulli »

rhabarber1848 hat geschrieben:Hast Du ein leeres Verzeichnis zum Kompilieren genutzt?
Hilft es, apps/tuxbox/neutrino/config.status zu löschen
und nochmals Neutrino zu bauen? Ist dann beim Neutrino
configure die Option --enable-radiotext enthalten?
Hi,

leeres Verzeichniss habe ich genommen und die Option --enable-radiotext ist auch enthalten.

Das mit "config.status" löschen probier ich jetzt mal.

MfG
trulli

Re: rass und radiotext

Beitrag von trulli »

@Rhabarber1848

Hi,

danke für den Tipp.

Nach dem löschen von config.status hat er jetzt korrekt mit Radio-Text gebaut und funktioniert auch.

MfG
seife
Developer
Beiträge: 4189
Registriert: Sonntag 2. November 2003, 12:36

Re: rass und radiotext

Beitrag von seife »

Ich habe den Patch mal updated:
radiotext-ringbuffer.diff
Das "this cannot happten" tritt bei mir sowohl auf der TD als auch auf der dm500 nicht auf. Es ist aber auch nicht zwangsläufig ein Fehler. Insofern habe ich erst mal die Meldung rausgenommen, bis ich es mal auf ner dbox reproduzieren kann.

Das "ringbuffer empty (x < 6)" ist eine debug-Meldung, die aber nur alle paar Sekunden kommen sollte, und deshalb die Performance nicht beeinträchtigen sollte.

Der Ringbuffer ist jetzt 64k, ob der besser 128k gross wäre, müssen Experimente zeigen.
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: rass und radiotext

Beitrag von GetAway »

GetAway hat geschrieben: Nach Wechsel auf TV geht es munter so weiter. :gruebel:

Code: Alles auswählen

RT pes_SyncBufferRead: ringbuffer empty (0 < 6)
RT pes_SyncBufferRead: ringbuffer empty (5 < 6)
RT pes_SyncBufferRead: ringbuffer empty (0 < 6)
RT pes_SyncBufferRead: ringbuffer empty (5 < 6)
RT pes_SyncBufferRead: ringbuffer empty (0 < 6)
RT pes_SyncBufferRead: ringbuffer empty (5 < 6)
Hast Du die Deaktivierung nach Wechsel zum TV Mode hinbekommen?
Kontrollieren, ohne Debugausgabe, kann man es ja jetzt nicht mehr.

Edit:
Mit diesem Patch erscheint Radiotext nicht mehr Zeitgleich mit Aufruf der Infobar.
Erst, unmittelbar nach dieser Ausgabe(alle15-20 Sek.):

Code: Alles auswählen

MEC=0x0a DSN=0x00 PSN=0x00 MEL=65 STATUS=0x01 MFL=69
erscheint der Radiotext, falls das Timeout der Infobar nicht schon zugeschlagen hat.
seife
Developer
Beiträge: 4189
Registriert: Sonntag 2. November 2003, 12:36

Re: rass und radiotext

Beitrag von seife »

GetAway hat geschrieben:
GetAway hat geschrieben: Nach Wechsel auf TV geht es munter so weiter. :gruebel:

Code: Alles auswählen

RT pes_SyncBufferRead: ringbuffer empty (0 < 6)
RT pes_SyncBufferRead: ringbuffer empty (5 < 6)
RT pes_SyncBufferRead: ringbuffer empty (0 < 6)
RT pes_SyncBufferRead: ringbuffer empty (5 < 6)
RT pes_SyncBufferRead: ringbuffer empty (0 < 6)
RT pes_SyncBufferRead: ringbuffer empty (5 < 6)
Hast Du die Deaktivierung nach Wechsel zum TV Mode hinbekommen?
Kontrollieren, ohne Debugausgabe, kann man es ja jetzt nicht mehr.
Bei mir hat es sich immer deaktiviert, das hat ja nur was mit den Änderungen in tvMode() etc zu tun, die schon im CVS sind.
Edit:
Mit diesem Patch erscheint Radiotext nicht mehr Zeitgleich mit Aufruf der Infobar.
Erst, unmittelbar nach dieser Ausgabe(alle15-20 Sek.):

Code: Alles auswählen

MEC=0x0a DSN=0x00 PSN=0x00 MEL=65 STATUS=0x01 MFL=69
erscheint der Radiotext, falls das Timeout der Infobar nicht schon zugeschlagen hat.
Und ohne geht es schneller? Ich ging davon aus, dass es wegen der geringen Datenrate des RT einfach eine Weile dauert, bis Daten kommen. Die Zitierte Ausgabe zeigt an, dass ein Radiotext-Paket empfangen wurde.
An der Anzeige habe ich ja nichts geändert, nur am Einlesen...
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: rass und radiotext

Beitrag von GetAway »

@seife
Du bist nicht Schuld, hatte nur ca. 1 Woche nicht mehr getestet.
Auch ohne Patch wird der Radiotext, also die Infobox, nur noch bei Eintreffen eines
Radiotextdatenpakets überhaupt erst gemalt.

Eigentlich sollten die gepufferten Daten, bei Aufruf der Infobar, sofort angezeigt werden.
Letzte Woche ging es noch. Da ist wohl Müll ins CVS geflossen.

Wie würde sich ein größerer Puffer auswirken. Noch schnelleres zappen?
seife
Developer
Beiträge: 4189
Registriert: Sonntag 2. November 2003, 12:36

Re: rass und radiotext

Beitrag von seife »

Nein, ein grösserer Ringbuffer wäre nur dann sinnvoll, wenn so grosse Pakete ankommen, dass sie nicht in den jetzigen Puffer passen und deswegen Content verloren geht.
liab
Einsteiger
Einsteiger
Beiträge: 111
Registriert: Samstag 9. Februar 2008, 15:07

Re: rass und radiotext

Beitrag von liab »

Bei der infoviewer.cpp v 1.272 fehlt noch nach der Zeile 627

Code: Alles auswählen

	if (((CNeutrinoApp::getInstance()->getMode()) == NeutrinoMessages::mode_radio) && (recordModeActive))
	{
	frameBuffer->paintIcon("radiotextoff.raw", BoxEndX - (ICON_LARGE_WIDTH + 2 + ICON_LARGE_WIDTH + 2 + ICON_SMALL_WIDTH + 2 + ICON_SMALL_WIDTH + 6),BoxEndY + (InfoHeightY_Info - ICON_HEIGHT) / 2);
	}
Sonst fehlt bei aktivem Radiotext und Aufnahme das Radiotext Icon.
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: rass und radiotext

Beitrag von rhabarber1848 »

liab hat geschrieben:Sonst fehlt bei aktivem Radiotext und Aufnahme das Radiotext Icon.
Leicht optimierte Version, bitte testen: EDIT: Patch ist im CVS
Zuletzt geändert von rhabarber1848 am Dienstag 15. September 2009, 15:30, insgesamt 1-mal geändert.
dwilx

Re: rass und radiotext

Beitrag von dwilx »

Mit letzten beiden Patches läuft es scheinbar jetzt am besten. Die könnte man doch eigentlich einchecken. :wink:
seife
Developer
Beiträge: 4189
Registriert: Sonntag 2. November 2003, 12:36

Re: rass und radiotext

Beitrag von seife »

ok. ich will noch ein paar Kleinigkeiten probieren, aber dann checke ich es ein.
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: rass und radiotext

Beitrag von rhabarber1848 »

Irgendwas läuft aber noch nicht ganz rund, ich zappe zwischen
verschiedenen Radiosendern hin und her, wenn ich dann wieder
auf meinen Testsender B5 aktuell zurückschalte, kam zwei Minuten
lang keine Radiotextanzeige, obwohl das RT-Icon sofort gelb wurde.
Nach einem Neutrino-Neustart dauerte es keine 20s, bis der
erste Radiotext von B5 aktuell auf dem Bildschirm war.
Logmeldungen waren vor dem Neustart nicht zu sehen.
Auf HR Info+ kommen folgende Logmeldungen:
RT PES_Receive: start: 1 subend-2-rdsl < 0 (2-2-255)
RT PES_Receive: i < 0 (-1)
RT-Icon ist gelb, es wird kein Radiotext angezeigt.
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: rass und radiotext

Beitrag von rhabarber1848 »

Der Zeitpunkt, an dem das Radiotext-Fenster beim zappen
entfernt wird, sollte früher sein. Wenn es angezeigt wird
und ich umschalte und dabei den FB-Knopf festhalte, sehe
ich in der Infobar die anderen Sender beim Vorbeizappen,
aber das Radiotext-Fenster bleibt solange erhalten, bis ich
den FB-Knopf loslasse.
liab
Einsteiger
Einsteiger
Beiträge: 111
Registriert: Samstag 9. Februar 2008, 15:07

Re: rass und radiotext

Beitrag von liab »

rhabarber1848 hat geschrieben:
liab hat geschrieben:Sonst fehlt bei aktivem Radiotext und Aufnahme das Radiotext Icon.
Leicht optimierte Version, bitte testen: radiotext_record_icon.diff
Kurztest erfolgreich. Herzlichen Dank für Eure gute Arbeit! Nach vielen Jahren mal eine yaad gemacht, ging auf Anhieb.
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: rass und radiotext

Beitrag von rhabarber1848 »

liab hat geschrieben:
rhabarber1848 hat geschrieben:
liab hat geschrieben:Sonst fehlt bei aktivem Radiotext und Aufnahme das Radiotext Icon.
Leicht optimierte Version, bitte testen: EDIT: Patch ist im CVS
Kurztest erfolgreich. Herzlichen Dank für Eure gute Arbeit! Nach vielen Jahren mal eine yaad gemacht, ging auf Anhieb.
committed: http://article.gmane.org/gmane.comp.vid ... x.scm/1119
seife
Developer
Beiträge: 4189
Registriert: Sonntag 2. November 2003, 12:36

Re: rass und radiotext

Beitrag von seife »

rhabarber1848 hat geschrieben:Auf HR Info+ kommen folgende Logmeldungen:
RT PES_Receive: start: 1 subend-2-rdsl < 0 (2-2-255)
RT PES_Receive: i < 0 (-1)
RT-Icon ist gelb, es wird kein Radiotext angezeigt.
Wenn das kommt, ist irgendwas an den Daten kaputt, im Prinzip könnte man bei der ersten Meldung schon raus.
Die 2. Meldung (und der break in derselben Zeile) ist dazu da, einen segfault zu vermeiden (der Arrayindex würde negativ werden). Warum das im Original nicht crashed ist mir unklar, aber jedenfalls liest es im besten Fall Müll ein.

Ich behaupte mal, dass der Dekoder noch nicht ganz Standardkonform ist, ich habe allerdings keine lesbare Spec gefunden und die "Referenzimplementationen" (VDR und Enigma) sind auch nicht so einfach zu durchschauen.