[erledigt] Neutrino (besonders beim Zappen) langsamer

[PauleFoul]

Genau. Mit Uhrzeit. Der Commit, den rhabarber vermutet, wurde abends um Acht eingecheckt. Wenn davor gebaut wurde, war der's sicher nicht.

Ich habe doch leider nur die beiden Stände:

- 21.09.2009 geht
- 11.10.2009 geht nicht Uhrzeit ca. 12:32

Genauer kann ich es leider nicht eingrenzen. Ich bau leider nicht selber, sonst könnte man das ja kurz testen...


Gruß

[rhabarber1848]

- 11.10.2009 geht nicht

Genauer kann ich es leider nicht eingrenzen.

ls -la /bin/neutrino

[seife]

"cat /.version", da steht die genaue Uhrzeit normalerweise mit drin.

[PauleFoul]

- 11.10.2009 geht nicht Uhrzeit 12:32


Gruß

[seife]

Ich setze nen Fünfer auf den Radiotext

[Miky1968]

- 11.10.2009 geht nicht Uhrzeit 12:32
Gruß

Vielleicht hilft es ja

Bei einem Image mit cvs Stand vom 03.11.09 kann ich persönlich keine Veränderung beim zappen feststellen.

Und da meine Antwort ja schön ignoriert wurde, noch einmal die Frage: Alte Zapid eingespielt oder neuer Scan mit dem Image?

[PauleFoul]

Ich setze nen Fünfer auf den Radiotext

Ok, den zahl ich wenn Du recht hast...

[GetAway]

Ich denke auch das die Radiotextimplementierung suboptimal ist.
Habe gestern mal kurz in die infoviewer.cpp geschaut. Soweit ich
mich erinnere, wird schon 3 mal Zapit für den gleichen Status
abgefragt.

[dbt]

Ich denke auch das die Radiotextimplementierung suboptimal ist.
Habe gestern mal kurz in die infoviewer.cpp geschaut. Soweit ich
mich erinnere, wird schon 3 mal Zapit für den gleichen Status
abgefragt.

Es schadet nicht, wenn man das einheitlich macht, aber das haut's auch nicht raus. Den Mode kann man auch von Neutrino abrufen und das passiert hier und da öfter mal.

Code: Alles auswählen

