amuttsch / multiqlock

Automatically exported from code.google.com/p/multiqlock
0 stars 0 forks source link

DisplayMatrix swapPages=true only if matrix[0] <> matrix[1] #12

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Bin mir nicht sicher ob es was bringt, aber wenn ihr DisplayMatrix 
überarbeitet zieht es mal in Betracht: 

Aktuell wird ja alle x_updateInterval show***() aufgerufen wo eigentlich immer 
writeMatrix() aufgerufen wird - unabhängig davon ob sich an der Anzeige etwas 
geändert hat. 

Idee ist die 'Multiplexer' nur dann zu ändern wenn sich die Anzeige auch 
wirklich geändert hat und sich dadurch evtl. flackern zu sparen und den 
Prozessor etwas zu entlasten. 

Original issue reported on code.google.com by goebernd...@gmail.com on 11 Jan 2012 at 11:31

GoogleCodeExporter commented 9 years ago
Man könnte dort wirklich prüfen, ob sich was geändert hat. Allerdings würde 
dies nicht viel performancegewinn bringen.

Du hast aber etwas verwechselt, die update***() werden alle x_updateInterval 
aufgerufen, die show**() wird jedes mal aufgerufen (siehe PluginLoader.cpp 
callPluginUpdate). Dies ist für das Original Layout notwendig, da die LEDs 
nicht über den Interrupt angesteuert werden.

Was bei dieser Anpassung passieren würde ist lediglich, dass zwei Statements 
in der iProcess Funktion nicht aufgerufen würden. Ob nun die Daten aus 
matrix[0] oder matrix[1] gelesen werden ist ja für iProcess egal. Was 
wegfallen würde ist folgendes:

if(swapPages)
  {
    currentPage = 1 - currentPage;
    swapPages = false;
  }

Ist dies was du gemeint hast oder habe ich da was falsch verstanden?

Original comment by asfa...@googlemail.com on 12 Jan 2012 at 11:20

GoogleCodeExporter commented 9 years ago
stimmt. vermute auch, dass deswegen das analog-plugin mit original-layout nicht 
funktionierte. 

war nur eine Idee. 
Bitte schließen. 

Original comment by goebernd...@gmail.com on 12 Jan 2012 at 12:02

GoogleCodeExporter commented 9 years ago
Okay, geschlossen

Original comment by asfa...@googlemail.com on 13 Jan 2012 at 9:11