diff --git a/tuxbox/neutrino/src/gui/infoviewer.cpp b/tuxbox/neutrino/src/gui/infoviewer.cpp
index 49fae12..9bd633f 100644
--- a/tuxbox/neutrino/src/gui/infoviewer.cpp
+++ b/tuxbox/neutrino/src/gui/infoviewer.cpp
@@ -975,7 +975,7 @@ void CInfoViewer::showIcon_RadioText(bool rt_available) const
 {
 	if (showButtonBar)
 	{
-		int mode = g_Zapit->getMode();
+		int mode = CNeutrinoApp::getInstance()->getMode();
 		std::string rt_icon = "radiotextoff.raw";
 		if ((!virtual_zap_mode) && (!recordModeActive) && (mode == NeutrinoMessages::mode_radio))
 		{
@@ -991,7 +991,7 @@ void CInfoViewer::showIcon_RadioText(bool rt_available) const
 void CInfoViewer::showIcon_16_9() const
 {
 #ifdef ENABLE_RADIOTEXT
-	if (g_Zapit->getMode() !=2)
+	if (CNeutrinoApp::getInstance()->getMode() != NeutrinoMessages::mode_radio)
 #endif
 	frameBuffer->paintIcon((aspectRatio != 0) ? NEUTRINO_ICON_16_9 : NEUTRINO_ICON_16_9_GREY,
 				BoxEndX - (ICON_LARGE_WIDTH + 2 + ICON_LARGE_WIDTH + 2 + ICON_SMALL_WIDTH + 2 + ICON_SMALL_WIDTH + 6),
@@ -1998,12 +1998,12 @@ No need to poll for EPG, we are getting events from sectionsd!
 void CInfoViewer::showEpgInfo()   //message on event change
 {
 	char nextStart[10];
-	int mode = g_Zapit->getMode();
+	int mode = CNeutrinoApp::getInstance()->getMode();
 	struct tm *pnStartZeit = localtime(&info_CurrentNext.next_zeit.startzeit);
 	sprintf((char*)&nextStart, "%02d:%02d", pnStartZeit->tm_hour, pnStartZeit->tm_min);
 
 	/* show epg info only if we in TV- or Radio mode and current event is not the same like before */
-	if ((eventname != info_CurrentNext.current_name) && (mode !=0))
+	if ((eventname != info_CurrentNext.current_name) && (mode != 0))
 	{
 		eventname = info_CurrentNext.current_name;
 		if (g_settings.infobar_show == 1)

Ich kann nach wie vor keine Änderungen feststellen, bei Sat.
Kann es sein, dass es an den Listen (service.xml usw.) liegt, weils eben scheinbar bei Kabel nur zu passieren scheint?

[Gaucho316]

Mir ist das von PauleFoul beschriebene Verhalten aber noch nicht aufgefallen. Ich nutze den letzen JtG-Snap vom 24.11.2009 (+ einem selbstgestrickten neutrino binary). Und ich habe Kabel und Radiotext ist aktiviert.

[PauleFoul]

Also ich flash jetzt noch einmal neu und mache einen Suchlauf...

Ansonsten drehe ich einen Film von dem Problem und stelle den Online

[PauleFoul]

Problem nach Suchlauf immer noch da. Film folgt...

[Striper]

Hab jetzt mal mit "--disable-radiotext" gebaut und ich würde meinen das das Verhalten vom Infoviewer nun fixer und nicht mehr so träge ist.

[PauleFoul]

Hab mal zwei kurze Videos aufgenommen, an denen man das unterschiedliche Zappverhalten erkennen kann.

http://jtgimage.dyndns.tv/v2.3/neutrino/Videos.zip


Gruß

[PauleFoul]

Hab jetzt mal mit "--disable-radiotext" gebaut und ich würde meinen das das Verhalten vom Infoviewer nun fixer und nicht mehr so träge ist.

Riker hat mit einen Testsnap ohne Radiotext gebaut. Jetzt geht es wieder wie vorher...

Kann man um den Code nicht ein

Code: Alles auswählen

If Mode==Radio

bauen??


Gruß

[seife]

Eigentlich sollte das schon so sein. Wir müssen halt mal rausfinden, wo's nicht richtig ist.

[dbt]

Hab jetzt mal mit "--disable-radiotext" gebaut und ich würde meinen das das Verhalten vom Infoviewer nun fixer und nicht mehr so träge ist.

Riker hat mit einen Testsnap ohne Radiotext gebaut. Jetzt geht es wieder wie vorher...

Kann man um den Code nicht ein

Code: Alles auswählen

If Mode==Radio

bauen??


Gruß

Ja, wie Seife schon sagt, das ist schon drin, hier hatte ich das nur ohne Zapit. Hat das jemand mal auf einer Kabelbox probiert?
http://www.tuxbox-cvs.sourceforge.net/f ... 51#p373051

[trulli]

Hab jetzt mal mit "--disable-radiotext" gebaut und ich würde meinen das das Verhalten vom Infoviewer nun fixer und nicht mehr so träge ist.

Riker hat mit einen Testsnap ohne Radiotext gebaut. Jetzt geht es wieder wie vorher...

Kann man um den Code nicht ein

Code: Alles auswählen

If Mode==Radio

bauen??


Gruß

Hi,

wie wäre es denn mal, ein Image selber zu erstellen, als hier immer mit "schlauen" Vorschlägen zu "glänzen".

Ist ja nicht nur hier in diesem Thread so. Selber nichts auf die Beine stellen, aber immer zu jedem Thema seinen Mist dazu zu geben.

Ist sowieso ein Witz, dass in einem Forum dieser Art hier, jemand Moderator wird, der noch nicht einmal ein Image selber erstellen kann.

MfG

[Miky1968]

Sag mal Trulli, Du hast aber schon mal gaaaanz rechts unter Deinen Namen geschaut?

Vorschlag:

Eigenen Server, eigenes Image und eitel Sonnenschein

Und bevor Du wieder kommst, ich kann auch nicht kompilieren

[GetAway]

Kann man um den Code nicht ein

Code: Alles auswählen

If Mode==Radio

bauen??


Gruß

Eine einzelne globale Abfrage im infoviewer sollte das ganze
auch schon leicht optimieren.

[Striper]

Eine einzelne globale Abfrage im infoviewer sollte das ganze
auch schon leicht optimieren.

So?

Code: Alles auswählen

Index: infoviewer.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/gui/infoviewer.cpp,v
retrieving revision 1.287
diff -u -r1.287 infoviewer.cpp
--- infoviewer.cpp	5 Dec 2009 17:39:33 -0000	1.287
+++ infoviewer.cpp	14 Dec 2009 11:31:48 -0000
@@ -85,6 +85,7 @@
 extern "C" int  tuxtxt_stop();
 #endif
 
+int mode;
 int time_left_width;
 int time_dot_width;
 int time_width;
@@ -109,6 +110,7 @@
 	is_visible       = false;
 	showButtonBar    = false;
 	gotTime          = g_Sectionsd->getIsTimeSet();
+	mode 		 = CNeutrinoApp::getInstance()->getMode();
 	CA_Status        = false;
 	virtual_zap_mode = false;
 }
@@ -625,7 +627,7 @@
 	}
 
 #ifdef ENABLE_RADIOTEXT
-	if (CNeutrinoApp::getInstance()->getMode() == NeutrinoMessages::mode_radio)
+	if (mode == NeutrinoMessages::mode_radio)
 	{
 		if ((g_settings.radiotext_enable) && (!recordModeActive) && (!calledFromNumZap))
 			showRadiotext();
@@ -751,7 +753,7 @@
 				 msg == NeutrinoMessages::SHOW_INFOBAR)
 			{
 #ifdef ENABLE_RADIOTEXT
-				if ((g_settings.radiotext_enable) && (CNeutrinoApp::getInstance()->getMode() == NeutrinoMessages::mode_radio))
+				if ((g_settings.radiotext_enable) && (mode == NeutrinoMessages::mode_radio))
 					hideIt =  true;
 				else
 #endif
@@ -777,7 +779,7 @@
 				show_dot = !show_dot;
 
 #ifdef ENABLE_RADIOTEXT
-				if ((g_settings.radiotext_enable) && (CNeutrinoApp::getInstance()->getMode() == NeutrinoMessages::mode_radio))
+				if ((g_settings.radiotext_enable) && (mode == NeutrinoMessages::mode_radio))
 					showRadiotext();
 #endif
 			}
@@ -975,7 +977,6 @@
 {
 	if (showButtonBar)
 	{
-		int mode = g_Zapit->getMode();
 		std::string rt_icon = "radiotextoff.raw";
 		if ((!virtual_zap_mode) && (!recordModeActive) && (mode == NeutrinoMessages::mode_radio))
 		{
@@ -1276,7 +1277,7 @@
 			if ( is_visible && showButtonBar )
 				showButton_Audio();
 #ifdef ENABLE_RADIOTEXT
-			if (g_settings.radiotext_enable && g_Radiotext && ((CNeutrinoApp::getInstance()->getMode()) == NeutrinoMessages::mode_radio))
+			if (g_settings.radiotext_enable && g_Radiotext && (mode == NeutrinoMessages::mode_radio))
 				g_Radiotext->setPid(g_RemoteControl->current_PIDs.APIDs[g_RemoteControl->current_PIDs.PIDs.selected_apid].pid);
 #endif
 		}
@@ -1998,7 +1999,6 @@
 void CInfoViewer::showEpgInfo()   //message on event change
 {
 	char nextStart[10];
-	int mode = g_Zapit->getMode();
 	struct tm *pnStartZeit = localtime(&info_CurrentNext.next_zeit.startzeit);
 	sprintf((char*)&nextStart, "%02d:%02d", pnStartZeit->tm_hour, pnStartZeit->tm_min);
 

[seife]

das wird
a) nicht funktionieren (weil mode nur einmal zugewiesen wird, im Konstruktor, wenn ich das recht sehe
b) nicht viel bringen. CNeutrinoApp::getInstance()->getMode() *sollte* nicht wirklich viel teurer sein, weil das auch nur innerhalb des Prozesses auf den Speicher zugreift. Wichtig ist eher unnötige daemon-Kommunikation zu minimieren.

Am wichtigsten wäre es aber, erst mal rauszufinden, wo es genau hängt, also welcher Teil nun länger braucht. Das kann aber nur Paule, weil es anscheinend nur bei ihm auftritt. Da er aber nicht selbst bauen will, wird das Problem wohl nicht gefixt werden.

[Striper]

a) nicht funktionieren (weil mode nur einmal zugewiesen wird, im Konstruktor, wenn ich das recht sehe

Dachte mir schon fast das das nicht geht so. Naja bin ein HW-Fuzzi...

Am wichtigsten wäre es aber, erst mal rauszufinden, wo es genau hängt, also welcher Teil nun länger braucht. Das kann aber nur Paule, weil es anscheinend nur bei ihm auftritt. Da er aber nicht selbst bauen will, wird das Problem wohl nicht gefixt werden.

Naja, ich merk hier schon auch einen Unterschied. Zwar nicht so drastisch, aber immerhin. Das Problem ist doch mittlerweile recht gut eingegrenzt. Der "g_Zapit->getMode()" scheint wohl ein bissl lahm zu sein. Wenn ich heute Abend dazu komme werd ich mal die Änderungen die dbt vorgeschlagen hat testen. Wenn du noch Tips dazu hast, immer her damit.

[seife]

ok, gib einfach an vielen stellen im Infoviewer genaue timestamps aus (im sectionsd gibts ne routine, die man sich da klauen kann und schaue, was genau lange dauert.

*Eigentlich* dürfte zapit->GetMode() nicht langsam sein, aber vielleicht dauert es eine weile, bis zapit wieder scheduled wird, oder bis die daten durch den socket sind.

Im Infoviewer sollte man den zapit aber auch kaum nach dem mode fragen müssen, denn Neutrino weiss selber ob es im radiomode ist, also zapit->getmode durch neutrino->getinstance()->getmode ersetzen (und schauen, ob die dazugehörigen Zahlen noch passen). Evtl. bringt es das schon.

Damals beim sectionsd waren es auch 1000 Sachen, die "eigentlich" nichts hätten ausmachen dürfen ;-)

[PauleFoul]

Das kann aber nur Paule, weil es anscheinend nur bei ihm auftritt. Da er aber nicht selbst bauen will, wird das Problem wohl nicht gefixt werden.

Naja, von wollen kann man hier wohl nicht sprechen

Ich versuche jedoch so gut es geht zu helfen...


Gruß